Re: [PATCH 1/2] regulator: whitespace

[Michael Jones]

On 07/26/2012 05:15 PM, Mark Brown wrote:
> On Thu, Jul 26, 2012 at 05:12:59PM +0200, Michael Jones wrote:
> > On 07/26/2012 04:44 PM, Mark Brown wrote:
>
> > > Your changelog says "whitespace" and is otherwise blank...  what is the
> > > problem you think you are fixing here?
>
> > I don't see what you're seeing.
>
> I'm not seeing anything at all.  That's the problem.
>
> > Here:
> > https://lkml.org/lkml/2012/7/26/255
> > I see the patch which replaces non-uniform spaces with tabs.
>
> ...and only has a one word changelog.

Sorry, I misunderstood.  I thought you saw _only_ a one word change log, 
and not even the patch.  I think I answered your question above now, right?
Q: what is the problem you think you are fixing here?
A: some lines were indented with spaces, while most of them are indented 
with tabs.

So I suppose the changelog should have been something like:
regulator: whitespace- indent with tabs not spaces

MATRIX VISION GmbH, Talstrasse 16, DE-71570 Oppenweiler
Registergericht: Amtsgericht Stuttgart, HRB 271090
Geschaeftsfuehrer: Gerhard Thullner, Werner Armingeon, Uwe Furtner, Erhard Meier
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/