Skip to content

Commit d5a3897

Browse files
committed
Merge branch 'rs/pickaxe-simplify'
* rs/pickaxe-simplify: diffcore-pickaxe: simplify has_changes and contains
2 parents 533a05f + 3bdb5b9 commit d5a3897

File tree

1 file changed

+4
-7
lines changed

1 file changed

+4
-7
lines changed

diffcore-pickaxe.c

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -131,8 +131,7 @@ static void diffcore_pickaxe_grep(struct diff_options *o)
131131
return;
132132
}
133133

134-
static unsigned int contains(mmfile_t *mf, struct diff_options *o,
135-
regex_t *regexp, kwset_t kws)
134+
static unsigned int contains(mmfile_t *mf, regex_t *regexp, kwset_t kws)
136135
{
137136
unsigned int cnt;
138137
unsigned long sz;
@@ -176,11 +175,9 @@ static int has_changes(mmfile_t *one, mmfile_t *two,
176175
struct diff_options *o,
177176
regex_t *regexp, kwset_t kws)
178177
{
179-
if (!one)
180-
return contains(two, o, regexp, kws) != 0;
181-
if (!two)
182-
return contains(one, o, regexp, kws) != 0;
183-
return contains(one, o, regexp, kws) != contains(two, o, regexp, kws);
178+
unsigned int one_contains = one ? contains(one, regexp, kws) : 0;
179+
unsigned int two_contains = two ? contains(two, regexp, kws) : 0;
180+
return one_contains != two_contains;
184181
}
185182

186183
static int pickaxe_match(struct diff_filepair *p, struct diff_options *o,

0 commit comments

Comments
 (0)