[FFmpeg-devel] avcodec : add prores_metadata bsf and fate test

James Almer jamrial at gmail.com
Mon Nov 12 00:25:52 EET 2018


On 11/11/2018 7:20 PM, James Almer wrote:
> On 11/8/2018 6:31 PM, Martin Vignali wrote:
>> Le dim. 28 oct. 2018 à 12:59, Martin Vignali <martin.vignali at gmail.com> a
>> écrit :
>>
>>> Thanks for the comments
>>>
>>> New patch in attach
>>>
>>> - Change the options, to only authorized few value in prores frame (based
>>> on rdd36).
>>> - Change option name to match setparams filter
>>> - Include intreadwrite instead of bytestream
>>> - Do not modify the property if set to -1
>>> - Move av_packet_make_writable at the start of the func
>>>
>>> Martin
>>>
>>>
>>>
>> i will pushed in few days, if no one is against.
>>
>> Martin
> 
> This broke fate.
> 
> --- /home/fate/ffmpeg/tests/ref/fate/prores-metadata	2018-11-11
> 20:20:19.486707760 +0000
> +++ tests/data/fate/prores-metadata	2018-11-11 20:23:35.356712028 +0000
> @@ -1 +1 @@
> -84814393e25b673e8ebd6a94f3d07349
> +b4e2c801d594b9614433b284b886be0d

Ah, nevermind, i missed you pushed a fix.


More information about the ffmpeg-devel mailing list