Messages in this thread | | | From | Denis Efremov <> | Subject | Re: sysfs output without newlines | Date | Sat, 29 Aug 2020 23:23:39 +0300 |
| |
Hi,
On 8/29/20 9:23 PM, Joe Perches wrote: > While doing an investigation for a possible treewide conversion of > sysfs output using sprintf/snprintf/scnprintf, I discovered > several instances of sysfs output without terminating newlines. > > It seems likely all of these should have newline terminations > or have the \n\r termination changed to a single newline.
I think that it could break badly written scripts in rare cases.
> > Anyone have any objection to patches adding newlines to these > in their original forms using sprintf/snprintf/scnprintf?
I'm not sure about existing cases, but I think it's a good checkpatch.pl warning for new patches. It should be possible to check sysfs_emit() calls.
Thanks, Denis
| |