On Mon, Aug 4, 2014 at 6:34 PM, Martin Storsjö <[email protected]> wrote: > > I think we rather should make sure we really initialize all of the side data > in the case of STEREO3D - then it's easier to spot missed initialization of > other side data types, while this patch would hide other such issues. > > OTOH, implicitly initializing all frame side data might be a useful detail > (and could be regarded as a safety fallback - if someone misses to > initialize something it will at least behave consistently).
My personal preference would be the latter: for a simple side data like stereo3d it might be enough to initialize 'flags' to zero, but there might be future side data that are more complex to initialize and would require a memset anyway. -- Vittorio _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
