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/
Home Products Support Community News


Re: [Xen-devel] [PATCH] make shadow code compile ok on FC4 which is usin

To: David Hopwood <david.nospam.hopwood@xxxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] make shadow code compile ok on FC4 which is using gcc4
From: Rik van Riel <riel@xxxxxxxxxx>
Date: Mon, 11 Jul 2005 16:01:59 -0400 (EDT)
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Mon, 11 Jul 2005 20:00:44 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <42D2C65E.5000601@xxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <16A54BF5D6E14E4D916CE26C9AD30575029F0BB7@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <42D2C65E.5000601@xxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
On Mon, 11 Jul 2005, David Hopwood wrote:
> Li, Xin B wrote:
> > We need this patch to make shadow code compile ok on FC4 which is using
> > gcc4.
> Given gcc4's propensity to warn about things that are often harmless, is 
> it still a good idea to be compiling with -Werror?

Fixing a compile warning is very little effort, but hunting
the occasional (1 in 50?) real bug can be extremely time

I certainly don't mind fixing an error when I run into one...

The Theory of Escalating Commitment: "The cost of continuing mistakes is
borne by others, while the cost of admitting mistakes is borne by yourself."
  -- Joseph Stiglitz, Nobel Laureate in Economics

Xen-devel mailing list