[x264-devel] Re: x264: svn commit r618 (pengvado)

[Loren Merritt]

On Fri, 12 Jan 2007, Loïc Le Loarer wrote:

> Hi,
>
> The Friday 12 January 2007 à 00:55:52 +0100, Subversion daemon wrote:
>> r618 | pengvado | 2007-01-12 00:55:51 +0100 (Fri, 12 Jan 2007) | 2 lines
>> make array_non_zero() compatible with -fstrict-aliasing
>>
>> > http://trac.videolan.org/x264/changeset/618
>
> +#define array_non_zero(a) array_non_zero_int(a, sizeof(a))
> +static inline int array_non_zero_int( void *v, int i_count )
>
> On the computers I have access to (one linux i386, one linux amd64 and
> one sparc 64), with gcc compiler, sizeof returns a value of type size_t,
> not int. size_t is unsigned and 32 bits on 32 bits computers and 64 bits
> on 64 bits computers.

So what? Both are integers, the difference only matters if the size of the 
array doesn't fit in an int. x264 doesn't allocate 2GB of memory. The 
arrays in question aren't even 1KB.

--Loren Merritt