[FFmpeg-devel] fate: Do not report side data size

Hendrik Leppkes h.leppkes at gmail.com
Thu Mar 9 00:54:59 EET 2017


On Wed, Mar 8, 2017 at 3:42 PM, Ronald S. Bultje <rsbultje at gmail.com> wrote:
> Hi,
>
> On Wed, Mar 8, 2017 at 9:31 AM, wm4 <nfxjfg at googlemail.com> wrote:
>
>> On Wed, 8 Mar 2017 14:09:53 +0100
>> Michael Niedermayer <michael at niedermayer.cc> wrote:
>>
>> If the size printing is removed then other code should be added
>> > to test for the size to match the correct value
>>
>> Then it would be more reasonable to make av_packet_add_side_data()
>> check whether the size is correct for the given side data type.
>
>
> I think you're both right here, this is a pretty good idea (for fixed-size
> side-data types).
>

So how do we fix fate now? Change the datatypes to uint32_t, remove
the size print out?
Shouldn't keep all 32-bit fate clients broken for much longer.

- Hendrik


More information about the ffmpeg-devel mailing list