[FFmpeg-devel] fate: Do not report side data size
wm4
nfxjfg at googlemail.com
Wed Mar 8 13:56:31 EET 2017
On Wed, 8 Mar 2017 12:51:06 +0100
Michael Niedermayer <michael at niedermayer.cc> wrote:
> On Wed, Mar 08, 2017 at 12:28:17PM +0100, Hendrik Leppkes wrote:
> > On Wed, Mar 8, 2017 at 1:17 AM, Vittorio Giovara
> > <vittorio.giovara at gmail.com> wrote:
> > > This should address the mismatch between different archs
>
> iam not in favor of this solution
>
>
> >
> > Removing the side_data_size from output should be fine, as its a
> > implementation detail and as seen here can even vary between
> > architecture or possibly even compiler.
> > Maybe someone that uses that ffprobe output more often can comment?
>
> I use all kinds of stuff
> if something is removed from ffprobes output it wont be tested anymore.
> We should test more not less.
>
>
> >
> > An alternative for fixing fate would be to use fixed size fields in
> > the new sidedata, although the possibility of it breaking similarly
> > again in the future then remains.
>
> I strongly prefer fixed size to be used in side data over platform
> dependant fields. Not only does size become testable but theres also
> a platform specific difference less in the interface which should
> help bug reproducability between platforms
>
> thanks
>
> [...]
So why don't let we fate test e.g. sizeof(AVPacket)? Makes as much
sense.
More information about the ffmpeg-devel
mailing list