From a606f83e7a0328705c44f89a714d7fb15d33feb1 Mon Sep 17 00:00:00 2001 From: Michael Meyer Date: Wed, 1 Dec 2021 09:31:31 -0500 Subject: [PATCH] Fix: 'you owe It for goods lost' Some messages about owing a shopkeeper money would use 'it' when blind, with weird results such as "You owe It 267 zorkmids for goods lost." It seems maybe like these were missed in 6591f8b since they were outside of shk.c/shknam.c. Bring those messages into alignment with most other shopkeeper-related messages, which use the shopkeeper's name even if the hero is blind or can't see them at the moment. Some of the 'it gets angry' ones don't seem so bad, but similar 'gets angry' messages in shk.c use Shknam so I changed those as well for consistency's sake. --- src/dig.c | 2 +- src/dokick.c | 6 +++--- src/zap.c | 4 ++-- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/dig.c b/src/dig.c index 8a38f71db..1ed57b428 100644 --- a/src/dig.c +++ b/src/dig.c @@ -1912,7 +1912,7 @@ bury_objs(int x, int y) newsym(x, y); if (costly && loss) { - You("owe %s %ld %s for burying merchandise.", mon_nam(shkp), loss, + You("owe %s %ld %s for burying merchandise.", shkname(shkp), loss, currency(loss)); } } diff --git a/src/dokick.c b/src/dokick.c index b55395dcb..25026d5e6 100644 --- a/src/dokick.c +++ b/src/dokick.c @@ -434,7 +434,7 @@ container_impact_dmg(struct obj *obj, xchar x, You("caused %ld %s worth of damage!", loss, currency(loss)); make_angry_shk(shkp, x, y); } else { - You("owe %s %ld %s for objects destroyed.", mon_nam(shkp), loss, + You("owe %s %ld %s for objects destroyed.", shkname(shkp), loss, currency(loss)); } } @@ -1459,7 +1459,7 @@ impact_drop(struct obj *missile, /* caused impact, won't drop itself */ if (ESHK(shkp)->customer[0] == 0) (void) strncpy(ESHK(shkp)->customer, g.plname, PL_NSIZ); if (angry) - pline("%s is infuriated!", Monnam(shkp)); + pline("%s is infuriated!", Shknam(shkp)); else pline("\"%s, you are a thief!\"", g.plname); } else @@ -1470,7 +1470,7 @@ impact_drop(struct obj *missile, /* caused impact, won't drop itself */ } if (ESHK(shkp)->debit > debit) { long amt = (ESHK(shkp)->debit - debit); - You("owe %s %ld %s for goods lost.", Monnam(shkp), amt, + You("owe %s %ld %s for goods lost.", shkname(shkp), amt, currency(amt)); } } diff --git a/src/zap.c b/src/zap.c index 0ae62798d..96ea5de3f 100644 --- a/src/zap.c +++ b/src/zap.c @@ -1814,11 +1814,11 @@ poly_obj(struct obj *obj, int id) && !costly_spot(u.ux, u.uy)) { make_angry_shk(shkp, ox, oy); } else { - pline("%s gets angry!", Monnam(shkp)); + pline("%s gets angry!", Shknam(shkp)); hot_pursuit(shkp); } } else - Norep("%s is furious!", Monnam(shkp)); + Norep("%s is furious!", Shknam(shkp)); } } delobj(obj);