WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-ia64-devel

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

To: "Tristan Gingold" <Tristan.Gingold@xxxxxxxx>, "Xu, Anthony" <anthony.xu@xxxxxxxxx>, <xen-ia64-devel@xxxxxxxxxxxxxxxxxxx>
Subject: RE: [Xen-ia64-devel] "-Wall" option doesn't work
From: "Tian, Kevin" <kevin.tian@xxxxxxxxx>
Date: Thu, 12 Jan 2006 16:44:40 +0800
Delivery-date: Thu, 12 Jan 2006 08:51:46 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-ia64-devel-request@lists.xensource.com?subject=help>
List-id: Discussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>
List-post: <mailto:xen-ia64-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcYXU5eMGx7WqdX2S+aQmZrCTZUJTQAAFOIg
Thread-topic: [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