Thread (43 messages) 43 messages, 3 authors, 2021-08-25

Re: [PATCH] USB: serial: iuu_phoenix: fix quoted string split across lines

From: Utkarsh Verma <hidden>
Date: 2021-08-06 13:12:26

On Thu, Aug 05, 2021 at 04:20:56PM +0200, Lukas Bulwahn wrote:
On Thu, Aug 5, 2021 at 3:53 PM Dwaipayan Ray [off-list ref] wrote:
quoted
On Tue, Aug 3, 2021 at 7:51 PM Utkarsh Verma [off-list ref] wrote:
quoted
Fix checkpatch warning "quoted string split across lines".
You could write a line about why this change is required.
And that there are no functional changes.
... and does this change actually increase the line length beyond 80,
or even 100 characters?

So, did you run checkpatch on your patch?
Yes, I did run the checkpatch on my patch, and all the warnings
regarding "quoted string split across lines" were removed.
Although the line length increased to more than 100 characters, there were
no warnings from the checkpatch.
If you are just swapping one violation of one rule by the violation of
another rule, you will need to make a judgement which rule is more
important and provide a good reasoning for that.
Since there were no "LONG_LINE" warnings, so I didn't mention about it
in the commit description. But will rectify this in v2.
Other than that, it looks good to me.

Lukas
_______________________________________________
Linux-kernel-mentees mailing list
Linux-kernel-mentees@lists.linuxfoundation.org
https://lists.linuxfoundation.org/mailman/listinfo/linux-kernel-mentees
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help