add-interactive: make sure to release rev.prune_data

dscho · gitster · commit 0c3944a628e4 · 2019-12-01T07:30:53.000-08:00
During a review, Junio Hamano pointed out that the `rev.prune_data` was
copied from another pathspec but never cleaned up.

Signed-off-by: Johannes Schindelin &lt;johannes.schindelin@gmx.de&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
diff --git a/add-interactive.c b/add-interactive.c
@@ -435,6 +435,9 @@ static int get_modified_files(struct repository *r, struct string_list *files,
 			rev.diffopt.flags.ignore_dirty_submodules = 1;
 			run_diff_files(&rev, 0);
 		}
+
+		if (ps)
+			clear_pathspec(&rev.prune_data);
 	}
 	hashmap_free_entries(&s.file_map, struct pathname_entry, ent);
 

Original file line number	Diff line number	Diff line change
`@@ -435,6 +435,9 @@ static int get_modified_files(struct repository r, struct string_list files,`
`435`	`435`	`rev.diffopt.flags.ignore_dirty_submodules = 1;`
`436`	`436`	`run_diff_files(&rev, 0);`
`437`	`437`	`}`
	`438`	`+`
	`439`	`+ if (ps)`
	`440`	`+ clear_pathspec(&rev.prune_data);`
`438`	`441`	`}`
`439`	`442`	`hashmap_free_entries(&s.file_map, struct pathname_entry, ent);`
`440`	`443`