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

Re: [Xen-devel] [PATCH v6 20/23] libxlu: record line and column number when parsing values

To: Wei Liu <wei.liu2@xxxxxxxxxx>
From: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Date: Fri, 6 Mar 2015 11:36:03 +0000
Cc: Ian Campbell <ian.campbell@xxxxxxxxxx>, xen-devel@xxxxxxxxxxxxx
Delivery-date: Fri, 06 Mar 2015 11:36:13 +0000
List-id: Xen developer discussion <xen-devel.lists.xen.org>

Wei Liu writes ("[PATCH v6 20/23] libxlu: record line and column number when 
parsing values"):
> Originally only setting has line number recorded. Since we're moving to
> more sophisticated API, record line number and column number for
> individual value. They are useful for error reporting.

This is a good idea.

Why not use the flex location type in your formal parameters
everywhere ?  That would mean plumbing one rather than two values.

Ian.

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

Follow-Ups:
- Re: [Xen-devel] [PATCH v6 20/23] libxlu: record line and column number when parsing values
  - From: Wei Liu

Prev by Date: Re: [Xen-devel] [PATCH OSSTEST] allow: Allow rumpuserxen-demo-xenstorels/xenstorels failures on all branches
Next by Date: Re: [Xen-devel] [PATCH v2] credit: generalize __vcpu_has_soft_affinity()
Previous by thread: [Xen-devel] [PATCH] libxc: use xc_dom_panic when decompressor is not supported
Next by thread: Re: [Xen-devel] [PATCH v6 20/23] libxlu: record line and column number when parsing values
Index(es):
- Date
- Thread

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