2010/12/14 Junio C Hamano <gitster@xxxxxxxxx>: > Nguyán ThÃi Ngác Duy <pclouds@xxxxxxxxx> writes: > >> diff --git a/diff-lib.c b/diff-lib.c >> index 392ce2b..3b809f2 100644 >> --- a/diff-lib.c >> +++ b/diff-lib.c >> @@ -501,7 +501,7 @@ int do_diff_cache(const unsigned char *tree_sha1, struct diff_options *opt) >> Â Â Â active_nr = dst - active_cache; >> >> Â Â Â init_revisions(&revs, NULL); >> - Â Â revs.prune_data = opt->paths; >> + Â Â revs.prune_data = opt->pathspec.raw; >> Â Â Â tree = parse_tree_indirect(tree_sha1); >> Â Â Â if (!tree) >> Â Â Â Â Â Â Â die("bad tree object %s", sha1_to_hex(tree_sha1)); > > Hopefully the prune_data will become opt->pathspec not "raw" and use your > generalied/unified pathspec matching code in later patches in the series, > yes? Yes. But that needs closer look. "prune_data" is void* and casted in many places. Compiler won't catch mistyping for us. -- 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