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

Re: [Xen-devel] [PATCH 0/2] Convert libxl__sprintf(gc) to GCSPRINTF

To: Wei Liu <wei.liu2@xxxxxxxxxx>, Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>
From: David Vrabel <david.vrabel@xxxxxxxxxx>
Date: Tue, 17 Nov 2015 15:34:30 +0000
Cc: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>, Ian Campbell <ian.campbell@xxxxxxxxxx>
Delivery-date: Tue, 17 Nov 2015 15:34:37 +0000
List-id: Xen developer discussion <xen-devel.lists.xen.org>

On 17/11/15 15:08, Wei Liu wrote:
> There are mixed usage of libxl__sprintf(gc) and GCSPRINTF. Convert all
> libxl__sprintf to GCSPRINTF.

This looks like pointless churn /and/ GCSPRINTF is /less/ readable than
libxl__sprintf(gc,...) because the reader now has to also look up what
GCSPRINTF does.

David

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel

Follow-Ups:
- Re: [Xen-devel] [PATCH 0/2] Convert libxl__sprintf(gc) to GCSPRINTF
  - From: Wei Liu

References:
- [Xen-devel] [PATCH 0/2] Convert libxl__sprintf(gc) to GCSPRINTF
  - From: Wei Liu

Prev by Date: [Xen-devel] [qemu-mainline test] 64450: regressions - FAIL
Next by Date: [Xen-devel] [PATCH] xen/events: Always allocate legacy interrupts on PV guests
Previous by thread: Re: [Xen-devel] [PATCH 1/2] libxl: replace libxl__sprintf(gc, ...) with GCSPRINTF
Next by thread: Re: [Xen-devel] [PATCH 0/2] Convert libxl__sprintf(gc) to GCSPRINTF
Index(es):
- Date
- Thread

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.