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

Marc-André Lureau marcandre.lureau at gmail.com
Tue Jul 17 10:57:24 UTC 2012


On Tue, Jul 17, 2012 at 10:57 AM, Christophe Fergeau
<cfergeau at redhat.com> wrote:
> I would have kept HINT_HIDE as the flag name as it's more
> expressive than _READY imo. Looks good either way.

"HIDE" isn't necessarily 0 anymore, there will be a DISABLED flag in a
following patch. Now that you say it, it could have been "ENABLED"
flag instead, so 0 would still map to !READY && !ENABLED, but I prefer
to DISABLED, since this is the exception, and we might had more flags
in the future, making 0 hard to keep meaningful perhaps.

-- 
Marc-André Lureau




More information about the virt-tools-list mailing list