Opened 20 years ago

Closed 18 years ago

Last modified 18 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: no Analyzed by developer: no

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 18 years ago.
Set aspect in -vo yuv4mpeg

Download all attachments as: .zip

Change History (6)

comment:1 by moritz@…, 20 years ago

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?

by reimar, 18 years ago

Attachment: y4m_asp.diff​ added

Set aspect in -vo yuv4mpeg

comment:2 by reimar, 18 years ago

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

comment:3 by reimar, 18 years ago

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

comment:4 by reimar, 18 years ago

Resolution: → fixed
Status: assigned → closed

Applied, thus fixed in SVN r20415

comment:5 by giles@…, 18 years ago

excellent, thanks!

Note: See TracTickets for help on using tickets.