[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-ia64-devel] "-Wall" option doesn't work



Le Jeudi 19 Janvier 2006 23:06, Magenheimer, Dan (HP Labs Fort Collins) a 
écrit :
> > The first one was very big (140KB) and Dan rejected it almost
> > for this reason.
> >
> > The second one was rather small (4KB) and fixed the most
> > boring warnings (the
> > most obvious and those in .h files which appear in every
> > file). It was not
> > accepted (without explaination).
> >
> > I am not sure Dan is interested in warnings.
>
> I forgot to discuss this at the summit.   Kevin is correct that I
> didn't accept the large 140KB patch because it changed so many
> files, including many linux and linux-xen files, at a time when
> the tree was not very stable.  I do agree that removing warnings
> is a good idea as they often hide real bugs.  However sometimes
> fixing them requires changes that might introduce new bugs.
>
> I had intended to commit the smaller patch some time ago, but
> forgot.   I committed it after Tristan resubmitted it.
>
> Now that -Wall is on and the tree is fairly stable, it might
> be a good time to fix more warnings, especially in the xen
> and vmx directories.  I would like to hold patches that fix
> warnings in the arch/ia64/linux and arch/ia64/linux-xen directories
> until we rebase to 2.6.15.
Thank you.
This is clear and makes sense.

Tristan.



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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.