Re: [PATCH] MIPS: kernel: proc: Remove spurious white space in cpuinfo
From: Joe Perches
Date: Fri Oct 20 2017 - 19:09:11 EST
On Fri, 2017-10-20 at 22:46 +0100, Maciej W. Rozycki wrote:
> On Fri, 20 Oct 2017, Joe Perches wrote:
>
> > > > That's somewhat unpleasant code as it formats a fmt string
> > > > and the compiler can not verify fmt and args.
> > > >
> > > > Perhaps something like the below is preferable:
> > >
> > > Hmm, what problem exactly are you trying to solve with code that has
> > > worked just fine for 16 years now?
> >
> > The compiler cannot verify fmt and args.
>
> You have stated that already. Why is that a problem?
Jeeze, perhaps you don't like the word perhaps.
There is no absolute defect here.
There are unnecessary pushes to stack that are
unwound by the compiler.
Stylistically, format/argument mismatches can
cause errors. It's
generally bad form and error
prone to use non constant strings as
formats.
Note it's not signed and is a simple suggestion.
If you don't like it, don't do anything with it.