Opened 15 years ago

Closed 12 years ago

Last modified 12 years ago

#15 closed defect (fixed)

yuv4mpeg output aspect fixed at 0:0

Reported by: giles@… Owned by: reimar
Priority: normal Component: vo
Version: HEAD Severity: normal
Keywords: Cc:
Blocked By: Blocking:
Reproduced by developer: Analyzed by developer:

Description

Files created with '-vo yuv4mpeg' always have the aspect ratio field of the
header set to 0:0. A quick look at vo_yuv4mpeg.c show this to be hard-coded in
the output.

This field should instead contain the pixel aspect ratio of the output video, if
it is known.

Attachments (1)

y4m_asp.diff (1.3 KB) - added by reimar 12 years ago.
Set aspect in -vo yuv4mpeg

Download all attachments as: .zip

Change History (6)

comment:1 Changed 14 years ago by moritz@…

As far as I know MPlayer only handles display aspect ratio down the line, so I'm
not certain if these fields COULD be filled with "correct" values. Any ideas?

Changed 12 years ago by reimar

Set aspect in -vo yuv4mpeg

comment:2 Changed 12 years ago by reimar

Please check if this patch works and sets the correct value.

comment:3 Changed 12 years ago by reimar

  • Owner changed from moritz@… to Reimar.Doeffinger@…
  • Status changed from new to assigned

comment:4 Changed 12 years ago by reimar

  • Resolution set to fixed
  • Status changed from assigned to closed

Applied, thus fixed in SVN r20415

comment:5 Changed 12 years ago by giles@…

excellent, thanks!

Note: See TracTickets for help on using tickets.