Date: Wed, 25 Apr 2001 00:14:50 +1000
To: Jason D Short <jshort@devon.dhs.org>
From: Donald Welsh <dwelsh@melbpc.org.au>
Subject: Re: NetHack patches

[...]

Back in November, there was a thread about the "verbose" option.  Dylan
O'Donnell commented that:

>On the other hand, it _doesn't_ suppress "A tower of flame erupts
>from the floor under the fire vortex!--More-- The fire vortex is
>uninjured." messages, so the one place it _would_ conceivably come in
>handy, it doesn't :-/

In my (unposted) reply, I wrote:

This is easy to fix.  Here's a patch.  Should it be extended to other
trap types?  Should there be a separate (noshieldeffect) option?

Apply patch, then use noverbose to suppress fire trap messages.

*** trap.c.old	Wed Sep  6 11:49:10 2000
--- trap.c	Mon Dec  4 18:02:42 2000
***************
*** 1380,1395 ****
  		case FIRE_TRAP:
   mfiretrap:
  			see_it = cansee(mtmp->mx, mtmp->my);
! 			if (in_sight)
! 			    pline("A %s erupts from the %s under %s!",
! 				  tower_of_flame,
! 				  surface(mtmp->mx,mtmp->my), mon_nam(mtmp));
! 			else if (see_it)  /* evidently `mtmp' is invisible */
! 			    You("see a %s erupt from the %s!",
! 				tower_of_flame, surface(mtmp->mx,mtmp->my));
  
  			if (resists_fire(mtmp)) {
! 			    if (in_sight) {
  				shieldeff(mtmp->mx,mtmp->my);
  				pline("%s is uninjured.", Monnam(mtmp));
  			    }
--- 1380,1396 ----
  		case FIRE_TRAP:
   mfiretrap:
  			see_it = cansee(mtmp->mx, mtmp->my);
! 			if (flags.verbose)
! 			    if (in_sight)
! 				pline("A %s erupts from the %s under %s!",
! 				      tower_of_flame,
! 				      surface(mtmp->mx,mtmp->my), mon_nam(mtmp));
! 			    else if (see_it)  /* evidently `mtmp' is invisible */
! 				You("see a %s erupt from the %s!",
! 				    tower_of_flame, surface(mtmp->mx,mtmp->my));
  
  			if (resists_fire(mtmp)) {
! 			    if (in_sight && flags.verbose) {
  				shieldeff(mtmp->mx,mtmp->my);
  				pline("%s is uninjured.", Monnam(mtmp));
  			    }