Source:NetHack 3.4.3/include/obj.h

From NetHackWiki
Jump to navigation Jump to search

Below is the full text to include/obj.h from NetHack 3.4.3. To link to a particular line, write Source:NetHack 3.4.3/include/obj.h#line123, for example.

This particular file contains C-language declarations for objects. In particular, it declares struct obj, the data structure representing an object in the dungeon. This data structure contains many fields about the object, such as the type of the object, the BUC status, the number of charges, the erosion, the quantity of this object in a stack, the pointer to the monster presently carrying it, the flag indicating whether the player has identified the object, and many other fields.

For the purposes of this annotation, we declare a pointer to an example object: struct obj *thing;

Top of file

/*	SCCS Id: @(#)obj.h	3.4	2002/01/07	*/
/* Copyright (c) Stichting Mathematisch Centrum, Amsterdam, 1985. */
/* NetHack may be freely redistributed.  See license for details. */

#ifndef OBJ_H
#define OBJ_H

/* #define obj obj_nh */ /* uncomment for SCO UNIX, which has a conflicting
			  * typedef for "obj" in <sys/types.h> */

union vptrs

union vptrs {
	    struct obj *v_nexthere;	/* floor location lists */
	    struct obj *v_ocontainer;	/* point back to container */
	    struct monst *v_ocarry;	/* point back to carrying monst */
};

Each struct obj (defined in the next section) contains a pointer called v, for example thing->v. Depending on where (thing->where) the object is, this pointer can have one of three meanings:

  • thing->v.v_nexthere, for an object on the dungeon floor, is the next object below this one in the pile.
  • thing->v.v_ocontainer, for an object in a container such as a chest or bag, points to that containing object.
  • thing->v.v_ocarry, for an object in the inventory of a monster, points to that monster.

The header file will soon define macros that allow you to say:

  • thing->nexthere for the next object in the pile
  • thing->ocontainer for the containing object
  • thing->ocarry for the carrying monster

Remember, it is a C union, so all three of these pointers share the same memory; you can use only one pointer at a time. For example, an object cannot simultaneously be on the floor and be carried by a monster.

struct obj

struct obj {
	struct obj *nobj;

Each object contains a pointer like thing->nobj which allows the object to be part of a linked list. (What does NetHack use this list for?)

	union vptrs v;
#define nexthere	v.v_nexthere
#define ocontainer	v.v_ocontainer
#define ocarry		v.v_ocarry

This declares the pointer v and the three macros explained in the #union vptrs section above.

	struct obj *cobj;	/* contents list for containers */
	unsigned o_id;
	xchar ox,oy;
	short otyp;		/* object class number */
	unsigned owt;
	long quan;		/* number of items */
  • thing->cobj is a linked list of objects contained by thing, for example if the thing is a bag or statue.
  • thing->o_id is a unique identifier for this object. It's set when the object is generated, and does not change. (Uniqueness across _all_ objects in the game is not guaranteed)
  • thing->ox and thing->oy are object coordinates on the map.
  • thing->otyp is the object type from onames.h. Those definitions are generated when compiling, each object definition in objects.c gets one.
  • thing->owt is the total weight of this object. use weight() in mkobj.c to calculate this.

Some items stack. That is when multiple items take only one spot in your inventory or other lists, for example, "2 daggers" or "7 candles". If thing->quan is 2 or 7, then 2 or 7 of those things are stacked.

	schar spe;		/* quality of weapon, armor or ring (+ or -)
				   number of charges for wand ( >= -1 )
				   marks your eggs, spinach tins
				   royal coffers for a court ( == 2)
				   tells which fruit a fruit is
				   special for uball and amulet
				   historic and gender for statues */
#define STATUE_HISTORIC 0x01
#define STATUE_MALE     0x02
#define STATUE_FEMALE   0x04

This gives a special statistic for certain types of items.

  • If thing was a +1 elven dagger, then thing->spe is 1.
  • If it is a wand of wishing (0:3), a wand with three charges, then thing->spe is 3.
  • This statistic also provides certain information about eggs, spinach, fruit, iron balls (?), and amulets.
  • For statues, this field is a bit mask. If thing->spe & STATUE_HISTORIC, then the statue is historic. Likewise, a statue can be male or female. It can be simultaneously historic, male, and female, though the last two might not combine well.
	char	oclass;		/* object class */
	char	invlet;		/* designation in inventory */
	char	oartifact;	/* artifact array index */
  • thing->oclass is one of the _CLASS definitions from objclass.h, eg. WEAPON_CLASS or POTION_CLASS, and is the class that represents the object on the screen.
  • An object in your inventory will have a letter [a-zA-Z], stored for example at thing->invlet. Even after an object is dropped, the object will try to remember its old inventory letter. Suppose you drop a spellbook which was inventory letter "B". If you retake the spellbook, the fixinv option is on, and the letter "B" is available, then the spellbook will again be letter "B" in your inventory.
  • NetHack has a separate array for artifacts, defined in artilist.h

The hero should be memorising the inventory letter, not the object! But internal to the game, NetHack stores the letter in the object, not the adventurer. This works because this is a single-player game.

	xchar where;		/* where the object thinks it is */
#define OBJ_FREE	0		/* object not attached to anything */
#define OBJ_FLOOR	1		/* object on floor */
#define OBJ_CONTAINED	2		/* object in a container */
#define OBJ_INVENT	3		/* object in the hero's inventory */
#define OBJ_MINVENT	4		/* object in a monster inventory */
#define OBJ_MIGRATING	5		/* object sent off to another level */
#define OBJ_BURIED	6		/* object buried */
#define OBJ_ONBILL	7		/* object on shk bill */
#define NOBJ_STATES	8

So thing->where indicates where this object is. A "shk" is a shopkeeper. If thing is in your inventory, then thing->where is either OBJ_INVENT or OBJ_ONBILL. A for loop that enumerated these locations would be like for(int i = 0; i < NOBJ_STATES; i++).

	xchar timed;		/* # of fuses (timers) attached to this obj */

Timers are for rotting eggs and corpses, for example. Do not change this by hand, rather use the code in timeout.c, see start_timer()

	Bitfield(cursed,1);
	Bitfield(blessed,1);
	Bitfield(unpaid,1);	/* on some bill */
	Bitfield(no_charge,1);	/* if shk shouldn't charge for this */
	Bitfield(known,1);	/* exact nature known */
	Bitfield(dknown,1);	/* color or text known */
	Bitfield(bknown,1);	/* blessing or curse known */
	Bitfield(rknown,1);	/* rustproof or not known */

Bitfields use less than one byte in a struct. Most of these are obvious, like cursed or blessed. Four of these indicate what the adventurer knows about this object. In a game with more than one adventurer, these fields could not be stored in the object. Programming multiplayer NetHack would require many changes.

	Bitfield(oeroded,2);	/* rusted/burnt weapon/armor */
	Bitfield(oeroded2,2);	/* corroded/rotted weapon/armor */
#define greatest_erosion(otmp) (int)((otmp)->oeroded > (otmp)->oeroded2 ? (otmp)->oeroded : (otmp)->oeroded2)
#define MAX_ERODE 3
#define orotten oeroded		/* rotten food */
#define odiluted oeroded	/* diluted potions */
#define norevive oeroded2

Objects can erode in two different ways.

  • thing->oeroded (or orotten or odiluted) indicates rusty metal or burnt organic material. For food such as corpses, this indicates how rotten it is. Also, NetHack uses this to mark a potion as diluted.
  • thing->oeroded2 (or norevive) indicated corroded metal or rotted organic material. Sometimes acid causes this damage. The thing->norevive is probably a flag for NetHack to check whether undead turning would revive something, but we would need to check.

Because thing might erode two different ways, a macro call to greatest_erosion returns the greater of the two erosions. There is a scale from 0 (no erosion) to 1 (eroded) to 2 (very eroded) to 3 (thoroughly eroded).

	Bitfield(oerodeproof,1); /* erodeproof weapon/armor */
	Bitfield(olocked,1);	/* object is locked */
	Bitfield(obroken,1);	/* lock has been broken */
	Bitfield(otrapped,1);	/* container is trapped */
				/* or accidental tripped rolling boulder trap */
#define opoisoned otrapped	/* object (weapon) is coated with poison */

Here are some more boolean bits.

  • Even though an object might erode two ways, there is exactly one "erodeproof" flag - setting it prevents both types of erosion. This is why using the scroll of enchant weapon while confused will erodeproof any weapon. If obj->rknown is true than the code in objnam.c will refer to an erodeproof object as "fixed" if it is a crysknife, else rustproof if the metal would normally rust, else corrodeproof if the metal would normally corrode, else fireproof if the object would be flammable. For example, an erodeproof dragon scale mail would never be marked as such in your inventory, because dragon scales are neither crysknives nor rustprone nor corrodeable nor flammable.
  • thing->olocked is set if an object such a chest or box is locked; thing->obroken if the lock is broken (though you could fix it with wizard lock).
  • thing->otrapped (or synonym thing->opoisoned) indicates that a weapon is poisoned or that the chest or box contains a trap (which might be a flame or poison needle when you open it).
	Bitfield(recharged,3);	/* number of times it's been recharged */
	Bitfield(lamplit,1);	/* a light-source -- can be lit */
#ifdef INVISIBLE_OBJECTS
	Bitfield(oinvis,1);	/* invisible */
#endif
	Bitfield(greased,1);	/* covered with grease */
	Bitfield(oattached,2);	/* obj struct has special attachment */
#define OATTACHED_NOTHING 0
#define OATTACHED_MONST   1	/* monst struct in oextra */
#define OATTACHED_M_ID    2	/* monst id in oextra */
#define OATTACHED_UNUSED3 3
  • thing->recharged is the number of times that something was recharged, for example if thing is a wand or the Bell of Opening. This can range from 0 to 7, though most wands explode much sooner than that. This field complements thing->spe which is the number of charges in the wand.
  • thing->lamplit is set if thing is a lamp, and you switch it on. You can switch on a lamp by applying it.
  • thing->oinvis can be set for an invisible object. NetHack 3.4.3 does not implement invisible objects, even though it has some code for it, so you will not actually find this field set in a game.
  • thing->greased if it is greased.
  • thing->oattached is a number from 0 to 3 (one of the defined constants OATTACHED_NOTHING, ...) to indicate the meaning of thing->oextra, which is defined at the end of this struct.
	Bitfield(in_use,1);	/* for magic items before useup items */
	Bitfield(bypass,1);	/* mark this as an object to be skipped by bhito() */
	/* 6 free bits */

...

	int	corpsenm;	/* type of corpse is mons[corpsenm] */
#define leashmon  corpsenm	/* gets m_id of attached pet */
#define spestudied corpsenm	/* # of times a spellbook has been studied */
#define fromsink  corpsenm	/* a potion from a sink */
	unsigned oeaten;	/* nutrition left in food, if partly eaten */
	long age;		/* creation date */

The "corpsenm", "leashmon", "spestudied", and "fromsink" all refer to the same field of the object. This works because an object can only be one of a corpse, leash, spellbook, or potion. (It might be possible to abolish this field and just widen thing->spe from schar to int, unless someone can find an object that needs to use both fields.)

	uchar onamelth;		/* length of name (following oxlth) */
	short oxlth;		/* length of following data */
	/* in order to prevent alignment problems oextra should
	   be (or follow) a long int */
	long owornmask;
	long oextra[1];		/* used for name of ordinary objects - length
				   is flexible; amount for tmp gold objects */
};

As C is not an object-oriented language with inheritance, the programmers decided to use this hack to allow objects to have extra information. A long is 32 bits on some computers but possibly 64 bits on others.

  • thing->oextra is usually a pointer to a monst struct, but this depends what thing->oattached (above) is set to.
  • thing->oxlth is the length of the data pointed to by thing->oextra

useful macros

#define newobj(xl)	(struct obj *)alloc((unsigned)(xl) + sizeof(struct obj))
#define ONAME(otmp)	(((char *)(otmp)->oextra) + (otmp)->oxlth)
  • newobj uses the alloc function of NetHack to dynamically allocate memory. For example, thing = newobj(10) allocates an object with 10 extra bytes, and stores a pointer to that object in thing.
  • ONAME(thing) returns a character pointer to text that NetHack can store in the extra memory after an object. Recall that thing->oextra[0] is the "long int" defined at the end of the struct, thus thing->oextra is a pointer into the end of the struct. Either thing->oxlth is zero, or there is extra information before the text that we need to skip.

weapons

/* Weapons and weapon-tools */
/* KMH -- now based on skill categories.  Formerly:
*	#define is_sword(otmp)	(otmp->oclass == WEAPON_CLASS && \
*			 objects[otmp->otyp].oc_wepcat == WEP_SWORD)
*	#define is_blade(otmp)	(otmp->oclass == WEAPON_CLASS && \
*			 (objects[otmp->otyp].oc_wepcat == WEP_BLADE || \
*			  objects[otmp->otyp].oc_wepcat == WEP_SWORD))
*	#define is_weptool(o)	((o)->oclass == TOOL_CLASS && \
*			 objects[(o)->otyp].oc_weptool)
*	#define is_multigen(otyp) (otyp <= SHURIKEN)
*	#define is_poisonable(otyp) (otyp <= BEC_DE_CORBIN)
*/
#define is_blade(otmp)	(otmp->oclass == WEAPON_CLASS && \
			 objects[otmp->otyp].oc_skill >= P_DAGGER && \
			 objects[otmp->otyp].oc_skill <= P_SABER)
#define is_axe(otmp)	((otmp->oclass == WEAPON_CLASS || \
			 otmp->oclass == TOOL_CLASS) && \
			 objects[otmp->otyp].oc_skill == P_AXE)
#define is_pick(otmp)	((otmp->oclass == WEAPON_CLASS || \
			 otmp->oclass == TOOL_CLASS) && \
			 objects[otmp->otyp].oc_skill == P_PICK_AXE)
#define is_sword(otmp)	(otmp->oclass == WEAPON_CLASS && \
			 objects[otmp->otyp].oc_skill >= P_SHORT_SWORD && \
			 objects[otmp->otyp].oc_skill <= P_SABER)
#define is_pole(otmp)	((otmp->oclass == WEAPON_CLASS || \
			otmp->oclass == TOOL_CLASS) && \
			 (objects[otmp->otyp].oc_skill == P_POLEARMS || \
			 objects[otmp->otyp].oc_skill == P_LANCE))
#define is_spear(otmp)	(otmp->oclass == WEAPON_CLASS && \
			 objects[otmp->otyp].oc_skill >= P_SPEAR && \
			 objects[otmp->otyp].oc_skill <= P_JAVELIN)
#define is_launcher(otmp)	(otmp->oclass == WEAPON_CLASS && \
			 objects[otmp->otyp].oc_skill >= P_BOW && \
			 objects[otmp->otyp].oc_skill <= P_CROSSBOW)
#define is_ammo(otmp)	((otmp->oclass == WEAPON_CLASS || \
			 otmp->oclass == GEM_CLASS) && \
			 objects[otmp->otyp].oc_skill >= -P_CROSSBOW && \
			 objects[otmp->otyp].oc_skill <= -P_BOW)
#define ammo_and_launcher(otmp,ltmp) \
			 (is_ammo(otmp) && (ltmp) && \
			 objects[(otmp)->otyp].oc_skill == -objects[(ltmp)->otyp].oc_skill)
#define is_missile(otmp)	((otmp->oclass == WEAPON_CLASS || \
			 otmp->oclass == TOOL_CLASS) && \
			 objects[otmp->otyp].oc_skill >= -P_BOOMERANG && \
			 objects[otmp->otyp].oc_skill <= -P_DART)
#define is_weptool(o)	((o)->oclass == TOOL_CLASS && \
			 objects[(o)->otyp].oc_skill != P_NONE)
#define bimanual(otmp)	((otmp->oclass == WEAPON_CLASS || \
			 otmp->oclass == TOOL_CLASS) && \
			 objects[otmp->otyp].oc_bimanual)
#define is_multigen(otmp)	(otmp->oclass == WEAPON_CLASS && \
			 objects[otmp->otyp].oc_skill >= -P_SHURIKEN && \
			 objects[otmp->otyp].oc_skill <= -P_BOW)
#define is_poisonable(otmp)	(otmp->oclass == WEAPON_CLASS && \
			 objects[otmp->otyp].oc_skill >= -P_SHURIKEN && \
			 objects[otmp->otyp].oc_skill <= -P_BOW)
#define uslinging()	(uwep && objects[uwep->otyp].oc_skill == P_SLING)

These macros act as functions to test various properties of weapons. Some of these work by querying the class of the object, others look in the objects global array for more information about objects of that type.

For example, a "launcher" (is_launcher(thing)) is any weapon that needs your "bow" or "crossbow" skill. That is, if you can use your bow or crossbow skill with the object, then the object qualifies as a bow or crossbow for the purposes of the game. So wield it, quiver those arrows or crossbow bolts, and fire!

armor

/* Armor */
#define is_shield(otmp) (otmp->oclass == ARMOR_CLASS && \
			 objects[otmp->otyp].oc_armcat == ARM_SHIELD)
#define is_helmet(otmp) (otmp->oclass == ARMOR_CLASS && \
			 objects[otmp->otyp].oc_armcat == ARM_HELM)
#define is_boots(otmp)	(otmp->oclass == ARMOR_CLASS && \
			 objects[otmp->otyp].oc_armcat == ARM_BOOTS)
#define is_gloves(otmp) (otmp->oclass == ARMOR_CLASS && \
			 objects[otmp->otyp].oc_armcat == ARM_GLOVES)
#define is_cloak(otmp)	(otmp->oclass == ARMOR_CLASS && \
			 objects[otmp->otyp].oc_armcat == ARM_CLOAK)
#define is_shirt(otmp)	(otmp->oclass == ARMOR_CLASS && \
			 objects[otmp->otyp].oc_armcat == ARM_SHIRT)
#define is_suit(otmp)	(otmp->oclass == ARMOR_CLASS && \
			 objects[otmp->otyp].oc_armcat == ARM_SUIT)
#define is_elven_armor(otmp)	((otmp)->otyp == ELVEN_LEATHER_HELM\
				|| (otmp)->otyp == ELVEN_MITHRIL_COAT\
				|| (otmp)->otyp == ELVEN_CLOAK\
				|| (otmp)->otyp == ELVEN_SHIELD\
				|| (otmp)->otyp == ELVEN_BOOTS)
#define is_orcish_armor(otmp)	((otmp)->otyp == ORCISH_HELM\
				|| (otmp)->otyp == ORCISH_CHAIN_MAIL\
				|| (otmp)->otyp == ORCISH_RING_MAIL\
				|| (otmp)->otyp == ORCISH_CLOAK\
				|| (otmp)->otyp == URUK_HAI_SHIELD\
				|| (otmp)->otyp == ORCISH_SHIELD)
#define is_dwarvish_armor(otmp)	((otmp)->otyp == DWARVISH_IRON_HELM\
				|| (otmp)->otyp == DWARVISH_MITHRIL_COAT\
				|| (otmp)->otyp == DWARVISH_CLOAK\
				|| (otmp)->otyp == DWARVISH_ROUNDSHIELD)
#define is_gnomish_armor(otmp)	(FALSE)

Each type of object has information about it in the global objects array. This information includes the armor category: shield, helm, boots, gloves, cloak, shirt. The game prevents you from wearing two helms simultaneously, which is why it needs to know about these classes. The is_helmet(thing) macro tests if thing qualifies as a helm. If it does, and you are already wearing a helment, then you cannot wear thing.

Eggs and other food

/* Eggs and other food */
#define MAX_EGG_HATCH_TIME 200	/* longest an egg can remain unhatched */
#define stale_egg(egg)	((monstermoves - (egg)->age) > (2*MAX_EGG_HATCH_TIME))
#define ofood(o) ((o)->otyp == CORPSE || (o)->otyp == EGG || (o)->otyp == TIN)
#define polyfodder(obj) (ofood(obj) && \
			 pm_to_cham((obj)->corpsenm) != CHAM_ORDINARY)
#define mlevelgain(obj) (ofood(obj) && (obj)->corpsenm == PM_WRAITH)
#define mhealup(obj)	(ofood(obj) && (obj)->corpsenm == PM_NURSE)

These are some macros related to eggs (which might hatch!) and food. It is apparent from this code which corpse will level you up or heal you when you eat it. No other corpse but that of a wraith levels you up; no other corpse but that of a nurse heals you.

Containers

/* Containers */
#define carried(o)	((o)->where == OBJ_INVENT)
#define mcarried(o)	((o)->where == OBJ_MINVENT)
#define Has_contents(o) (/* (Is_container(o) || (o)->otyp == STATUE) && */ \
			 (o)->cobj != (struct obj *)0)
#define Is_container(o) ((o)->otyp >= LARGE_BOX && (o)->otyp <= BAG_OF_TRICKS)
#define Is_box(otmp)	(otmp->otyp == LARGE_BOX || otmp->otyp == CHEST)
#define Is_mbag(otmp)	(otmp->otyp == BAG_OF_HOLDING || \
			 otmp->otyp == BAG_OF_TRICKS)

There is are macros to check if an object is in your inventory or that of a monster. It is slightly easier to say carried(thing) than thing->where == OBJ_INVENT. There are also several macros for containers.

An "mbag" (magic bag) is either a bag of holding or bag of tricks; the trait is that both bags magically adjust the weight of their contents. Normally, these bags feel lighter than the actual weight of the items or monsters inside, but a curse could cause a problem.

Items from dragons, dwarves, elves, gnomes

/* dragon gear */
#define Is_dragon_scales(obj)	((obj)->otyp >= GRAY_DRAGON_SCALES && \
				 (obj)->otyp <= YELLOW_DRAGON_SCALES)
#define Is_dragon_mail(obj)	((obj)->otyp >= GRAY_DRAGON_SCALE_MAIL && \
				 (obj)->otyp <= YELLOW_DRAGON_SCALE_MAIL)
#define Is_dragon_armor(obj)	(Is_dragon_scales(obj) || Is_dragon_mail(obj))
#define Dragon_scales_to_pm(obj) &mons[PM_GRAY_DRAGON + (obj)->otyp \
				       - GRAY_DRAGON_SCALES]
#define Dragon_mail_to_pm(obj)	&mons[PM_GRAY_DRAGON + (obj)->otyp \
				      - GRAY_DRAGON_SCALE_MAIL]

These macros facilitate code that needs to know about dragon scales, dragon scale mail and their matching dragons. It assumes that the range of dragon colors starts with gray and ends with yellow. The obj parameter is a pointer to an object, but these macros only examine the object type.

If a monster does polymorph while wearing dragon scales or dragon scale mail, the code uses Dragon_scales_to_pm or Dragon_mail_to_pm to polymorph you into the matching dragon: mon.c#line2351, muse.c#line1684. Then the game disables the message that the dragon would break out of its armor: worn.c#line605. (The code in polyself.c uses a separate function called armor_to_dragon.)

#define Dragon_to_scales(pm)	(GRAY_DRAGON_SCALES + (pm - mons))

Do not use the Dragon_to_scales macro! The DevTeam left this broken and incorrect definition of a macro, but never called this macro from the NetHack source code. If pm is a monster type (an index into the mons array), then the correct version of the macro would expand to (GRAY_DRAGON_SCALES + (pm - PM_GRAY_DRAGON)). NetHack uses the correct formula at mon.c#line194 to pick the correct scales for the death drop of a dragon.

/* Elven gear */
#define is_elven_weapon(otmp)	((otmp)->otyp == ELVEN_ARROW\
				|| (otmp)->otyp == ELVEN_SPEAR\
				|| (otmp)->otyp == ELVEN_DAGGER\
				|| (otmp)->otyp == ELVEN_SHORT_SWORD\
				|| (otmp)->otyp == ELVEN_BROADSWORD\
				|| (otmp)->otyp == ELVEN_BOW)
#define is_elven_obj(otmp)	(is_elven_armor(otmp) || is_elven_weapon(otmp))

/* Orcish gear */
#define is_orcish_obj(otmp)	(is_orcish_armor(otmp)\
				|| (otmp)->otyp == ORCISH_ARROW\
				|| (otmp)->otyp == ORCISH_SPEAR\
				|| (otmp)->otyp == ORCISH_DAGGER\
				|| (otmp)->otyp == ORCISH_SHORT_SWORD\
				|| (otmp)->otyp == ORCISH_BOW)

/* Dwarvish gear */
#define is_dwarvish_obj(otmp)	(is_dwarvish_armor(otmp)\
				|| (otmp)->otyp == DWARVISH_SPEAR\
				|| (otmp)->otyp == DWARVISH_SHORT_SWORD\
				|| (otmp)->otyp == DWARVISH_MATTOCK)

/* Gnomish gear */
#define is_gnomish_obj(otmp)	(is_gnomish_armor(otmp))

These macros tests if an object is elven, orcish, dwarvish or gnomish. (How is this important, and where are the definitions of the is_foo_armor macros?) These macros are boolean expressions that check if the object is of one type or another; the macro for is_elven_weapon literally checks the object against the macro's list of six known elven weapons. If you added a new elven weapon to src/objects.c, then you would also have to add it to this macro.

One may consider refactoring this, so that struct objclass in objclass.h contains new bitfields oc_elven, oc_orcish, oc_dwarvish, oc_gnomish. Then one would have macros like so,

/* not part of NetHack */
#define is_elven_weapon(otmp) (is_elven_object(otmp) && objects[otmp->otyp].oc_class == WEAPON_CLASS)
#define is_elven_object(otmp) (objects[otmp->otyp].oc_elven)

Then we could adjust the macros in objects.c to clear the oc_orcish on most objects, but to set the flag on orcish objects. The advantage is that it slightly easier for a variant to add a new type of orcish object to the game. The disadvantage is that the vast majority of object types are neither elven nor orcish nor dwarvish nor gnomish, so we would be allocating four extra zero bits (negligible with yesterday's computers) for each object type.

Candles, lamps

/* Light sources */
#define Is_candle(otmp) (otmp->otyp == TALLOW_CANDLE || \
			 otmp->otyp == WAX_CANDLE)
#define MAX_OIL_IN_FLASK 400	/* maximum amount of oil in a potion of oil */

/* MAGIC_LAMP intentionally excluded below */
/* age field of this is relative age rather than absolute */
#define age_is_relative(otmp)	((otmp)->otyp == BRASS_LANTERN\
				|| (otmp)->otyp == OIL_LAMP\
				|| (otmp)->otyp == CANDELABRUM_OF_INVOCATION\
				|| (otmp)->otyp == TALLOW_CANDLE\
				|| (otmp)->otyp == WAX_CANDLE\
				|| (otmp)->otyp == POT_OIL)
/* object can be ignited */
#define ignitable(otmp)	((otmp)->otyp == BRASS_LANTERN\
				|| (otmp)->otyp == OIL_LAMP\
				|| (otmp)->otyp == CANDELABRUM_OF_INVOCATION\
				|| (otmp)->otyp == TALLOW_CANDLE\
				|| (otmp)->otyp == WAX_CANDLE\
				|| (otmp)->otyp == POT_OIL)

Stones and other miscellaneous

/* special stones */
#define is_graystone(obj)	((obj)->otyp == LUCKSTONE || \
				 (obj)->otyp == LOADSTONE || \
				 (obj)->otyp == FLINT     || \
				 (obj)->otyp == TOUCHSTONE)

/* misc */
#ifdef KOPS
#define is_flimsy(otmp)		(objects[(otmp)->otyp].oc_material <= LEATHER || \
				 (otmp)->otyp == RUBBER_HOSE)
#else
#define is_flimsy(otmp)		(objects[(otmp)->otyp].oc_material <= LEATHER)
#endif

/* helpers, simple enough to be macros */
#define is_plural(o)	((o)->quan > 1 || \
			 (o)->oartifact == ART_EYES_OF_THE_OVERWORLD)

/* Flags for get_obj_location(). */
#define CONTAINED_TOO	0x1
#define BURIED_TOO	0x2

#endif /* OBJ_H */