Skip to content

Commit 1e45db1

Browse files
peffgitster
authored andcommitted
xdiff: drop unused flags parameter from recs_match
Since 6b13bc3 (xdiff: simplify comparison, 2021-11-17), we do not call xdl_recmatch() from xdiffi.c's recs_match(), so we no longer need the "flags" parameter. That in turn lets us drop the flags parameters from the group-slide functions which call it. There's no functional change here; it's just making these functions a little simpler. Signed-off-by: Jeff King <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent 2544945 commit 1e45db1

File tree

1 file changed

+9
-9
lines changed

1 file changed

+9
-9
lines changed

xdiff/xdiffi.c

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -390,7 +390,7 @@ static xdchange_t *xdl_add_change(xdchange_t *xscr, long i1, long i2, long chg1,
390390
}
391391

392392

393-
static int recs_match(xrecord_t *rec1, xrecord_t *rec2, long flags)
393+
static int recs_match(xrecord_t *rec1, xrecord_t *rec2)
394394
{
395395
return (rec1->ha == rec2->ha);
396396
}
@@ -756,10 +756,10 @@ static inline int group_previous(xdfile_t *xdf, struct xdlgroup *g)
756756
* following group, expand this group to include it. Return 0 on success or -1
757757
* if g cannot be slid down.
758758
*/
759-
static int group_slide_down(xdfile_t *xdf, struct xdlgroup *g, long flags)
759+
static int group_slide_down(xdfile_t *xdf, struct xdlgroup *g)
760760
{
761761
if (g->end < xdf->nrec &&
762-
recs_match(xdf->recs[g->start], xdf->recs[g->end], flags)) {
762+
recs_match(xdf->recs[g->start], xdf->recs[g->end])) {
763763
xdf->rchg[g->start++] = 0;
764764
xdf->rchg[g->end++] = 1;
765765

@@ -777,10 +777,10 @@ static int group_slide_down(xdfile_t *xdf, struct xdlgroup *g, long flags)
777777
* into a previous group, expand this group to include it. Return 0 on success
778778
* or -1 if g cannot be slid up.
779779
*/
780-
static int group_slide_up(xdfile_t *xdf, struct xdlgroup *g, long flags)
780+
static int group_slide_up(xdfile_t *xdf, struct xdlgroup *g)
781781
{
782782
if (g->start > 0 &&
783-
recs_match(xdf->recs[g->start - 1], xdf->recs[g->end - 1], flags)) {
783+
recs_match(xdf->recs[g->start - 1], xdf->recs[g->end - 1])) {
784784
xdf->rchg[--g->start] = 1;
785785
xdf->rchg[--g->end] = 0;
786786

@@ -830,7 +830,7 @@ int xdl_change_compact(xdfile_t *xdf, xdfile_t *xdfo, long flags) {
830830
end_matching_other = -1;
831831

832832
/* Shift the group backward as much as possible: */
833-
while (!group_slide_up(xdf, &g, flags))
833+
while (!group_slide_up(xdf, &g))
834834
if (group_previous(xdfo, &go))
835835
BUG("group sync broken sliding up");
836836

@@ -845,7 +845,7 @@ int xdl_change_compact(xdfile_t *xdf, xdfile_t *xdfo, long flags) {
845845

846846
/* Now shift the group forward as far as possible: */
847847
while (1) {
848-
if (group_slide_down(xdf, &g, flags))
848+
if (group_slide_down(xdf, &g))
849849
break;
850850
if (group_next(xdfo, &go))
851851
BUG("group sync broken sliding down");
@@ -872,7 +872,7 @@ int xdl_change_compact(xdfile_t *xdf, xdfile_t *xdfo, long flags) {
872872
* other file that it can align with.
873873
*/
874874
while (go.end == go.start) {
875-
if (group_slide_up(xdf, &g, flags))
875+
if (group_slide_up(xdf, &g))
876876
BUG("match disappeared");
877877
if (group_previous(xdfo, &go))
878878
BUG("group sync broken sliding to match");
@@ -915,7 +915,7 @@ int xdl_change_compact(xdfile_t *xdf, xdfile_t *xdfo, long flags) {
915915
}
916916

917917
while (g.end > best_shift) {
918-
if (group_slide_up(xdf, &g, flags))
918+
if (group_slide_up(xdf, &g))
919919
BUG("best shift unreached");
920920
if (group_previous(xdfo, &go))
921921
BUG("group sync broken sliding to blank line");

0 commit comments

Comments
 (0)