Quoting wm4 (2014-11-03 16:33:07) > On Mon, 03 Nov 2014 12:32:13 +0100 > Anton Khirnov <an...@khirnov.net> wrote: > > > Quoting Luca Barbato (2014-10-29 16:55:31) > > > On 29/10/14 16:46, Anton Khirnov wrote: > > > > It is required by (at least) the ogg demuxer. > > > > > > > > Mark the current semi-public apriv API for removal. > > > > --- > > > > doc/APIchanges | 3 +++ > > > > libavcodec/Makefile | 12 ++++++---- > > > > libavcodec/version.h | 4 ++-- > > > > libavcodec/vorbis_parse.h | 58 > > > > ++++++++++++++++++++++++++++++++++++++++++++++ > > > > libavcodec/vorbis_parser.c | 52 > > > > ++++++++++++++++++++++++++++++++++++----- > > > > libavcodec/vorbis_parser.h | 7 ++++-- > > > > 6 files changed, 121 insertions(+), 15 deletions(-) > > > > create mode 100644 libavcodec/vorbis_parse.h > > > > > > > > > > It is really specific, why making it public? > > > > > > > So it can be used in lavf properly. We've already done the same thing in > > other places (e.g. DV). > > > > You should at least put a warning into the public header, which should > indicate that the API might be removed in the future. (Or do you really > want to maintain this essentially forever?)
I don't see why not. It should require little to no maintenance. And the ogg demuxer won't just stop requiring this. -- Anton Khirnov _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel