revert box naming

doc/fixes36.1

@@ -609,8 +609,6 @@ when clairvoyance lets you move the cursor to examine the map (if it occurs
 prevent Mjollnir from being auto-quivered if it's been thrown without return
 	and then picked back up while quiver slot is empty
 plural of "fox" is not "foxen"
-change wording from "broken chest" to "chest with a broken lock" and during
-	#force" suppress redundant info when lock state is already known
 
 
 Platform- and/or Interface-Specific Fixes

src/lock.c

@@ -548,18 +548,9 @@ doforce()
     xlock.box = (struct obj *) 0;
     for (otmp = level.objects[u.ux][u.uy]; otmp; otmp = otmp->nexthere)
         if (Is_box(otmp)) {
-            if (otmp->obroken) {
+            if (otmp->obroken || !otmp->olocked) {
-                There("is %s here%s.", doname(otmp),
+                There("is %s here, but its lock is already %s.", doname(otmp),
-                      /* The displayed name will have already stated
+                      otmp->obroken ? "broken" : "unlocked");
-                       * "with a broken lock" if otmp->lknown is already set
-                       * so suppress the additional notification about the
-                       * lock in that case. */
-                      !otmp->lknown ? ", but its lock is already broken" : "");
-                otmp->lknown = 1;
-                continue;
-            } else if (!otmp->olocked) {
-                There("is %s here, but its lock is already unlocked.",
-                      doname(otmp));
                 otmp->lknown = 1;
                 continue;
             }

src/objnam.c

@@ -995,10 +995,10 @@ unsigned doname_flags;
 
     if (lknown && Is_box(obj)) {
         if (obj->obroken)
-            /* 3.6.0 used an "unlockable" prefix here but that could be
+            /* 3.6.0 used "unlockable" here but that could be misunderstood
-               misunderstood to mean "capable of being unlocked" rather
+               to mean "capable of being unlocked" rather than the intended
-               than the intended "not capable of being locked" */
+               "not capable of being locked" */
-            Strcat(bp, " with a broken lock");
+            Strcat(prefix, "broken ");
         else if (obj->olocked)
             Strcat(prefix, "locked ");
         else