nate@xxxxxxxxxxxxx writes: > if (*load_refs && !strcmp(k, "notes.displayref")) { > if (!v) > - config_error_nonbool(k); > + return config_error_nonbool(k); "git grep config_error_nonbool" tells us that this is the only instance that ignores the return value from the function and does not cause the caller to die. Looks good. Thanks.