laumann/ani - Change FMYCPGKDAPOUFLQOCXXKZ6TR75HT3TNCG4X3GAJDM4ARWKNWOSWAC

change: extract hash list decoding into reused function

Created by laumann on September 20, 2022

FMYCPGKDAPOUFLQOCXXKZ6TR75HT3TNCG4X3GAJDM4ARWKNWOSWAC

Dependencies

In channels

main

Change contents

Insertion in change.c at line 155 [4.1047]

[3.1067]


void
change_decode_hashlist(struct bincode_state *binstat, struct hash_list *target)
{
	size_t len;
	int i;
	len = bincode_getu64(binstat);
	target->len = len;
	target->entries = xmalloc(sizeof(struct hash) * len);

Insertion in change.c at line 166 [4.1047]

[3.1068]

	for (i = 0; i < target->len; i++) {
		target->entries[i].variant = bincode_getu32(binstat);
		switch (target->entries[i].variant) {
		case HASH_BLAKE3:
			memcpy(target->entries[i].bytes, binstat->buf,
			       BLAKE3_BYTES);
			binstat->buf += BLAKE3_BYTES;
			binstat->avail -= BLAKE3_BYTES;
			break;
		case HASH_NONE:
			/* Do nothing, no bytes to consume */
			break;
		default:
			abort();
		}
	}
}

Insertion in change.c at line 284 [4.1047]
[5.842]
[5.842]
```
	printf("# Dependencies\n");
```

Replacement in change.c at line 286 [4.1047]

B:BD[5.881] → [5.881:1013]

	len = bincode_getu64(&binstat);
	hashed.dependencies.len = len;
	hashed.dependencies.entries = xmalloc(sizeof(struct hash) * len);

[5.881]

[6.1176]

	change_decode_hashlist(&binstat, &hashed.dependencies);

Deletion in change.c at line 288 [4.1047]

B:BD[6.1177] → [5.1014:1094]

B:BD[5.1094] → [2.1618:1690]

∅:D[2.1690] → [5.1163:1235]

B:BD[5.1163] → [5.1163:1235]

B:BD[5.1235] → [2.1691:1777]

∅:D[2.1777] → [5.1311:1530]

B:BD[5.1311] → [5.1311:1530]

∅:D[5.1530] → [6.1177:1180]

B:BD[6.1177] → [6.1177:1180]

∅:D[6.1227] → [7.1639:1640]

B:BD[7.1639] → [7.1639:1640]

	printf("[[dependencies]]\n");
	for (i = 0; i < hashed.dependencies.len; i++) {
		hashed.dependencies.entries[i].variant =
			bincode_getu32(&binstat);
		switch (hashed.dependencies.entries[i].variant) {
		case HASH_BLAKE3:
			memcpy(hashed.dependencies.entries[i].bytes,
			       binstat.buf, BLAKE3_BYTES);
			binstat.buf += BLAKE3_BYTES;
			binstat.avail -= BLAKE3_BYTES;
			break;
		case HASH_NONE:
			/* Do nothing, no bytes to consume */
			break;
		default:
			abort();
		}
		hash_print(&hashed.dependencies.entries[i]);
	}

Replacement in change.c at line 289 [4.1047]

B:BD[5.1575] → [5.1575:1705]

∅:D[5.1705] → [7.1640:1641]

B:BD[7.1640] → [7.1640:1641]

B:BD[7.1641] → [5.1706:1784]

B:BD[5.1784] → [2.1778:1849]

∅:D[2.1849] → [5.1852:1923]

B:BD[5.1852] → [5.1852:1923]

B:BD[5.1923] → [2.1850:1935]

∅:D[2.1935] → [5.1998:2219]

B:BD[5.1998] → [5.1998:2219]

	len = bincode_getu64(&binstat);
	hashed.extra_known.len = len;
	hashed.extra_known.entries = xmalloc(sizeof(struct hash) * len);
	printf("[[extra_known]]\n");
	for (i = 0; i < hashed.extra_known.len; i++) {
		hashed.extra_known.entries[i].variant =
			bincode_getu32(&binstat);
		switch (hashed.extra_known.entries[i].variant) {
		case HASH_BLAKE3:
			memcpy(hashed.extra_known.entries[i].bytes, binstat.buf,
			       BLAKE3_BYTES);
			binstat.buf += BLAKE3_BYTES;
			binstat.avail -= BLAKE3_BYTES;
			break;
		case HASH_NONE:
			/* Do nothing, no bytes to consume */
			break;
		default:
			abort();
		}
		hash_print(&hashed.extra_known.entries[i]);
	}

[5.1575]

[8.0]

	change_decode_hashlist(&binstat, &hashed.extra_known);