On Fri, Aug 14, 2015 at 2:06 PM, Eric Sunshine <sunshine@xxxxxxxxxxxxxx> wrote: > const char *value; > > if (!git_config_get_string_const(key, &value)) { > if (parse_notes_strategy(value, strategy)) > git_die_config(key, "unknown notes merge strategy '%s'", value); > return 0; > } > return 1; > > Or, the equivalent, but less indented: > > if (git_config_get_string_const(key, &value)) > return 1; > if (parse_notes_strategy(value, strategy)) > git_die_config(key, "unknown notes merge strategy '%s'", value); > return 0; I like it. Will do. -- 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