On Mon, Feb 27, 2012 at 08:17:37PM +0100, Anton Khirnov wrote: > In most places where it's used, it's as a pointless write-only field. > > Only rv10 decoder actually reads from it, but it stores some internal > version info in it. There is no reason for it to be in a public field. > --- > libavcodec/avcodec.h | 11 ++++------- > libavcodec/mpeg12.c | 4 ---- > libavcodec/mpegaudiodec.c | 2 -- > libavcodec/mpegaudiodecheader.c | 1 - > libavcodec/mpegvideo.h | 2 ++ > libavcodec/mpegvideo_parser.c | 2 -- > libavcodec/options.c | 2 ++ > libavcodec/pthread.c | 1 - > libavcodec/rv10.c | 18 +++++++++--------- > libavcodec/version.h | 3 +++ > 10 files changed, 20 insertions(+), 26 deletions(-)
LGTM _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
