From: PatR <rankin@nethack.org>
Date: Sat, 4 Jun 2016 13:06:20 +0000 (-0700)
Subject: more /? of spaces
X-Git-Tag: NetHack-3.6.1_RC01~701
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=8e307be556385fa9a95e65b18d747fa0620d778d;p=nethack

more /? of spaces

Restore the ability to look up a single space by 'name'.

I thought mungspaces("<all spaces>") kept one space, but it doesn't.
It's a lucky accident that unnaming monsters and objects still works.
There may be other places which intend to give a special meaning to
a single space that don't still work....
---

diff --git a/doc/fixes36.1 b/doc/fixes36.1
index b3d6a6a9f..6b9242148 100644
--- a/doc/fixes36.1
+++ b/doc/fixes36.1
@@ -283,6 +283,8 @@ when poly'd into a hider and engulfed, attempt to hide via #monster was blocked
 various monster/object/food/gold/trap detections were inconsistent in how they
 	behaved if performed while engulfed or underwater
 show in inventory which monster a leash is attached to
+using /? to look up something by name, supplying multiple spaces (with no
+	other characters) as the name triggered impossible "singular of null?"
 
 
 Fixes to Post-3.6.0 Problems that Were Exposed Via git Repository
diff --git a/src/pager.c b/src/pager.c
index 92c72787d..77ade35b0 100644
--- a/src/pager.c
+++ b/src/pager.c
@@ -1060,9 +1060,10 @@ coord *click_cc;
         case '?':
             from_screen = FALSE;
             getlin("Specify what? (type the word)", out_str);
-            /* mungspaces prevents querying for a space glyph (eg. a ghost),
-               but players almost always use '/' instead to look up glyphs */
-            mungspaces(out_str);
+            if (strcmp(out_str, " ")) /* keep single space as-is */
+                /* remove leading and trailing whitespace and
+                   condense consecutive internal whitespace */
+                mungspaces(out_str);
             if (out_str[0] == '\0' || out_str[0] == '\033')
                 return 0;