On Wed, 8 Mar 2017 12:51:06 +0100 Michael Niedermayer <mich...@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.giov...@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. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
