Tim Henigan <tim.henigan@xxxxxxxxx> writes: >>> @@ -273,5 +273,6 @@ void diff_no_index(struct rev_info *revs, >>> * The return code for --no-index imitates diff(1): >>> * 0 = no changes, 1 = changes, else error >>> */ >>> - exit(revs->diffopt.found_changes); >>> + result = !!diff_result_code(&revs->diffopt, 0); >>> + exit(result); > > I assume the '!!' before 'diff_result_code' is a typo. Not a typo. I meant to use that idiom to turn 0 or not into boolean, as diff_result_code() can return values other than 0 or 1. -- 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