On Wed, Jan 11, 2017 at 6:10 PM, Diego Biurrun <[email protected]> wrote: > From: Alexandra Hájková <[email protected]> > > --- > libavcodec/tak.c | 49 +++++++-------- > libavcodec/tak.h | 10 +-- > libavcodec/tak_parser.c | 15 ++--- > libavcodec/takdec.c | 158 > ++++++++++++++++++++++++------------------------ > libavformat/takdec.c | 13 ++-- > 5 files changed, 125 insertions(+), 120 deletions(-) > > -static int get_bits_esc4(GetBitContext *gb) > +static int get_bits_esc4(BitstreamContext *bc) > { > - if (get_bits1(gb)) > - return get_bits(gb, 4) + 1; > + if (bitstream_read_bit(bc)) > + return bitstream_read(bc, 4) + 1; > else > return 0; > }
should this function get a rename too? something that is unrelated to bitstream_ and at the same time doesn't use the old prefix, like "bits_esc4"? maybe it could be done later it should be ok if you don't want to rename it now -- Vittorio _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
