On Wed, Nov 23, 2016 at 09:32:57AM +0100, Anton Khirnov wrote: > Quoting Diego Biurrun (2016-11-23 09:23:10) > > On Wed, Nov 23, 2016 at 09:15:06AM +0100, Anton Khirnov wrote: > > > Quoting Diego Biurrun (2016-11-22 15:17:22) > > > > On Tue, Nov 22, 2016 at 02:36:31PM +0100, Anton Khirnov wrote: > > > > > --- > > > > > doc/bitstream_filters.texi | 3 +++ > > > > > libavcodec/Makefile | 1 + > > > > > libavcodec/bitstream_filters.c | 1 + > > > > > libavcodec/null_bsf.c | 44 > > > > > ++++++++++++++++++++++++++++++++++++++++++ > > > > > 4 files changed, 49 insertions(+) > > > > > create mode 100644 libavcodec/null_bsf.c > > > > > > > > Why? > > > > > > Why what? > > > > Why add a null bitstream filter? It's completely unclear why you are > > adding a component that does nothing. > > We have similar muxers/demuxers/filters etc., it's a useful thing to > have in certain situations. > > It's in this set because it is used in 6/9.
Please add an explanatory line to the log message then. Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
