[virt-tools-list] [PATCH virt-viewer 06/19] Turn display:show-hint into flags type

[Christophe Fergeau]

On Tue, Jul 17, 2012 at 12:59:22PM +0200, Marc-André Lureau wrote:
> On Tue, Jul 17, 2012 at 10:59 AM, Christophe Fergeau
> <cfergeau at redhat.com> wrote:
> > On second thought, shouldn't this be if (!(hint & _HINT_READY)) { } ? I'm
> > not sure which flags you are going to add.
> 
> VIRT_VIEWER_DISPLAY_SHOW_HINT_HIDE used to be an enum of value 0, now
> there is only a READY flag. The following patch will make this easier
> to read.

I've assumed that _READY/_HIDE were just 2 named constant for the same
concept, one name for TRUE, another one for FALSE, hence my question. If
they are 2 different things, then it would be less confusing to have a HIDE
flag as well. Though it's not really important as it's indeed changed in
one of the next patches.

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/191c5b1c/attachment.sig>