[FFmpeg-devel] [PATCH] Rename put_sbit() params

Stefano Sabatini stefano.sabatini-lala
Sun Nov 29 10:51:36 CET 2009


On date Saturday 2009-11-28 12:05:37 +0100, Michael Niedermayer encoded:
> On Sat, Nov 28, 2009 at 11:43:34AM +0100, Stefano Sabatini wrote:
> > On date Saturday 2009-04-11 12:05:32 +0200, Stefano Sabatini encoded:
> > > Hi,
> > > 
> > > this should make more apparent the similarities between put_bits() and
> > > put_sbits().
> > > 
> > > Decrease the user sense of wonder when reading the declaration.
> > 
> > Patch updated.
> > -- 
> > FFmpeg = Foolish and Fundamentalist Magic Philosofic Extreme Glue
> 
> >  put_bits.h |    6 +++---
> >  1 file changed, 3 insertions(+), 3 deletions(-)
> > b0bf9d0b47ef3c8b818a2fb49b62ab5caa2ab830  bs-rename-put-sbits-params.patch
> > Index: ffmpeg/libavcodec/put_bits.h
> > ===================================================================
> > --- ffmpeg.orig/libavcodec/put_bits.h	2009-11-28 11:40:12.000000000 +0100
> > +++ ffmpeg/libavcodec/put_bits.h	2009-11-28 11:40:52.000000000 +0100
> > @@ -257,11 +257,11 @@
> >  }
> >  #endif
> >  
> > -static inline void put_sbits(PutBitContext *pb, int bits, int32_t val)
> > +static inline void put_sbits(PutBitContext *s, int n, int32_t value)
> 
> pb is better than s

I agree, but then all the other functions use 's' for the putbit
context, would be to change them globally be an option? (I suppose
that is overkill).

Regards.
-- 
FFmpeg = Fundamentalist Forgiving Mega Plastic Ecumenical Gorilla



More information about the ffmpeg-devel mailing list