On Wed, Mar 16, 2016 at 05:37:03PM -0400, Eric Sunshine wrote: > A much easier solution would be to update OPT_VERBOSE() to understand > that negative values are "unspecified", and then --verbose would > (pseudocode): > > if (value < 0) > value = 0 > value++; > > and --no-verbose would: > > value = 0 > > That should be compatible with existing clients of OPT__VERBOSE() > which initialize the value to 0, and should satisfy Pranit's case; he > can initialize it to -1, and if it is still -1 when option parsing is > done, then he knows that neither --verbose nor --no-verbose was seen. Yes, that makes much more sense to me. Thanks for the back-story. -Peff -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html