[LTP] [PATCH] typo: Add necessary spaces in the outputs
Petr Vorel
pvorel@suse.cz
Wed Jun 16 13:38:47 CEST 2021
> Hi Ziyao,
> > Hi!
> > Ping. I checked it based on the latest code. Are there any changes that need
> > to be added?
> Well, as I note previously on some places it'd be better to join string, e.g.
> testcases/kernel/syscalls/setsid/setsid01.c
> testcases/open_posix_testsuite/conformance/interfaces/sched_setparam/2-1.c
> testcases/kernel/syscalls/sysinfo/sysinfo02.c
> The reason is that string on single line is easier to grep.
> That's why checkpatch.pl from kernel relaxed max line from 80 to 100
> (good at least for string).
And because this I'd personally go for joining string most of the time, even
long strings like
testcases/kernel/syscalls/sched_get_priority_max/sched_get_priority_max01.c
Kind regards,
Petr
> But I'm ok to merge it without.
> Kind regards,
> Petr
> > Kind Regards,
> > Ziyao
More information about the ltp
mailing list