On Sun, 07 Oct 2007 23:59:56 -0700 (PDT)
David Miller <davem@davemloft.net> wrote:

quoted text
> From: Stephen Hemminger <shemminger@linux-foundation.org>
> Date: Fri, 5 Oct 2007 17:14:07 -0700
> 
> > Fix a bunch of sparse warnings. Mostly about 0 used as
> > NULL pointer, and shadowed variable declarations.
> > 
> > The two noteable changes are:
> >  * hash size should have been unsigned
> >  * cls_u32 had a case where error code wasn't been
> >    propogated properly, so it could return 0 but still
> >    have an error.
> > 
> > Signed-off-by: Stephen Hemminger <shemminger@linux-foundation.org>
> 
> Applied, and I stuffed the cls_u32 fix into net-2.6 as well.
> 
> But please fix the following, this is the second time I've
> had to remove this bit from one of your patches.
> 
> > @@ -6,7 +6,7 @@
> >   *
> >   * Alexey Kuznetsov  <kuznet@ms2.inr.ac.ru>
> >   * Ben Greear <greearb@candelatech.com>
> > - * Jens Låås <jens.laas@data.slu.se>
> > + * Jens Låås <jens.laas@data.slu.se>
> >   *
> >   * This program is free software; you can redistribute it and/or
> >   * modify it under the terms of the GNU General Public License
> 
> Please please please please, for the second time, fix the encoding
> your editor uses when you save copies of this source file, this hunk
> of your patches never applies cleanly.
> 

It's either an Emacs or quilt problem. Probably emacs.

-- 
Stephen Hemminger <shemminger@linux-foundation.org>
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html