On Thu, Jan 10, 2013 at 2:26 AM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Martin von Zweigbergk <martinvonz@xxxxxxxxx> writes: > >> We use the path arguments in two places in reset.c: in >> interactive_reset() and read_from_tree(). Both of these call >> get_pathspec(), so we pass the (prefix, arv) pair to both >> functions. Move the call to get_pathspec() out of these methods, for >> two reasons: 1) One argument is simpler than two. 2) It lets us use >> the (arguably clearer) "if (pathspec)" in place of "if (i < argc)". >> --- >> If I understand correctly, this should be rebased on top of >> nd/parse-pathspec. Please let me know. > > Yeah, this will conflict with the get_pathspec-to-parse_pathspec > conversion Duy has been working on. Or I could hold off nd/parse-pathspec if this series has a better chance of graduation first. Decision? -- Duy -- 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