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

[Adam Heath <doogie@xxxxxxxxxxxxx>]

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.
I wrote a script that I used when I was working on kaffe.  It would scan 
the build output, and sort warnings based on the -W option that produces 
them.  It'd then give output sorted by warning option, and file.  Made 
it much easier to work with.
I should put that up someplace.

Was really handy when a warning was in some .h file, so would be shown 
several thousand times in the build log, but was really only one problem.
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel