diff3: Use variables for old and new in A scripts

This makes the code easier to follow, no functional changes intended

Sponsored by:	Klara, Inc.
This commit is contained in:
Tom Jones 2022-08-01 14:48:35 +01:00
parent ca53a8445a
commit 2c4276aaa2

View file

@ -631,19 +631,21 @@ Ascript(int n)
bool deletenew;
bool deleteold;
for (; n > 0; n--) {
struct range *new, *old;
deletenew = (de[n].new.from == de[n].new.to);
deleteold = (de[n].old.from == de[n].old.to);
startmark = de[n].old.from + (de[n].old.to - de[n].old.from) - 1;
for (; n > 0; n--) {
new = &de[n].new;
old = &de[n].old;
deletenew = (new->from == new->to);
deleteold = (old->from == old->to);
startmark = old->from + (old->to - old->from) - 1;
if (de[n].type == DIFF_TYPE2) {
if (!oflag || !overlap[n]) {
prange(&de[n].old, deletenew);
printrange(fp[2], &de[n].new);
prange(old, deletenew);
printrange(fp[2], new);
} else {
startmark = de[n].new.from +
(de[n].new.to - de[n].new.from);
startmark = new->from + (new->to - new->from);
if (!deletenew)
startmark--;
@ -654,31 +656,31 @@ Ascript(int n)
printf(".\n");
printf("%da\n", startmark -
(de[n].new.to - de[n].new.from));
(new->to - new->from));
printf("%s %s\n", oldmark, f2mark);
if (!deleteold)
printrange(fp[1], &de[n].old);
printrange(fp[1], old);
printf("%s\n.\n", divider);
}
} else if (de[n].type == DIFF_TYPE3) {
if (!oflag || !overlap[n]) {
prange(&de[n].old, deletenew);
printrange(fp[2], &de[n].new);
prange(old, deletenew);
printrange(fp[2], new);
} else {
printf("%da\n", startmark);
printf("%s %s\n", orgmark, f2mark);
if (deleteold) {
struct range r;
r.from = de[n].old.from-1;
r.to = de[n].new.to;
r.from = old->from-1;
r.to = new->to;
printrange(fp[1], &r);
} else
printrange(fp[1], &de[n].old);
printrange(fp[1], old);
printf("%s\n", divider);
printrange(fp[2], &de[n].new);
printrange(fp[2], new);
}
if (!oflag || !overlap[n]) {
@ -692,7 +694,7 @@ Ascript(int n)
* file and append lines
*/
printf("%da\n%s %s\n.\n",
startmark - (de[n].old.to - de[n].old.from),
startmark - (old->to - old->from),
oldmark, f1mark);
}
}