[FFmpeg-devel] [PATCH] add FFARRAY_SIZE macro

[Benoit Fouet]

Michael Niedermayer wrote:
> On Mon, Oct 13, 2008 at 01:49:51PM +0200, Benoit Fouet wrote:
>   
>> 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
>>     
>
> ok
>
>
> [...]
>
>   

applied

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