Thread (47 messages) flat view 47 messages, 2 authors, 2021-09-10

Re: [PATCH 31/31] staging: wfx: indent functions arguments

From: Kari Argillander <hidden>
Date: 2021-09-10 17:28:43
Also in: lkml, netdev

On Fri, Sep 10, 2021 at 07:12:28PM +0200, Jérôme Pouiller wrote:
On Friday 10 September 2021 18:57:43 CEST Kari Argillander wrote:
quoted
On Fri, Sep 10, 2021 at 06:05:04PM +0200, Jerome Pouiller wrote:
quoted
From: Jérôme Pouiller <jerome.pouiller@silabs.com>

Function arguments must be aligned with left parenthesis. Apply that
rule.
To my eyes something still go wrong with this patch. Might be my email
fault, but every other patch looks ok. Now these are too left.
I don't try anymore to check alignments with my email viewer. The
original patch is as I expect (and I take care to send my patch with
base64 to avoid pitfalls with MS Exchange). So, I think the is correct.
It was correct. Nice to now know about that something funny is happening
with my part.
quoted
Also it should alight with first argument not left parenthesis?
Absolutely.


-- 
Jérôme Pouiller
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help