[FFmpeg-devel] Broken compile with latest libavutil/common.h

[Michael Niedermayer]

On Fri, Jan 16, 2009 at 12:53:24AM +0100, Aurelien Jacobs wrote:
> Michael Niedermayer wrote:
[...]
> > [...]
> > > 
> > > > +/* math */
> > > > +int64_t av_const ff_gcd(int64_t a, int64_t b);
> > > 
> > > The function is in mathematics.c.  Maybe mathematics.h would be a good
> > > place for this prototype.  
> > 
> > > Maybe we should even make that function
> > > public.
> > 
> > I think so too
> 
> Done in attached patch.

ok


> 
> > > > +/**
> > > > + * converts fourcc string to int
> > > > + */
> > > > +static inline av_pure int ff_get_fourcc(const char *s){
> > > > +    assert( strlen(s)==4 );
> > > > +    return (s[0]) + (s[1]<<8) + (s[2]<<16) + (s[3]<<24);
> > > > +}
> > > 
> > > This looks a lot like AV_RL32().  
> > 
> > indeed and i wouldnt mind seeing all ff_get_fourcc replaced by it
> 
> See attached patch.

ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

When you are offended at any man's fault, turn to yourself and study your
own failings. Then you will forget your anger. -- Epictetus
-------------- 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/20090117/29aadfd4/attachment.pgp>