[FFmpeg-devel] [PATCH 1/2] lavc/mpegvideo_parser: export additional properties

Rodger Combs rodger.combs at gmail.com
Thu Apr 14 02:43:55 CEST 2016


This seems to be determined by whether information is copied from the codec context to codecpar for the last time before or after the input file's aspect ratio changes. I'm not sure what the "right" value to return here is, nor why this is different with this patch.

> On Apr 13, 2016, at 06:28, Michael Niedermayer <michael at niedermayer.cc> wrote:
> 
> On Wed, Apr 13, 2016 at 03:26:03AM -0500, Rodger Combs wrote:
>> ---
>> libavcodec/mpegvideo_parser.c | 67 +++++++++++++++++++++++++++++++++++++++++++
>> 1 file changed, 67 insertions(+)
> 
> is this intended to change aspect of some files ?
> like
> http://samples.ffmpeg.org/MPEG-VOB/transport/test16.ts
> 
> testcase that shows this clearly:
> ./ffmpeg -i test16.ts -vf 'scale=iw/ih*oh*sar:288,pad=512:288:(ow-iw)/2:(oh-ih)/2:blue' aspectchangetoconstantres-rcombs.ts
> 
> [...]
> -- 
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
> 
> No snowflake in an avalanche ever feels responsible. -- Voltaire
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 3579 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160413/546276a0/attachment.bin>


More information about the ffmpeg-devel mailing list