From 7abf85a9c6b460698994d9cfa538b7b26fed5e87 Mon Sep 17 00:00:00 2001 From: Simon Tatham Date: Thu, 20 Apr 2023 14:00:23 +0100 Subject: [PATCH] Remove a direct use of dsf internals in Filling. The expression 'dsf[foo] >> 2' already has a sensible wrapper function, but Filling wasn't bothering to call it. --- filling.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/filling.c b/filling.c index 2164205..cd4a01e 100644 --- a/filling.c +++ b/filling.c @@ -559,7 +559,7 @@ static int expandsize(const int *board, int *dsf, int w, int h, int i, int n) { root = dsf_canonify(dsf, idx); for (m = 0; m < nhits && root != hits[m]; ++m); if (m < nhits) continue; - printv("\t (%d, %d) contrib %d to size\n", x, y, dsf[root] >> 2); + printv("\t (%d, %d) contrib %d to size\n", x, y, dsf_size(dsf, root)); size += dsf_size(dsf, root); assert(dsf_size(dsf, root) >= 1); hits[nhits++] = root;