Felipe Contreras <felipe.contreras@xxxxxxxxx> writes: > + if (HAS_MULTI_BITS(types)) { > + error("only one type at a time."); > + usage_with_options(builtin_config_usage, builtin_config_options); > + } You would be able to catch "config --bool --get-color color.diff.whitespace" as an error by doing something similar to the "fake actions" for get-color* as I suggested in my review on [3/8]. I did not find anything objectionable in the remainder of the series. Thanks. -- 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