[virt-tools-list] [PATCH virt-viewer 13/19] Number display starting from 1

[Christophe Fergeau]

On Tue, Jul 17, 2012 at 01:16:19PM +0200, Marc-André Lureau wrote:
> On Tue, Jul 17, 2012 at 11:46 AM, Christophe Fergeau
> <cfergeau at redhat.com> wrote:
> > Why? To match the window subtitle? Please add this to the commit log.
> 
> It just is more user friendly, no? I think human start counting from 1
> in general.

I don't disagree with the change, just curious as to why it's needed since
the commit log doesn't mention it.

> 
> > Id add a +1 as well to
> > DEBUG_LOG("Remove window %d %p", nth, win);
> > and
> > DEBUG_LOG("Insert window %d %p", nth, win);
> 
> However, this is not meant to be user friendly, but rather developper
> friendly, so we should just print the value of these variable, imho

This means that when the GUI talks about display 1 and 2, and the debug log
talks about window 1, you have to know it's really about display 2, imo
this is not developer friendly.

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/virt-tools-list/attachments/20120717/6eb49d8b/attachment.sig>