Re: [PATCH 1/1] net: i10e: use matching format indentifiers

From: David Miller
Date: Sun Jul 31 2016 - 23:48:26 EST

From: Heinrich Schuchardt <xypron.glpk@xxxxxx>
Date: Sun, 31 Jul 2016 11:39:28 +0200

> i is defined as int but output as %u several times.
> Change the definition to unsigned.
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@xxxxxx>

I hate changes like this.

The canonical way to loop over a value is to use a signed integer,
named 'i', and go:

for (i = 0; i < X; i ++)

'i' can never be negative, therefore we know that this usage
is safe.

I'd rather you convert the "%u" to "%d".

I'm very disappointed in your patches, and it is clear you
are just finding the easiest and simplest way to shut up
compiler warnings you are seeing rather than analyzing the
code you are making changes to in order to determine what
the best and most appropriate fix might be.