Messages in this thread | | | Subject | Re: Spurious warning in checkpatch.pl? | From | Joe Perches <> | Date | Fri, 12 Oct 2012 11:38:34 -0700 |
| |
On Fri, 2012-10-12 at 13:23 -0500, Larry Finger wrote: > Andy, > > The checkpatch.pl version in mainline is issuing what I think are false warnings > of the type > > WARNING: networking block comments put the trailing */ on a separate line > #93: FILE: drivers/net/wireless/rtlwifi/wifi.h:208: > + u32 rf_rb; /* rflssi_readback */ > > Is a trailing comment for the member of a struct not allowed?
It's allowed. Maybe even better, encouraged.
It's a change I made in commit 058806007450 ("checkpatch: check networking specific block comment style") without sufficient testing.
I'll see about fixing it.
| |