Alex Riesen <raa.lkml@xxxxxxxxx> writes: > Junio C Hamano, Fri, Nov 09, 2007 08:32:01 +0100: >> "Shawn O. Pearce" <spearce@xxxxxxxxxxx> writes: >> >> > @@ -640,7 +656,9 @@ int cmd_rev_list(int argc, const char **argv, const char *prefix) >> > } >> > } >> > >> > - traverse_commit_list(&revs, show_commit, show_object); >> > + traverse_commit_list(&revs, >> > + nooutput ? noshow_commit : show_commit, >> > + nooutput ? noshow_object : show_object); >> > >> > return 0; >> > } >> >> The function names noshow_xxx() looked a bit funny, but I do not >> offhand have better alternatives to offer. > > "hide", "skip", "ignore"? But look at what the functions do. The original show_xxx() was to print and then process. Shawn splitted them into show_xxx() and noshow_xxx(), leaft the printing part in the former, made the former call the latter at the end, and moved the processing to the latter. So it is not any of the three words. - 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
