color of mithril armor

From a bug report two years ago, mithril armor should have color
HI_SILVER rather than HI_METAL.  Tolkien describes mithril as "like
silver which never tarnishes".  (The bug report had a more precise
quote, and a github URL for a patch which I've ignored.)  The Amulet
of Yendor has material set to MITHRIL but I haven't changed its color.
All amulets are HI_METAL, so if any modification is needed for it,
change the material rather than the color.

Mithril-coats are displayed as cyan both before and after this patch,
so it's hard to tell whether any change actually happened.

diff --git a/doc/fixes35.0 b/doc/fixes35.0
index 6a3e06d37aa8b85050452d90d4f727e07a8d5d83..4c4754c6bfcf88090fea595875745e2ec00fc835 100644 (file)
--- a/doc/fixes35.0
+++ b/doc/fixes35.0
@@ -896,6 +896,7 @@ allow showing legal positions for stinking cloud, jumping and polearms
        when asked for a location
 cloned creatures (of any type) don't deathdrop items
 pudding corpses behave somewhat differently than before
+mithril armor should have silver color
 
 
 Platform- and/or Interface-Specific Fixes

diff --git a/src/objects.c b/src/objects.c
index fc17962441bf372fa06cb4f933da9e04ae94bfae..510fed9e645466ad766f0e736f29bda1bd581b27 100644 (file)
--- a/src/objects.c
+++ b/src/objects.c
@@ -1,4 +1,4 @@
-/* NetHack 3.5 objects.c       $NHDT-Date: 1426470348 2015/03/16 01:45:48 $  $NHDT-Branch: derek-farming $:$NHDT-Revision: 1.27 $ */
+/* NetHack 3.5 objects.c       $NHDT-Date: 1428802519 2015/04/12 01:35:19 $  $NHDT-Branch: master $:$NHDT-Revision: 1.34 $ */
 /* Copyright (c) Mike Threepoint, 1989.                                  */
 /* NetHack may be freely redistributed.  See license for details. */
 
@@ -373,9 +373,9 @@ ARMOR("splint mail", (char *)0,
 ARMOR("banded mail", (char *)0,
        1, 0, 1, 0,     72, 5, 350,  90,  4, 1, ARM_SUIT, IRON, HI_METAL),
 ARMOR("dwarvish mithril-coat", (char *)0,
-       1, 0, 0, 0,     10, 1, 150, 240,  4, 2, ARM_SUIT, MITHRIL, HI_METAL),
+       1, 0, 0, 0,     10, 1, 150, 240,  4, 2, ARM_SUIT, MITHRIL, HI_SILVER),
 ARMOR("elven mithril-coat", (char *)0,
-       1, 0, 0, 0,     15, 1, 150, 240,  5, 2, ARM_SUIT, MITHRIL, HI_METAL),
+       1, 0, 0, 0,     15, 1, 150, 240,  5, 2, ARM_SUIT, MITHRIL, HI_SILVER),
 ARMOR("chain mail", (char *)0,
        1, 0, 0, 0,     72, 5, 300,  75,  5, 1, ARM_SUIT, IRON, HI_METAL),
 ARMOR("orcish chain mail", "crude chain mail",