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

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



Hi Anthony,
I am also working on this issue now. Fixing these warnings as Tristan said is 
very boring indeed. Since Tristan have made the patch for it, and I won't spend 
more time on this again. OK? :)

Zhang Xiantao
CSD-OTC PRC Virtualization 
Intel (China) Limited 

-----Original Message-----
From: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx 
[mailto:xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx] On Behalf Of Tian, Kevin
Sent: 2006年1月12日 16:45
To: Tristan Gingold; Xu, Anthony; xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
Subject: RE: [Xen-ia64-devel] "-Wall" option doesn't work

>From: Tristan Gingold
>Sent: 2006年1月12日 17:37
>I have already sent two patches enabling warnings.
>
>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.

Hi, Tristan,
        I think above patches to fix warning are very important and always 
welcomed for stability. Maybe Dan was in busy with other more important threads 
at time you sent out them. Could you re-send them out?

>
>For sure, fixing warnings is *very* boring.  Some are in fact bugs: function
>called with less arguments, very dubious casts...
>I really think warnings must be enabled and the earlier is the better.
>
>Tristan.

Absolutely agree.

Thanks,
Kevin

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

_______________________________________________
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®.