From 31ea7c1f62aa95060e9e11b1e7878423ef46b5ba Mon Sep 17 00:00:00 2001 From: copperwater Date: Sun, 6 Jan 2019 00:56:05 -0500 Subject: [PATCH 1/4] Add hallucinatory trap names This adds many funny, realistic, and nonsensical traps to the game, to be shown when the player is hallucinating. Architecturally, the biggest change is merging the what_trap macro and the "defsyms[trap_to_defsym(ttyp)].explanation" pattern into a single function "trapname", which returns the name of the trap, handling the hallucination case. There is also a second parameter used for overriding hallucination in the occasional cases where the actual trap name should always be returned. In addition, the what_trap and random_trap macros are now obsolete and not used anywhere, so they are removed. --- include/display.h | 8 ++---- include/extern.h | 1 + src/apply.c | 14 ++++----- src/cmd.c | 2 +- src/detect.c | 3 +- src/dig.c | 2 +- src/dothrow.c | 3 +- src/hack.c | 7 ++--- src/invent.c | 3 +- src/objnam.c | 8 ++---- src/pager.c | 10 +++---- src/steed.c | 2 +- src/trap.c | 73 +++++++++++++++++++++++++++++++++++++++-------- 13 files changed, 84 insertions(+), 52 deletions(-) diff --git a/include/display.h b/include/display.h index 1334be2cc..68251158f 100644 --- a/include/display.h +++ b/include/display.h @@ -142,25 +142,21 @@ /* * random_monster() * random_object() - * random_trap() * - * Respectively return a random monster, object, or trap number. + * Respectively return a random monster or object. */ #define random_monster() rn2(NUMMONS) #define random_object() rn1(NUM_OBJECTS - 1, 1) -#define random_trap() rn1(TRAPNUM - 1, 1) /* * what_obj() * what_mon() - * what_trap() * * If hallucinating, choose a random object/monster, otherwise, use the one * given. */ #define what_obj(obj) (Hallucination ? random_object() : obj) #define what_mon(mon) (Hallucination ? random_monster() : mon) -#define what_trap(trp) (Hallucination ? random_trap() : trp) /* * covers_objects() @@ -319,7 +315,7 @@ ((((expltype) * MAXEXPCHARS) + ((idx) - S_explode1)) + GLYPH_EXPLODE_OFF) #define trap_to_glyph(trap) \ - cmap_to_glyph(trap_to_defsym(what_trap((trap)->ttyp))) + cmap_to_glyph(trap_to_defsym((trap)->ttyp)) /* Not affected by hallucination. Gives a generic body for CORPSE */ /* MRKR: ...and the generic statue */ diff --git a/include/extern.h b/include/extern.h index dbef64c22..7f5f80000 100644 --- a/include/extern.h +++ b/include/extern.h @@ -2499,6 +2499,7 @@ E boolean FDECL(uteetering_at_seen_pit, (struct trap *)); E boolean NDECL(lava_effects); E void NDECL(sink_into_lava); E void NDECL(sokoban_guilt); +E const char * FDECL(trapname, (int, BOOLEAN_P)); /* ### u_init.c ### */ diff --git a/src/apply.c b/src/apply.c index dbde93fba..9372b95fe 100644 --- a/src/apply.c +++ b/src/apply.c @@ -2501,7 +2501,7 @@ struct obj *otmp; ttyp = (otmp->otyp == LAND_MINE) ? LANDMINE : BEAR_TRAP; if (otmp == trapinfo.tobj && u.ux == trapinfo.tx && u.uy == trapinfo.ty) { You("resume setting %s%s.", shk_your(buf, otmp), - defsyms[trap_to_defsym(what_trap(ttyp))].explanation); + trapname(ttyp, FALSE)); set_occupation(set_trap, occutext, 0); return; } @@ -2526,7 +2526,7 @@ struct obj *otmp; chance = (rnl(10) > 5); You("aren't very skilled at reaching from %s.", mon_nam(u.usteed)); Sprintf(buf, "Continue your attempt to set %s?", - the(defsyms[trap_to_defsym(what_trap(ttyp))].explanation)); + the(trapname(ttyp, FALSE))); if (yn(buf) == 'y') { if (chance) { switch (ttyp) { @@ -2536,9 +2536,7 @@ struct obj *otmp; break; case BEAR_TRAP: /* drop it without arming it */ reset_trapset(); - You("drop %s!", - the(defsyms[trap_to_defsym(what_trap(ttyp))] - .explanation)); + You("drop %s!", the(trapname(ttyp, FALSE))); dropx(otmp); return; } @@ -2548,8 +2546,7 @@ struct obj *otmp; return; } } - You("begin setting %s%s.", shk_your(buf, otmp), - defsyms[trap_to_defsym(what_trap(ttyp))].explanation); + You("begin setting %s%s.", shk_your(buf, otmp), trapname(ttyp, FALSE)); set_occupation(set_trap, occutext, 0); return; } @@ -2581,8 +2578,7 @@ set_trap() add_damage(u.ux, u.uy, 0L); /* schedule removal */ } if (!trapinfo.force_bungle) - You("finish arming %s.", - the(defsyms[trap_to_defsym(what_trap(ttyp))].explanation)); + You("finish arming %s.", the(trapname(ttyp, FALSE))); if (((otmp->cursed || Fumbling) && (rnl(10) > 5)) || trapinfo.force_bungle) dotrap(ttmp, diff --git a/src/cmd.c b/src/cmd.c index 04df10458..1f9191ad6 100644 --- a/src/cmd.c +++ b/src/cmd.c @@ -2293,7 +2293,7 @@ int final; Strcpy(predicament, "trapped"); if ((t = t_at(u.ux, u.uy)) != 0) Sprintf(eos(predicament), " in %s", - an(defsyms[trap_to_defsym(t->ttyp)].explanation)); + an(trapname(t->ttyp, FALSE))); } if (u.usteed) { /* not `Riding' here */ Sprintf(buf, "%s%s ", anchored ? "you and " : "", steedname); diff --git a/src/detect.c b/src/detect.c index e6c5778af..f89689742 100644 --- a/src/detect.c +++ b/src/detect.c @@ -1591,7 +1591,6 @@ void find_trap(trap) struct trap *trap; { - int tt = what_trap(trap->ttyp); boolean cleared = FALSE; trap->tseen = 1; @@ -1606,7 +1605,7 @@ struct trap *trap; cleared = TRUE; } - You("find %s.", an(defsyms[trap_to_defsym(tt)].explanation)); + You("find %s.", an(trapname(trap->ttyp, FALSE))); if (cleared) { display_nhwindow(WIN_MAP, TRUE); /* wait */ diff --git a/src/dig.c b/src/dig.c index 29d0941d4..0702ad0c8 100644 --- a/src/dig.c +++ b/src/dig.c @@ -581,7 +581,7 @@ int ttyp; if (ttyp != PIT && (!Can_dig_down(&u.uz) && !lev->candig)) { impossible("digactualhole: can't dig %s on this level.", - defsyms[trap_to_defsym(ttyp)].explanation); + trapname(ttyp, TRUE)); ttyp = PIT; } diff --git a/src/dothrow.c b/src/dothrow.c index de1afca9c..3226ed0b8 100644 --- a/src/dothrow.c +++ b/src/dothrow.c @@ -755,8 +755,7 @@ int x, y; return TRUE; } else { if (ttmp->tseen) - You("pass right over %s.", - an(defsyms[trap_to_defsym(ttmp->ttyp)].explanation)); + You("pass right over %s.", an(trapname(ttmp->ttyp, FALSE))); } } if (--*range < 0) /* make sure our range never goes negative */ diff --git a/src/hack.c b/src/hack.c index 0e3f3f6f2..734ced2d6 100644 --- a/src/hack.c +++ b/src/hack.c @@ -1453,10 +1453,8 @@ domove() if (context.run >= 2) { if (iflags.mention_walls) { if (trap && trap->tseen) { - int tt = what_trap(trap->ttyp); - You("stop in front of %s.", - an(defsyms[trap_to_defsym(tt)].explanation)); + an(trapname(trap->ttyp, FALSE))); } else if (is_pool_or_lava(x,y) && levl[x][y].seenv) { You("stop at the edge of the %s.", hliquid(is_pool(x,y) ? "water" : "lava")); @@ -2678,9 +2676,8 @@ lookaround() goto bcorr; /* if you must */ if (x == u.ux + u.dx && y == u.uy + u.dy) { if (iflags.mention_walls) { - int tt = what_trap(trap->ttyp); You("stop in front of %s.", - an(defsyms[trap_to_defsym(tt)].explanation)); + an(trapname(trap->ttyp, FALSE))); } goto stop; } diff --git a/src/invent.c b/src/invent.c index c28a2120d..58fcd8408 100644 --- a/src/invent.c +++ b/src/invent.c @@ -3382,8 +3382,7 @@ boolean picked_some; return !!Blind; } if (!skip_objects && (trap = t_at(u.ux, u.uy)) && trap->tseen) - There("is %s here.", - an(defsyms[trap_to_defsym(trap->ttyp)].explanation)); + There("is %s here.", an(trapname(trap->ttyp, FALSE))); otmp = level.objects[u.ux][u.uy]; dfeature = dfeature_at(u.ux, u.uy, fbuf2); diff --git a/src/objnam.c b/src/objnam.c index 597a79249..6e365e737 100644 --- a/src/objnam.c +++ b/src/objnam.c @@ -3363,10 +3363,8 @@ struct obj *no_wish; goto typfnd; } else if (trapped == 1 || *zp != '\0') { /* "trapped " or " trap" (actually "*") */ - int idx = trap_to_defsym(beartrap ? BEAR_TRAP : LANDMINE); - /* use canonical trap spelling, skip object matching */ - Strcpy(bp, defsyms[idx].explanation); + Strcpy(bp, trapname(beartrap ? BEAR_TRAP : LANDMINE, TRUE)); goto wiztrap; } /* [no prefix or suffix; we're going to end up matching @@ -3568,7 +3566,7 @@ wiztrap: struct trap *t; const char *tname; - tname = defsyms[trap_to_defsym(trap)].explanation; + tname = trapname(trap, TRUE); if (strncmpi(tname, bp, strlen(tname))) continue; /* found it; avoid stupid mistakes */ @@ -3576,7 +3574,7 @@ wiztrap: trap = ROCKTRAP; if ((t = maketrap(x, y, trap)) != 0) { trap = t->ttyp; - tname = defsyms[trap_to_defsym(trap)].explanation; + tname = trapname(trap, TRUE); pline("%s%s.", An(tname), (trap != MAGIC_PORTAL) ? "" : " to nowhere"); } else diff --git a/src/pager.c b/src/pager.c index 5b7f43c48..52d26339a 100644 --- a/src/pager.c +++ b/src/pager.c @@ -285,8 +285,7 @@ int x, y; /* newsym lets you know of the trap, so mention it here */ if (tt == BEAR_TRAP || is_pit(tt) || tt == WEB) - Sprintf(eos(buf), ", trapped in %s", - an(defsyms[trap_to_defsym(tt)].explanation)); + Sprintf(eos(buf), ", trapped in %s", an(trapname(tt, FALSE))); } /* we know the hero sees a monster at this location, but if it's shown @@ -435,7 +434,7 @@ char *buf, *monbuf; } else if (glyph_is_object(glyph)) { look_at_object(buf, x, y, glyph); /* fill in buf[] */ } else if (glyph_is_trap(glyph)) { - int tnum = what_trap(glyph_to_trap(glyph)); + int tnum = glyph_to_trap(glyph); /* Trap detection displays a bear trap at locations having * a trapped door or trapped container or both. @@ -447,7 +446,7 @@ char *buf, *monbuf; else if (trapped_door_at(tnum, x, y)) Strcpy(buf, "trapped door"); /* not "trap door"... */ else - Strcpy(buf, defsyms[trap_to_defsym(tnum)].explanation); + Strcpy(buf, trapname(tnum, FALSE)); } else if (glyph_is_warning(glyph)) { int warnindx = glyph_to_warning(glyph); @@ -1487,9 +1486,8 @@ doidtrap() if (u.dz < 0 ? is_hole(tt) : tt == ROCKTRAP) break; } - tt = what_trap(tt); pline("That is %s%s%s.", - an(defsyms[trap_to_defsym(tt)].explanation), + an(trapname(tt, FALSE)), !trap->madeby_u ? "" : (tt == WEB) diff --git a/src/steed.c b/src/steed.c index feede89c5..081cb475f 100644 --- a/src/steed.c +++ b/src/steed.c @@ -285,7 +285,7 @@ boolean force; /* Quietly force this animal */ struct trap *t = t_at(mtmp->mx, mtmp->my); You_cant("mount %s while %s's trapped in %s.", mon_nam(mtmp), - mhe(mtmp), an(defsyms[trap_to_defsym(t->ttyp)].explanation)); + mhe(mtmp), an(trapname(t->ttyp, FALSE))); return (FALSE); } diff --git a/src/trap.c b/src/trap.c index 965c9347e..f70a90c7c 100644 --- a/src/trap.c +++ b/src/trap.c @@ -916,14 +916,14 @@ unsigned trflags; */ pline("Air currents pull you down into %s %s!", a_your[trap->madeby_u], - defsyms[trap_to_defsym(ttype)].explanation); + trapname(ttype, TRUE)); /* do force "pit" while hallucinating */ /* then proceed to normal trap effect */ } else if (already_seen && !forcetrap) { if ((Levitation || (Flying && !plunged)) && (is_pit(ttype) || ttype == HOLE || ttype == BEAR_TRAP)) { You("%s over %s %s.", Levitation ? "float" : "fly", a_your[trap->madeby_u], - defsyms[trap_to_defsym(ttype)].explanation); + trapname(ttype, FALSE)); return; } if (!Fumbling && ttype != MAGIC_PORTAL && ttype != VIBRATING_SQUARE @@ -934,7 +934,7 @@ unsigned trflags; You("escape %s %s.", (ttype == ARROW_TRAP && !trap->madeby_u) ? "an" : a_your[trap->madeby_u], - defsyms[trap_to_defsym(ttype)].explanation); + trapname(ttype, FALSE)); return; } } @@ -1283,7 +1283,7 @@ unsigned trflags; if (!Can_fall_thru(&u.uz)) { seetrap(trap); /* normally done in fall_through */ impossible("dotrap: %ss cannot exist on this level.", - defsyms[trap_to_defsym(ttype)].explanation); + trapname(ttype, TRUE)); break; /* don't activate it after all */ } fall_through(TRUE); @@ -2461,7 +2461,7 @@ register struct monst *mtmp; case TRAPDOOR: if (!Can_fall_thru(&u.uz)) { impossible("mintrap: %ss cannot exist on this level.", - defsyms[trap_to_defsym(tt)].explanation); + trapname(tt, TRUE)); break; /* don't activate it after all */ } if (is_flyer(mptr) || is_floater(mptr) || mptr == &mons[PM_WUMPUS] @@ -4053,8 +4053,7 @@ boolean force_failure; if ((invent && (inv_weight() + weight_cap() > 600)) || bigmonst(youmonst.data)) { /* don't allow untrap if they can't get thru to it */ - You("are unable to reach the %s!", - defsyms[trap_to_defsym(ttype)].explanation); + You("are unable to reach the %s!", trapname(ttype, FALSE)); return 0; } } @@ -4063,8 +4062,7 @@ boolean force_failure; if (u.usteed && P_SKILL(P_RIDING) < P_BASIC) rider_cant_reach(); else - You("are unable to reach the %s!", - defsyms[trap_to_defsym(ttype)].explanation); + You("are unable to reach the %s!", trapname(ttype, FALSE)); return 0; } @@ -4104,7 +4102,7 @@ boolean force_failure; } else { pline("%s %s is difficult to %s.", ttmp->madeby_u ? "Your" : under_u ? "This" : "That", - defsyms[trap_to_defsym(ttype)].explanation, + trapname(ttype, FALSE), (ttype == WEB) ? "remove" : "disarm"); } return 1; @@ -4383,7 +4381,7 @@ boolean force; ttmp = t_at(x, y); if (ttmp && !ttmp->tseen) ttmp = 0; - trapdescr = ttmp ? defsyms[trap_to_defsym(ttmp->ttyp)].explanation : 0; + trapdescr = ttmp ? trapname(ttmp->ttyp, FALSE) : 0; here = (x == u.ux && y == u.uy); /* !u.dx && !u.dy */ if (here) /* are there are one or more containers here? */ @@ -4642,7 +4640,7 @@ boolean *noticed; /* set to true iff hero notices the effect; */ } if (!trapdescr) - trapdescr = defsyms[trap_to_defsym(t->ttyp)].explanation; + trapdescr = trapname(t->ttyp, FALSE); if (!which) which = t->tseen ? the_your[t->madeby_u] : index(vowels, *trapdescr) ? "an" : "a"; @@ -5438,4 +5436,55 @@ maybe_finish_sokoban() } } +/* Return the string name of the trap type passed in, unless the player is + * hallucinating, in which case return a random or hallucinatory trap name. + * If the second argument is true, return the correct trap name even when + * hallucinating (for things like wizard mode wishing for traps and impossible + * calls). + * Originally I had intended for messages like "You begin setting the bear trap" + * to override as well, but the context in those bits of code indicated that it + * was meant to take a random name if the hero was hallucinating. + */ +const char * +trapname(ttyp, override) +int ttyp; +boolean override; +{ + const char * halu_trapnames[] = { + /* riffs on actual nethack traps */ + "bottomless pit", "polymorphism trap", "devil teleporter", + "falling boulder trap", "anti-anti-magic field", "weeping gas trap", + "queasy board", "electrified web", "owlbear trap", "sand mine", + /* some traps found in nethack variants */ + "death trap", "disintegration trap", "ice trap", "monochrome trap", + /* plausible real-life traps */ + "axeblade trap", "pool of boiling oil", "pool of quicksand", + "field of caltrops", "buzzsaw trap", "spiked floor", "revolving wall", + "uneven floor", "finger trap", "jack-in-a-box", "yellow snow", + "booby trap", "rat trap", "poisoned nail", "snare", "whirlpool", + "trip wire", + /* sci-fi */ + "negative space", "tensor field", "singularity", "imperial fleet", + "black hole", "thermal detonator", "event horizon", + "entoptic phenomenon", + /* miscellaneous suggestions */ + "sweet-smelling gas vent", "phone booth", "exploding runes", + "never-ending elevator", "slime pit", "warp zone", "illusory floor", + "pile of poo", "honey trap", "tourist trap" + }; + int total_names = TRAPNUM + SIZE(halu_trapnames); + int nameidx = rn2(total_names); + if (override || !Hallucination) { + return defsyms[trap_to_defsym(ttyp)].explanation; + } + if (nameidx < TRAPNUM) { + /* random but real trap name */ + return defsyms[trap_to_defsym(nameidx)].explanation; + } + else { + nameidx -= TRAPNUM; + return halu_trapnames[nameidx]; + } +} + /*trap.c*/ From 9b0d78d26514651154caac27160bb04adfc717bf Mon Sep 17 00:00:00 2001 From: copperwater Date: Mon, 30 Sep 2019 20:56:03 -0400 Subject: [PATCH 2/4] Port the autounlock feature from UnNetHack This adds a boolean option, autounlock, defaulting to true. When this is set to TRUE, messages stating that some door or container is locked are automatically followed by a prompt asking if you would like to unlock it, if you are carrying an unlocking tool (key, lock pick, or credit card). Architecturally, this extends the pick_lock function to take three additional arguments (door coordinates or a box on the ground you are autounlocking). Because this adds a new field to struct flag, this is not a save-compatible change. I have not adjusted EDITLEVEL or VERSION_COMPATIBILITY, though. The code that selects an unlocking tool will always look first for a skeleton key, then a lock pick, then a credit card. Since curses, rust, and other attributes don't really have an effect on the viability of the unlocking device, it didn't seem to warrant making a more complex function for that. --- include/extern.h | 2 +- include/flag.h | 1 + src/apply.c | 2 +- src/lock.c | 59 ++++++++++++++++++++++++++++++++++++------------ src/options.c | 1 + src/pickup.c | 9 ++++++++ 6 files changed, 57 insertions(+), 17 deletions(-) diff --git a/include/extern.h b/include/extern.h index 7c37297b3..e408f4845 100644 --- a/include/extern.h +++ b/include/extern.h @@ -1100,7 +1100,7 @@ E boolean FDECL(picking_at, (int, int)); E void FDECL(breakchestlock, (struct obj *, BOOLEAN_P)); E void NDECL(reset_pick); E void FDECL(maybe_reset_pick, (struct obj *)); -E int FDECL(pick_lock, (struct obj *)); +E int FDECL(pick_lock, (struct obj *, xchar, xchar, struct obj *)); E int NDECL(doforce); E boolean FDECL(boxlock, (struct obj *, struct obj *)); E boolean FDECL(doorlock, (struct obj *, int, int)); diff --git a/include/flag.h b/include/flag.h index daef72bc4..957981a57 100644 --- a/include/flag.h +++ b/include/flag.h @@ -20,6 +20,7 @@ struct flag { boolean autodig; /* MRKR: Automatically dig */ boolean autoquiver; /* Automatically fill quiver */ boolean autoopen; /* open doors by walking into them */ + boolean autounlock; /* automatically apply unlocking tools */ boolean beginner; boolean biff; /* enable checking for mail */ boolean bones; /* allow saving/loading bones */ diff --git a/src/apply.c b/src/apply.c index 11bd09d03..5bb06aef6 100644 --- a/src/apply.c +++ b/src/apply.c @@ -3606,7 +3606,7 @@ doapply() case LOCK_PICK: case CREDIT_CARD: case SKELETON_KEY: - res = (pick_lock(obj) != 0); + res = (pick_lock(obj, 0, 0, NULL) != 0); break; case PICK_AXE: case DWARVISH_MATTOCK: diff --git a/src/lock.c b/src/lock.c index 270ab1866..8645b66a1 100644 --- a/src/lock.c +++ b/src/lock.c @@ -303,14 +303,18 @@ struct obj *container; /* passed from obfree() */ /* player is applying a key, lock pick, or credit card */ int -pick_lock(pick) +pick_lock(pick, rx, ry, container) struct obj *pick; +xchar rx, ry; /* coordinates of doors/container, for autounlock: does not + prompt for direction if these are set */ +struct obj *container; /* container, for autounlock */ { int picktyp, c, ch; coord cc; struct rm *door; struct obj *otmp; char qbuf[QBUFSZ]; + boolean autounlock = (rx != 0 && ry != 0) || (container != NULL); picktyp = pick->otyp; @@ -357,8 +361,13 @@ struct obj *pick; } ch = 0; /* lint suppression */ - if (!get_adjacent_loc((char *) 0, "Invalid location!", u.ux, u.uy, &cc)) + if (rx != 0 && ry != 0) { /* autounlock; caller has provided coordinates */ + cc.x = rx; + cc.y = ry; + } + else if (!get_adjacent_loc((char *) 0, "Invalid location!", u.ux, u.uy, &cc)) { return PICKLOCK_DID_NOTHING; + } if (cc.x == u.ux && cc.y == u.uy) { /* pick lock on a container */ const char *verb; @@ -381,7 +390,9 @@ struct obj *pick; count = 0; c = 'n'; /* in case there are no boxes here */ for (otmp = level.objects[cc.x][cc.y]; otmp; otmp = otmp->nexthere) - if (Is_box(otmp)) { + /* autounlock on boxes: only the one that just informed you it was + * locked. Don't include any other boxes which might be here. */ + if ((!autounlock && Is_box(otmp)) || (otmp == container)) { ++count; if (!can_reach_floor(TRUE)) { You_cant("reach %s from up here.", the(xname(otmp))); @@ -397,17 +408,24 @@ struct obj *pick; else verb = "pick"; - /* "There is here; ?" */ - Sprintf(qsfx, " here; %s %s?", verb, it ? "it" : "its lock"); - (void) safe_qbuf(qbuf, "There is ", qsfx, otmp, doname, - ansimpleoname, "a box"); - otmp->lknown = 1; + if (autounlock) { + Sprintf(qbuf, "Unlock it with %s?", yname(pick)); + c = yn(qbuf); + if (c == 'n') + return 0; + } else { + /* "There is here; ?" */ + Sprintf(qsfx, " here; %s %s?", verb, it ? "it" : "its lock"); + (void) safe_qbuf(qbuf, "There is ", qsfx, otmp, doname, + ansimpleoname, "a box"); + otmp->lknown = 1; - c = ynq(qbuf); - if (c == 'q') - return 0; - if (c == 'n') - continue; + c = ynq(qbuf); + if (c == 'q') + return 0; + if (c == 'n') + continue; + } if (otmp->obroken) { You_cant("fix its broken lock with %s.", doname(pick)); @@ -493,8 +511,10 @@ struct obj *pick; return PICKLOCK_LEARNED_SOMETHING; } - Sprintf(qbuf, "%s it?", - (door->doormask & D_LOCKED) ? "Unlock" : "Lock"); + Sprintf(qbuf, "%s it%s%s?", + (door->doormask & D_LOCKED) ? "Unlock" : "Lock", + autounlock ? " with " : "", + autounlock ? yname(pick) : ""); c = yn(qbuf); if (c == 'n') @@ -694,6 +714,8 @@ int x, y; if (!(door->doormask & D_CLOSED)) { const char *mesg; + boolean locked = FALSE; + struct obj* unlocktool; switch (door->doormask) { case D_BROKEN: @@ -707,9 +729,16 @@ int x, y; break; default: mesg = " is locked"; + locked = TRUE; break; } pline("This door%s.", mesg); + if (locked && flags.autounlock && + ((unlocktool = carrying(SKELETON_KEY)) || + (unlocktool = carrying(LOCK_PICK)) || + (unlocktool = carrying(CREDIT_CARD)))) { + pick_lock(unlocktool, cc.x, cc.y, (struct obj *) 0); + } return res; } diff --git a/src/options.c b/src/options.c index 50f92ca86..605e42b03 100644 --- a/src/options.c +++ b/src/options.c @@ -87,6 +87,7 @@ static struct Bool_Opt { { "autoopen", &flags.autoopen, TRUE, SET_IN_GAME }, { "autopickup", &flags.pickup, TRUE, SET_IN_GAME }, { "autoquiver", &flags.autoquiver, FALSE, SET_IN_GAME }, + { "autounlock", &flags.autounlock, TRUE, SET_IN_GAME }, #if defined(MICRO) && !defined(AMIGA) { "BIOS", &iflags.BIOS, FALSE, SET_IN_FILE }, #else diff --git a/src/pickup.c b/src/pickup.c index e47d989da..e3ab9ae11 100644 --- a/src/pickup.c +++ b/src/pickup.c @@ -1720,6 +1720,7 @@ int cindex, ccount; /* index of this container (1..N), number of them (N) */ if (!cobj) return 0; if (cobj->olocked) { + struct obj *unlocktool; if (ccount < 2) pline("%s locked.", cobj->lknown ? "It is" : "Hmmm, it turns out to be"); @@ -1728,6 +1729,14 @@ int cindex, ccount; /* index of this container (1..N), number of them (N) */ else pline("Hmmm, %s turns out to be locked.", the(xname(cobj))); cobj->lknown = 1; + + if (flags.autounlock && + ((unlocktool = carrying(SKELETON_KEY)) || + (unlocktool = carrying(LOCK_PICK)) || + (unlocktool = carrying(CREDIT_CARD)))) { + /* pass ox and oy to avoid direction prompt */ + pick_lock(unlocktool, cobj->ox, cobj->oy, cobj); + } return 0; } cobj->lknown = 1; /* floor container, so no need for update_inventory() */ From ca9ccbe8179b987b2299e3ae67c38fc9dabd3d3b Mon Sep 17 00:00:00 2001 From: nhmall Date: Wed, 2 Oct 2019 13:05:08 -0400 Subject: [PATCH 3/4] reinstate anti-rng abuse bit on hallucination --- src/trap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/trap.c b/src/trap.c index 42acfb4af..59eee97bb 100644 --- a/src/trap.c +++ b/src/trap.c @@ -5454,7 +5454,7 @@ boolean override; "pile of poo", "honey trap", "tourist trap" }; int total_names = TRAPNUM + SIZE(halu_trapnames); - int nameidx = rn2(total_names); + int nameidx = rn2_on_display_rng(total_names); if (override || !Hallucination) { return defsyms[trap_to_defsym(ttyp)].explanation; } From c52058d028c2cf9f142f3f8eeebaa5b3a9c423d8 Mon Sep 17 00:00:00 2001 From: nhmall Date: Wed, 2 Oct 2019 13:20:23 -0400 Subject: [PATCH 4/4] updates to hallucinatory trap names and fixes37.0 entry --- doc/fixes37.0 | 1 + src/apply.c | 1 - src/detect.c | 5 ++--- src/display.c | 2 +- 4 files changed, 4 insertions(+), 5 deletions(-) diff --git a/doc/fixes37.0 b/doc/fixes37.0 index 2e2f93d51..d5e47aaf6 100644 --- a/doc/fixes37.0 +++ b/doc/fixes37.0 @@ -32,6 +32,7 @@ Platform- and/or Interface-Specific New Features NetHack Community Patches (or Variation) Included ------------------------------------------------- +hallucinatory trap names from github pull request #174 Code Cleanup and Reorganization diff --git a/src/apply.c b/src/apply.c index cd99d5d5b..d197be8e3 100644 --- a/src/apply.c +++ b/src/apply.c @@ -2513,7 +2513,6 @@ struct obj *otmp; You("aren't very skilled at reaching from %s.", mon_nam(u.usteed)); Sprintf(buf, "Continue your attempt to set %s?", the(trapname(ttyp, FALSE))); - .explanation)); if (yn(buf) == 'y') { if (chance) { switch (ttyp) { diff --git a/src/detect.c b/src/detect.c index 7c29ac208..f2e08aa0f 100644 --- a/src/detect.c +++ b/src/detect.c @@ -1616,8 +1616,7 @@ struct trap *trap; behaviour might need a rework in the hallucination case (e.g. to not prompt if any trap glyph appears on the square). */ if (Hallucination || - levl[trap->tx][trap->ty].glyph != - trap_to_glyph(trap, rn2_on_display_rng)) { + levl[trap->tx][trap->ty].glyph != trap_to_glyph(trap)) { /* There's too much clutter to see your find otherwise */ cls(); map_trap(trap, 1); @@ -1864,7 +1863,7 @@ int default_glyph, which_subset; || glyph_is_invisible(glyph)) && keep_traps && !covers_traps(x, y)) { if ((t = t_at(x, y)) != 0 && t->tseen) - glyph = trap_to_glyph(t, rn2_on_display_rng); + glyph = trap_to_glyph(t); } if ((glyph_is_object(glyph) && !keep_objs) || (glyph_is_trap(glyph) && !keep_traps) diff --git a/src/display.c b/src/display.c index 48e3cc96b..fc8977f01 100644 --- a/src/display.c +++ b/src/display.c @@ -228,7 +228,7 @@ register struct trap *trap; register int show; { register int x = trap->tx, y = trap->ty; - register int glyph = trap_to_glyph(trap, newsym_rn2); + register int glyph = trap_to_glyph(trap); if (g.level.flags.hero_memory) levl[x][y].glyph = glyph;