[FFmpeg-devel] [PATCH] move put_v from nutenc.c and make it public in avio.h and aviobuf.c

[Michael Niedermayer]

On Thu, Jul 08, 2010 at 12:17:05AM +0100, M?ns Rullg?rd wrote:
> Michael Niedermayer <michaelni at gmx.at> writes:
> 
> > On Wed, Jul 07, 2010 at 11:00:56PM +0100, M?ns Rullg?rd wrote:
> >> Michael Chinen <mchinen at gmail.com> writes:
> >> 
> >> > --- a/libavformat/avio.h
> >> > +++ b/libavformat/avio.h
> >> > @@ -352,12 +352,12 @@ void put_strz(ByteIOContext *s, const char *buf);
> >> >  /**
> >> >   * Put val using a variable number of bytes.
> >> >   */
> >> > -void put_v(ByteIOContext *bc, uint64_t val);
> >> > +void ff_put_v(ByteIOContext *bc, uint64_t val);
> >> >  
> >> >  /**
> >> >   * Get the length in bytes which is needed to store val as v.
> >> >   */
> >> > -int get_length(uint64_t val);
> >> > +int ff_get_v_length(uint64_t val);
> >> 
> >> Since this is a public header, wouldn't an av_ prefix be appropriate?
> >
> > i dont think we are planing to use them from outside lavf so it should be ff_
> 
> They seem useful enough, but whatever.

i dont mind making them public if someone wants to use them
iam not aware of anyone wanting that though

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Many things microsoft did are stupid, but not doing something just because
microsoft did it is even more stupid. If everything ms did were stupid they
would be bankrupt already.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100708/5b8424ef/attachment.pgp>