getpos moving to 'cmap' characters

While testing something I noticed that moving the cursor to visible '^'
by typing '^' while getpos was asking me to pick a location, it didn't
always cycle through all visible traps.  The most straightforward
culprit was after trap detection (via confused gold detection, not ^F)
had found a trap door or level teleporter in a closet that itself was
a secret corridor spot.  But it turned out to be any location that
hadn't been seen yet.  This is a substantial overhaul of the relevant
code and so far works for all the cases I've tried, but there are
bound to be cases I haven't tried yet and those may or may not work
correctly.

There's also a bunch of formatting cleanup, and some simplification of
the m/M/o/O/d/D/x/X handling.
This commit is contained in:
PatR
2016-05-30 17:56:47 -07:00
parent 5addbf88dd
commit 7404597ac5
2 changed files with 99 additions and 75 deletions
+2
View File
@@ -272,6 +272,8 @@ reviving one of a stack of N corpses in a shop charged a usage fee for all N;
remaining N-1 were owned by hero if carried but by shop if on floor
gremlin wailing in agony should wake up nearby monsters
add more lighting variance to the second bigroom variant
when getpost was picking a location, typing '^' to move to the next known trap
skipped some detected traps if their location was unseen
Fixes to Post-3.6.0 Problems that Were Exposed Via git Repository