Merge branch 'dl/range-diff-with-notes'

Code clean-up.

* dl/range-diff-with-notes:
  range-diff: clear `other_arg` at end of function
  range-diff: mark pointers as const
  t3206: fix incorrect test name
This commit is contained in:
Junio C Hamano 2019-12-16 13:08:46 -08:00
commit d1c0fe8d9b
4 changed files with 7 additions and 6 deletions

View File

@ -84,6 +84,7 @@ int cmd_range_diff(int argc, const char **argv, const char *prefix)
res = show_range_diff(range1.buf, range2.buf, creation_factor,
simple_color < 1, &diffopt, &other_arg);
argv_array_clear(&other_arg);
strbuf_release(&range1);
strbuf_release(&range2);

View File

@ -41,7 +41,7 @@ static size_t find_end_of_line(char *buffer, unsigned long size)
* as struct object_id (will need to be free()d).
*/
static int read_patches(const char *range, struct string_list *list,
struct argv_array *other_arg)
const struct argv_array *other_arg)
{
struct child_process cp = CHILD_PROCESS_INIT;
struct strbuf buf = STRBUF_INIT, contents = STRBUF_INIT;
@ -506,8 +506,8 @@ static struct strbuf *output_prefix_cb(struct diff_options *opt, void *data)
int show_range_diff(const char *range1, const char *range2,
int creation_factor, int dual_color,
struct diff_options *diffopt,
struct argv_array *other_arg)
const struct diff_options *diffopt,
const struct argv_array *other_arg)
{
int res = 0;

View File

@ -13,7 +13,7 @@
*/
int show_range_diff(const char *range1, const char *range2,
int creation_factor, int dual_color,
struct diff_options *diffopt,
struct argv_array *other_arg);
const struct diff_options *diffopt,
const struct argv_array *other_arg);
#endif

View File

@ -638,7 +638,7 @@ test_expect_success 'format-patch --range-diff with --notes' '
test_cmp expect actual
'
test_expect_success 'format-patch --range-diff with --notes' '
test_expect_success 'format-patch --range-diff with format.notes config' '
git notes add -m "topic note" topic &&
git notes add -m "unmodified note" unmodified &&
test_when_finished git notes remove topic unmodified &&