lkml.org 
[lkml]   [2014]   [Jul]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2] declance: Fix 64-bit compilation warnings
On Sat, 5 Jul 2014, Joe Perches wrote:

> > > I don't think %#p is valid so it
> > > shouldn't have been set by #.
> >
> > Huh? As recently as last Wednesday you pointed me at the specific commit
> > from Grant that made it valid (GCC format complaints aside).
>
> Those gcc complaints are precisely the thing
> that makes it invalid.

So enforce that in code then, clear the SPECIAL flag where appropriate
and do not try to handle it in one place while leaving other ones to
behave randomly (i.e. a supposedly fixed field width varies depending on
the two uppermost digits). Please note that it's only your proposed
change that introduces that randomness, right now code does what's
supposed and documented to, except a bit inconsistently.

> I believe you're tilting at windmills.
>
> Hey, it works sometimes. Knock yourself out.

I pointed out an inconsistency with the intent to propose a fix once a
consensus have been reached, one way or another. And I think shifting the
inconsistency to a different place, which is what your proposal does,
isn't really a complete solution, although I do recognise the improvement.

Maciej


\
 
 \ /
  Last update: 2014-07-05 23:21    [W:0.212 / U:0.332 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site