On Fri, 28 Sep 2012, Glauber Costa wrote:
> One nitpick:
>
> >
> > @@ -185,26 +169,19 @@ static __always_inline void *kmalloc_nod
> > struct kmem_cache *cachep;
> >
> > if (__builtin_constant_p(size)) {
> > - int i = 0;
> > + int i;
> >
>
> Although this is technically correct, the former is correct as well, and
> this end up only adding churn to the patch.
What does the "former" refer to?
> Should you decide to remove it, there is another instance of this a bit
> more down.
Remove what?
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@xxxxxxxxx. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>