On Thu, Jan 12, 2012 at 08:20:02AM -0700, Eric Blake wrote:
On 01/12/2012 05:49 AM, Peter Krempa wrote:
From: "Daniel P. Berrange"<berrange@redhat.com>
Domain IDs are at least 16 bits for most hypervisors, theoretically event 32-bits. 3 characters is clearly too small an alignment. Increase alignment to 10 characters to allow 32-bit domain IDs to display cleanly. Commonly seen with LXC where domain IDs are the process IDs by default. Also increase the 'name' field from 20 to 30 characters to cope with longer guest names which are quite common
In v2:
- Increase size to 10 chars - Left align ID values - Also increase name field to 30 - Increase number of '-' to compensate
--- I think that 10 characters is a lot of space wasted in most of cases :/. What about calculating the alignment dynamicaly? I'm adding code to the
On 01/12/2012 01:19 PM, Daniel P. Berrange wrote: list command, so I could add that as a part of my patch.
A generic helper function for dynamically sizing columns would indeed be nice, but I think that can be a follow up. It would require callers to allocate a char*** - an array rows to be formatted, where each row is represented by an array of columns.
I'm okay with Daniel's patch going in now, as it is a strict improvement, even if we make further improvements down the road.
I will reduce the 'id' column back down to '5' for now, since there is not an immediate critical need to cope with the full 32bits. Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|