Thread (5 messages) 5 messages, 2 authors, 2017-06-02

Fixing warnings

From: valdis.kletnieks at vt.edu <hidden>
Date: 2017-06-02 21:56:38

On Sat, 03 Jun 2017 02:59:56 +0530, srishti sharma said:
Thanks a lot !
Also, large parts of the kernel have maintainers who do *not* want to
see checkpatch fixes, because there's 2 basic possibilities:

1) The code is being actively worked on by others as well, in which case
your patches can cause merge conflicts for no real benefit.

2) The code isn't being worked on, which means your patches (if incorrect)
may cause code instability for no real benefit.

End result is that most of the kernel only wants style fixups as part of
a series of actual code patches - if you're fixing bugs or redoing an
algorithm, then send style patches as part of the series.

The rules are different for anything under drivers/staging - all code in there
is assumed to be manure that needs fixing, so feel free to fix it. :)

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 486 bytes
Desc: not available
Url : http://lists.kernelnewbies.org/pipermail/kernelnewbies/attachments/20170602/fd366067/attachment.bin 
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help