On Thu, 16 Jun 2005, Andrew Morton wrote:
> Jesper Juhl <[email protected]> wrote:
> >
> > I send in the patch below a while back but never recieved any response.
> > Now I'm resending it in the hope that it might be added to -mm.
>
> There are surely many warnings in the tree, hence I'm not really interested
> in patches which only fix `gcc -W' warnings.
>
Ok, in that case I won't bother you directly with such patches any more
but instead let them trickle into maintainers trees when they will take
them.
And yes, I know it's very trivial stuff and it doesn't make much of a
difference to the "big picture", but my attitude towards that is that no
issue is too small to be addressed, and since I'm not able to adress many
of the larger issues I try to address the smaller ones that I'm able to
handle, and when I run out of those I start nitpicking with the really
trivial stuff (like gcc -W warnings) - all with the purpose of helping our
kernel be the very best it can, even if my contribution might be very
minor in some cases.
> How many are there?
>
With the .config I use here a regular build gives me 10 warnings. A build
with gcc -W of the same config gives me 100177 warnings.
> > It looks to me like a significantly large 'len' passed in could cause the
> > loop to never end. Isn't it safer to make 'i' an unsigned long as well?
>
> Nope. All operations which mix signed and unsigned types promote the
> signed type to unsigned.
>
Hmm, right, then the only bennefit of the patch as-is is to silence the
gcc -W warning. But since it can be done 100% safe and the change to use
an unsigned value for the counter is (at least to me) the logical and
obviously correct thing to do, I still think the patch has merrit as a
purely "for pedantic correctness" fix.
--
Jesper
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
