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/

Home

xen-api

[Top] [All Lists]

Re: [Xen-API] [PATCH] imported patch removing-warnings

from [Jonathan Knowles]

[Permanent Link][Original]

To:	Vincent Hanquez <Vincent.Hanquez@xxxxxxxxxxxxx>
Subject:	Re: [Xen-API] [PATCH] imported patch removing-warnings
From:	Jonathan Knowles <jonathan.knowles@xxxxxxxxxxxxx>
Date:	Wed, 10 Nov 2010 14:37:55 +0000
Cc:	"xen-api@xxxxxxxxxxxxxxxxxxx" <xen-api@xxxxxxxxxxxxxxxxxxx>
Delivery-date:	Wed, 10 Nov 2010 06:38:05 -0800
Envelope-to:	www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to:	<4CDA94FA.1020605@xxxxxxxxxxxxx>
List-help:	<mailto:xen-api-request@lists.xensource.com?subject=help>
List-id:	Discussion of API issues surrounding Xen <xen-api.lists.xensource.com>
List-post:	<mailto:xen-api@lists.xensource.com>
List-subscribe:	<http://lists.xensource.com/mailman/listinfo/xen-api>, <mailto:xen-api-request@lists.xensource.com?subject=subscribe>
List-unsubscribe:	<http://lists.xensource.com/mailman/listinfo/xen-api>, <mailto:xen-api-request@lists.xensource.com?subject=unsubscribe>
References:	<403764544b97988f2e3d.1289323667@rok-desktop> <4CDA61E7.5060509@xxxxxxxxxxxxx> <B462D1536FED1140871BC97AB218A6598EFA3A1CC7@xxxxxxxxxxxxxxxxxxxxxxxxx> <4CDA8827.1060607@xxxxxxxxxxxxx> <B462D1536FED1140871BC97AB218A6598EFA3A1CCA@xxxxxxxxxxxxxxxxxxxxxxxxx> <4CDA94FA.1020605@xxxxxxxxxxxxx>
Sender:	xen-api-bounces@xxxxxxxxxxxxxxxxxxx
User-agent:	Mozilla/5.0 (X11; U; Linux i686; en-GB; rv:1.9.2.9) Gecko/20100915 Lightning/1.0b2 Thunderbird/3.1.4

On 10/11/10 12:50, Vincent Hanquez wrote:

yes, the partial application would not have triggered either when it's
linked to a name, however you do remove potential audit site (warning
lines), since "unused variable" warnings has been silenced.

I think this is a good point, and one that I agree with.
Silencing the warnings (by replacing unused identifiers
with "_") means that we'll no longer be able to use the
compiler as a tool for discovery of these sites.

Cheers,
Jonathan

_______________________________________________
xen-api mailing list
xen-api@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/mailman/listinfo/xen-api

[More with this subject...]

<Prev in Thread]	Current Thread	[Next in Thread>
RE: [Xen-API] [PATCH] imported patch removing-warnings, (continued) RE: [Xen-API] [PATCH] imported patch removing-warnings, Rok Strnisa Re: [Xen-API] [PATCH] imported patch removing-warnings, Vincent Hanquez RE: [Xen-API] [PATCH] imported patch removing-warnings, Rok Strnisa Re: [Xen-API] [PATCH] imported patch removing-warnings, Jonathan Knowles Re: [Xen-API] [PATCH] imported patch removing-warnings, Rok Strnisa Re: [Xen-API] [PATCH] imported patch removing-warnings, Jonathan Knowles RE: [Xen-API] [PATCH] imported patch removing-warnings, Dave Scott Re: [Xen-API] [PATCH] imported patch removing-warnings, Jonathan Knowles Re: [Xen-API] [PATCH] imported patch removing-warnings, Jonathan Knowles Re: [Xen-API] [PATCH] imported patch removing-warnings, Vincent Hanquez Re: [Xen-API] [PATCH] imported patch removing-warnings, Jonathan Knowles <= RE: [Xen-API] [PATCH] imported patch removing-warnings, Rok Strnisa

Previous by Date:	Re: [Xen-API] [PATCH] imported patch removing-warnings, Jonathan Knowles
Next by Date:	RE: [Xen-API] [PATCH] imported patch removing-warnings, Rok Strnisa
Previous by Thread:	Re: [Xen-API] [PATCH] imported patch removing-warnings, Vincent Hanquez
Next by Thread:	RE: [Xen-API] [PATCH] imported patch removing-warnings, Rok Strnisa
Indexes:	[Date] [Thread] [Top] [All Lists]

This site is hosted by Citrix