[FFmpeg-devel] Bug report : build problem with "configure --disable-encoders"

Michael Niedermayer michaelni
Wed Nov 26 03:09:32 CET 2008


On Wed, Nov 26, 2008 at 01:50:15AM +0100, Diego Biurrun wrote:
> On Tue, Nov 25, 2008 at 06:24:07PM +0100, Michael Niedermayer wrote:
> > On Tue, Nov 25, 2008 at 09:29:28AM +0100, David Geldreich wrote:
> > >
> > > --- ./libavcodec/aandcttab.c	2008-11-25 09:18:43.000000000 +0100
> > > +++ /Users/dgeld/src/ffmpeg/libavcodec/aandcttab.c	2008-11-25 09:18:50.000000000 +0100
> > > @@ -23,7 +23,7 @@
> > >  
> > > -const uint16_t ff_inv_aanscales[64] = {
> > > +const uint16_t ff_aan_inv_scales[64] = {
> > 
> > i would prefer if you wouldnt rename ff_inv_aanscales at least not in the
> > same patch that adds ff_ to aanscales
> 
> Do you agree that ff_aan_inv_scales is a better name, yes or no?  If
> yes, I will rename it, if no we will just leave it as it is.

ive no strong oppinion on this but to me inv_aanscales looks nicer than
aan_inv_scales ...

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The misfortune of the wise is better than the prosperity of the fool.
-- Epicurus
-------------- 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/20081126/1328f044/attachment.pgp>



More information about the ffmpeg-devel mailing list