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

RE: [Xen-devel] [PATCH][XM] Extend width of ID to 5.


  • To: "Otavio Salvador" <otavio@xxxxxxxxxxxxxxxx>
  • From: "Petersson, Mats" <Mats.Petersson@xxxxxxx>
  • Date: Fri, 20 Apr 2007 15:13:49 +0200
  • Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
  • Delivery-date: Fri, 20 Apr 2007 06:13:06 -0700
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: AceDRpOnOvKplK1zT5uXpU2qiqvOsgABqkjA
  • Thread-topic: [Xen-devel] [PATCH][XM] Extend width of ID to 5.

 

> -----Original Message-----
> From: Otavio Salvador [mailto:otavio@xxxxxxxxxxxxxxxx] 
> Sent: 20 April 2007 13:22
> To: Petersson, Mats
> Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
> Subject: Re: [Xen-devel] [PATCH][XM] Extend width of ID to 5.
> 
> "Petersson, Mats" <Mats.Petersson@xxxxxxx> writes:
> 
> > Attached (trivial) patch extends the width to 5 digits (and 
> changes some
> > %-ns to %ns, as the ID for example is right justified, %-ns is
> > left-justified, which didn't look too good). 
> 
> The only thing it can mess is automated tests and routines that uses
> the fixed width and will need a fix after this patch gets in.

Yes, of course, but such scrips would also be broken when the numbers
get too big to fit in the fields, so those scripts are already possibly
broken. Any script that parses the output of a program (and wishes to
remain functional after revisions of the software) should use flexible
field parsing rather than column-grabbing, in my opinion. 

But it's also why I suggested that it may go into unstable only, rather
than into the near-release 3.0.5 tree. 

--
Mats
> 
> -- 
> Otavio Salvador                  O.S. Systems
> E-mail: otavio@xxxxxxxxxxxxxxxx  http://www.ossystems.com.br
> Mobile: +55 53 9981-7854         http://projetos.ossystems.com.br
> 
> 
> 



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

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