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

Nicolas George george at nsup.org
Thu Mar 9 01:07:38 EET 2017


L'octidi 18 ventôse, an CCXXV, Hendrik Leppkes a écrit :
> 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.

Changing the types like that will not guarantee that all architectures
have have the same struct size, it will only make it happen because all
32-bits architectures are basically identical and all 64-bits
architectures too.

If we want a guarantee, we need to consider side data as an octet buffer
where fields must be explicitly serialized and deserialized. But I think
this would be piling worse design on top of bad design.

In the long run, we should either get rid of side data entirely or turn
it into something actually useful. Right now, it only brings complexity
with zero benefit.

In the short run, I would say that printing the side data size in
ffprobe is wrong: this size is an artifact of FFmpeg's internal data
structures, not a property of the file, printing it makes as much sense
as printing sizeof(AVPacket) instead of packet->size.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170309/9433bb77/attachment.sig>


More information about the ffmpeg-devel mailing list