Thomas Rast <trast@xxxxxxxxxxxxxxx> writes: > +void line_log_init(struct rev_info *rev, struct diff_line_range *r) > +{ > + struct commit *commit = NULL; > + struct diff_options *opt = &rev->diffopt; > + > + commit = (struct commit *)verify_commit(rev); > + parse_lines(commit, r); > + > + add_line_range(rev, commit, r); > + /* > + * Note we support -M/-C to detect file rename > + */ > + opt->nr_paths = 0; > + diff_tree_release_paths(opt); > +} Note that opt->nr_paths may be going away soon (cf. nd/struct-pathspec topic). Do you need this assignment here? -- 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