On Tue, Jul 17, 2012 at 10:57 AM, Christophe Fergeau <cfergeau@xxxxxxxxxx> 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
