Skip to content

Commit 307c33b

Browse files
ezekielnewrengitster
authored andcommitted
xdiff: make xdfile_t.nreff a size_t instead of long
size_t is used because nreff describes the number of elements in memory for rindex. Signed-off-by: Ezekiel Newren <ezekielnewren@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
1 parent f90a98a commit 307c33b

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

xdiff/xprepare.c

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -264,7 +264,7 @@ static bool xdl_clean_mmatch(uint8_t const *action, long i, long s, long e) {
264264
* might be potentially discarded if they appear in a run of discardable.
265265
*/
266266
static int xdl_cleanup_records(xdlclassifier_t *cf, xdfile_t *xdf1, xdfile_t *xdf2) {
267-
long i, nm, nreff, mlim;
267+
long i, nm, mlim;
268268
xrecord_t *recs;
269269
xdlclass_t *rcrec;
270270
uint8_t *action1 = NULL, *action2 = NULL;
@@ -307,29 +307,29 @@ static int xdl_cleanup_records(xdlclassifier_t *cf, xdfile_t *xdf1, xdfile_t *xd
307307
* Use temporary arrays to decide if changed[i] should remain
308308
* false, or become true.
309309
*/
310-
for (nreff = 0, i = xdf1->dstart, recs = &xdf1->recs[xdf1->dstart];
310+
xdf1->nreff = 0;
311+
for (i = xdf1->dstart, recs = &xdf1->recs[xdf1->dstart];
311312
i <= xdf1->dend; i++, recs++) {
312313
if (action1[i] == KEEP ||
313314
(action1[i] == INVESTIGATE && !xdl_clean_mmatch(action1, i, xdf1->dstart, xdf1->dend))) {
314-
xdf1->rindex[nreff++] = i;
315+
xdf1->rindex[xdf1->nreff++] = i;
315316
/* changed[i] remains false, i.e. keep */
316317
} else
317318
xdf1->changed[i] = true;
318319
/* i.e. discard */
319320
}
320-
xdf1->nreff = nreff;
321321

322-
for (nreff = 0, i = xdf2->dstart, recs = &xdf2->recs[xdf2->dstart];
322+
xdf2->nreff = 0;
323+
for (i = xdf2->dstart, recs = &xdf2->recs[xdf2->dstart];
323324
i <= xdf2->dend; i++, recs++) {
324325
if (action2[i] == KEEP ||
325326
(action2[i] == INVESTIGATE && !xdl_clean_mmatch(action2, i, xdf2->dstart, xdf2->dend))) {
326-
xdf2->rindex[nreff++] = i;
327+
xdf2->rindex[xdf2->nreff++] = i;
327328
/* changed[i] remains false, i.e. keep */
328329
} else
329330
xdf2->changed[i] = true;
330331
/* i.e. discard */
331332
}
332-
xdf2->nreff = nreff;
333333

334334
cleanup:
335335
xdl_free(action1);

xdiff/xtypes.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ typedef struct s_xdfile {
5050
size_t nrec;
5151
bool *changed;
5252
long *rindex;
53-
long nreff;
53+
size_t nreff;
5454
ptrdiff_t dstart, dend;
5555
} xdfile_t;
5656

0 commit comments

Comments
 (0)