Re: [Xen-devel] Re: [PATCH 01/04] Kexec / Kdump: Generic code

[Horms <horms@xxxxxxxxxxxx>]

On Wed, Oct 18, 2006 at 08:13:49AM +0100, Keir Fraser wrote:
> On 18/10/06 1:56 am, "Horms" <horms@xxxxxxxxxxxx> wrote:
> 
> >> I'll check this in, so please just get rid of the if(xchg()) hacky
> >> workaround.
> > 
> > I have to say that I entirely disagree with your solution.
> > I think the solution is to get rid of -Werror. But I will
> > remove the if(xchg()) foo as you request.
> 
> Holy crap! Getting rid of Werror makes Wall much less useful. We're too lazy
> to scan build output for warnings if they don't break the build.
> 
> The GCC developers have basically broken the unused-value checking, at least
> for Linux-style code which is heavy on macro usage. If a particular warning
> option is no longer useful, the correct answer is to switch it off.

I'm quite happy with that argument. However I personally prefer
a severity differenation between warnings and errors, that is all.

-- 
Horms
  H: http://www.vergenet.net/~horms/
  W: http://www.valinux.co.jp/en/


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel