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

xen-devel

[Top] [All Lists]

Re: [Xen-devel] [PATCH] libxl: new xlu_disk_parse function

To: Stefano Stabellini <Stefano.Stabellini@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] libxl: new xlu_disk_parse function
From: Ian Campbell <Ian.Campbell@xxxxxxxxxx>
Date: 2011年3月29日 10:10:44 +0100
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>, Gianni Tedesco <gianni.tedesco@xxxxxxxxxx>
Delivery-date: 2011年3月29日 02:12:06 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <alpine.DEB.2.00.1103281909590.5516@kaball-desktop >
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/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Organization: Citrix Systems, Inc.
References: <1300988989-18305-1-git-send-email-ian.jackson@xxxxxxxxxxxxx> <1300988989-18305-2-git-send-email-ian.jackson@xxxxxxxxxxxxx> <1301333477.1691.7.camel@xxxxxxxxxxxxxxxxxxxxxx> <alpine.DEB.2.00.1103281909590.5516@kaball-desktop >
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Please trim your quotes guys!
On Mon, 2011年03月28日 at 19:13 +0100, Stefano Stabellini wrote:
> On 2011年3月28日, Gianni Tedesco wrote:
> Besides when doing refactoring the code produced should be either shorter or
> at least easier to read but this code is neither of them.
Without reference to this specific patchset I disagree with "shorter"
here. The important things are clarity, maintainability, correctness etc
(and possibly performance depending on circumstances).
Brevity is a poor indicator for any of these (or anything really, unless
you are playing Perl golf) although it may be a partial factor in some
of them.
It is perfectly possible that a refactoring doubles the number of lines
of code yet is still a clear improvement.
Ian.
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
<Prev in Thread] Current Thread [Next in Thread>
Previous by Date: Re: [Xen-devel] [PATCH 5 of 5] tools: ocaml: rename nic_remove to nic_del, for consistency with libxl , Ian Campbell
Next by Date: Re: [Xen-devel] Segfault When Building Snapshot xen-3d14ce7 kernel From linux/kernel/git/jeremy/xen.git/shortlog , Ian Campbell
Previous by Thread: Re: [Xen-devel] [PATCH] libxl: new xlu_disk_parse function , Stefano Stabellini
Next by Thread: Re: [Xen-devel] [PATCH] libxl: new xlu_disk_parse function , Ian Jackson
Indexes: [Date] [Thread] [Top] [All Lists]

Copyright ©, Citrix Systems Inc. All rights reserved. Legal and Privacy
Citrix This site is hosted by Citrix

AltStyle によって変換されたページ (->オリジナル) /