[FFmpeg-devel] [PATCH] add FFARRAY_SIZE macro

Michael Niedermayer michaelni
Thu Oct 2 20:58:08 CEST 2008


On Wed, Oct 01, 2008 at 10:37:53PM +0100, M?ns Rullg?rd wrote:
> Michael Niedermayer <michaelni at gmx.at> writes:
> 
> > esOn Mon, Sep 29, 2008 at 07:04:59PM +0200, Aurelien Jacobs wrote:
> >> Benoit Fouet wrote:
> >> 
> >> > hi,
> >> > 
> >> > patch to $subj attached
> >> 
> >> That's at least the third time such a patch is proposed. It seems to be
> >> a quite popular request :-)
> >> IIRC, last time this came up, Michael was not happy with the name of
> >> the macro. Well, another popular name for this macro is COUNTOF().
> >> So maybe attached patch would be acceptable ?
> >
> > iam not happy about this one either, to see why, just try to form a
> > english sentence with it
> > the size of an array (in bytes?)
> > the length of an array (in bytes?)
> > the count of arrays
> >
> > what my problem is with the suggestions is just that they subtetly mean
> > something ambigous or wrong compared to what they actually are.
> >
> > i think
> > entry/item/element/cell count/of/in
> >
> > are maybe better choices
> 
> FF_ARRAY_ELEMS?

its certainly better than the other suggestions ...


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

Why not whip the teacher when the pupil misbehaves? -- Diogenes of Sinope
-------------- 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/20081002/6ac58358/attachment.pgp>



More information about the ffmpeg-devel mailing list