Re: [PATCH spice-common] test-quic: Fix -Wsign-compare warning

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

 



On 31/05/18 17:44, Frediano Ziglio wrote:
>>
>> ../tests/test-quic.c: In function ‘gdk_pixbuf_new_random’:
>> ../tests/test-quic.c:205:19: warning: comparison between signed and unsigned
>> integer expressions [-Wsign-compare]
>>      for (i = 0; i < gdk_pixbuf_get_byte_length(random_pixbuf); i++) {
>>                    ^
>>
>> Signed-off-by: Eduardo Lima (Etrunko) <etrunko@xxxxxxxxxx>
>> ---
>>  tests/test-quic.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/tests/test-quic.c b/tests/test-quic.c
>> index 7ac00f5..2a7ff3d 100644
>> --- a/tests/test-quic.c
>> +++ b/tests/test-quic.c
>> @@ -197,7 +197,7 @@ static GdkPixbuf *gdk_pixbuf_new_random(void)
>>      gint width = g_random_int_range(100, 2000);
>>      gint height = g_random_int_range(100, 2000);
>>      GdkPixbuf *random_pixbuf;
>> -    gint i;
>> +    guint i;
>>      guint8 *pixels;
>>  
>>      random_pixbuf = gdk_pixbuf_new(GDK_COLORSPACE_RGB, has_alpha, 8, width,
>>      height);
> 
> Acked, never negative and used for indexing an array.
> 

Pushed, Thanks.



-- 
Eduardo de Barros Lima (Etrunko)
Software Engineer - RedHat
etrunko@xxxxxxxxxx
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://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]