simon-git: puzzles (main): Simon Tatham
Commits to Tartarus hosted VCS
tartarus-commits at lists.tartarus.org
Fri Jul 14 08:09:55 BST 2023
TL;DR:
40d0de7 grid_edge_bydots_cmpfn: remove dangerous pointer comparison.
Repository: https://git.tartarus.org/simon/puzzles.git
On the web: https://git.tartarus.org/?p=simon/puzzles.git
Branch updated: main
Committer: Simon Tatham <anakin at pobox.com>
Date: 2023-07-14 08:09:55
commit 40d0de7a668ea4c95cdf06af4a1554ff0be6936d
web diff https://git.tartarus.org/?p=simon/puzzles.git;a=commitdiff;h=40d0de7a668ea4c95cdf06af4a1554ff0be6936d;hp=a95796ebca53a7b7e0412860f3a38ec518d838be
Author: Simon Tatham <anakin at pobox.com>
Date: Fri Jul 14 08:09:51 2023 +0100
grid_edge_bydots_cmpfn: remove dangerous pointer comparison.
Commit e6cdd70df867f06 made the grid_dot structures for a grid no
longer be elements of the same array. But I didn't notice that
grid_edge_bydots_cmpfn was doing pointer subtraction on them on the
assumption that they were.
Fixed by comparing the dots' new index fields, which should correspond
exactly to their previous positions in the single array, so the
behaviour should be just what it was before the change.
grid.c | 16 +++++++++-------
1 file changed, 9 insertions(+), 7 deletions(-)
More information about the tartarus-commits
mailing list