[FFmpeg-devel] [PATCHv3] lavf: add av_guess_frame_sample_aspect_ratio function

Michael Niedermayer michaelni at gmx.at
Mon May 7 23:03:30 CEST 2012

On Mon, May 07, 2012 at 08:39:32PM +0200, Joakim Plate wrote:
> Just thought i'd chime in here. We (xbmc) ended up using that special
> handling of stream aspect being 1:1 and codec aspect not being 1:1 to
> solve a few samples we found in the wild. Thing the where mp4's
> recorded by some camera if i remember correctly. So i'm sort of in
> favor of that solution even if it's ugly.

iam in favor of whatever works best and is simple

in that sense, does anyone know how common in the wild files are that
change the sample aspect ratio midstream but have a valid and not 1:1
sample aspect at the format level ?

I also think its better to commit early and improve than discuss
such patches for ever ...

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The misfortune of the wise is better than the prosperity of the fool.
-- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120507/999357fa/attachment.asc>

More information about the ffmpeg-devel mailing list