Re: [PATCH 2/2] python: Fix -Wsign-compare

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




----- Original Message -----
> On Wed, 2014-09-03 at 06:01 -0400, Marc-André Lureau wrote:
> > Can limit be negative?
> 
> I don't think so: http://paste.stg.fedoraproject.org/4623/39072140/
> 
> > Can limit be made unsigned instead?
> 
> Actually, what I'm doing is making the limit unsigned, thanks for
> catching the error in the commit log :-)
> I'll rewrite the log and re-submit the patch.

No, I mean the argument passed as the limit to for_loop(self, index, limit)

ex: *__nelements, n_ptr etc

If those value can be negative, we need a check before entering the for loop with the (unsigned) cast.
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel





[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]