Messages in this thread | | | Subject | Re: [PATCH] MIPS: kernel: proc: Remove spurious white space in cpuinfo | From | Joe Perches <> | Date | Fri, 20 Oct 2017 16:09:00 -0700 |
| |
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.
| |