[FFmpeg-devel] [PATCH] add FFARRAY_SIZE macro

[Benoit Fouet]

Michael Niedermayer wrote:
> 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 ...
>
>
>   

patch attached

-- 
Benoit Fouet
Purple Labs S.A.
www.purplelabs.com

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: common.h.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081013/427a6cea/attachment.txt>