[vlc-devel] vlc: svn commit r24680 (funman)

[Rafaël Carré]

Le Tue, 29 Jan 2008 15:26:16 +0100,
Remi Denis-Courmont <rdenis at simphalempin.com> a écrit :

> 
> On Tue, 29 Jan 2008 14:34:31 +0100 (CET), Subversion daemon
> <svn at videolan.org> wrote:
> >     use size_t instead of int to represent sizes
> 
> Mixing int and size_t is wrong:
> It should be ssize_t if you need to support negative values. ssize_t
> is bigger than int on, e.g. amd64.

The point is that negative values aren't supported.

> Second, there are places where size_t does not make much sense, and
> should have unsigned.
> picture dimensions for instance??
> 

Isn't *size*_t suited for the *size* of a picture ?

-- 
Rafaël Carré
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20080129/1e5fe387/attachment.sig>