ref: 4a8edc65bbe906093787645b98122c7750ae83e8
parent: 3017455259f955e6257381e449419fea780a40f2
author: Simon Tatham <anakin@pobox.com>
date: Wed Oct 29 17:41:38 EDT 2014
Fix error highlighting of closed subgraphs in Bridges. When I did the recent rewrite of the Bridges drawing code, I spotted and replicated the code in game_redraw that set G_WARN at redraw time on an island with unsatisfiable clues, but didn't spot the code elsewhere (e.g. map_group_check) which might have set G_WARN ahead of time, so I failed to check for that. As a result, if you join some islands together into a subgraph with no further room for expansion (e.g. connect a pair of 1s together, or a pair of 2s with a double bridge) then all the bridges in that subgraph light up red but the islands themselves forget to. Trivial one-line fix.
--- a/bridges.c
+++ b/bridges.c
@@ -2967,7 +2967,7 @@
if (is_drag_src && (is == is_drag_src ||
(is_drag_dst && is == is_drag_dst)))
idata |= DI_COL_SELECTED;
- else if (island_impossible(is, v & G_MARK))
+ else if (island_impossible(is, v & G_MARK) || (v & G_WARN))
idata |= DI_COL_WARNING;
else
idata |= DI_COL_NORMAL;