Messages in this thread | | | Date | Tue, 19 Mar 2019 17:36:38 +0100 | From | Peter Zijlstra <> | Subject | Re: [PATCH v2 05/19] locking/lockdep: Adjust indents for function definitions |
| |
On Tue, Mar 19, 2019 at 09:33:00AM -0700, Bart Van Assche wrote: > On Mon, 2019-03-18 at 16:57 +0800, Yuyang Du wrote: > > Being paranoid to see function arguments lines are aligned. > > This patch changes code that conforms to the kernel coding style into > code that does not conform to the kernel coding style. If you have a > look at the c-lineup-arglist-tabs-only function in > Documentation/process/coding-style.rst you will see that only tabs and > no spaces should be used to indent function arguments.
So while I generally do align things with tabs and spaces, but have no strong opinions either way as longs as its readable; I do hate pure whitespaces patches. They make git-blame so much harder to use :/
| |