[FFmpeg-devel] [PATCH] add FFARRAY_SIZE macro
Michael Niedermayer
michaelni
Wed Oct 1 23:10:19 CEST 2008
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
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Its not that you shouldnt use gotos but rather that you should write
readable code and code with gotos often but not always is less readable
-------------- 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/20081001/93f55edd/attachment.pgp>
More information about the ffmpeg-devel
mailing list