Opened 9 years ago

Closed 9 years ago

#1335 closed bug (fixed (in master))

Definite article gives away unIDd artifacts on walkover

Reported by: magnate Owned by:
Milestone: 3.3.0 Keywords: recall
Cc:

Description

We currently reveal artifact status on pickup. This post:

http://angband.oook.cz/forum/showthread.php?p=47655#post47655

notes that if you walk over an unknown artifact and have no room to pick it up, you don't get its name but you go get the definite article ("you have no room for The longsword").

Should we make artifacts ID on walkover, or change this to "you have no room for a longsword"?

Change History (3)

comment:1 Changed 9 years ago by eddiegrove

It's also true in targeting mode, so it is definitely a bug.

As part of code cleanup, I guess, the call to make the prefix sends the mode when it should send known status. I'm doing major hacking so I can't be sure, but I think the change is

end = obj_desc_name_prefix(buf, max, end, o_ptr, known,

in object/obj-desc.c

comment:2 Changed 9 years ago by d_m

  • Status changed from new to pending

comment:3 Changed 9 years ago by GabeCunningham

  • Resolution set to fixed
  • Status changed from pending to closed

In master as of [ea04e5a].

Note: See TracTickets for help on using tickets.