On Sat, May 2, 2015 at 5:23 AM, Anton Khirnov <[email protected]> wrote: > Quoting Vittorio Giovara (2015-05-02 01:17:08) >> The enum is used by lavc, lavf and lavfi, and it is referenced by lavu, >> so it sementically belongs to lavu more than any other. >> >> This change allows to drop an avcodec.h inclusion from avfilter.h. >> > > I would disagree here, since this logic would apply to any side data > struct whatsoever. And I don't think they should all be in lavu.
Why? It only applies to side data that is shared between packets and frames (which is already fine since lavu is a dep of lavc anyway). Also, this creates an unnecessary compile time dependency between lavfi and lavc too. -- Vittorio _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
