Crossfire Mailing List Archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Mailing list problems



The crossfire-request address for subscribing to the mailing list seems to be 
currently non-operational. I've tried subscribing using the given instructions
using both dial-u and work e-mail addresses without getting any sort
of repsonse. I see there was some discussion about this in the archives and 
just wish to post another couple of data-points if anybody's looking at the problem.

Just so this is not a complete waste of everyones time: There seems to be a 
one-line thinko in the function examine_monster in server/input.c (checked 
in the latest cvs snapshot). Before it reports on the health of the monster, 
it checks if monster is of type PLAYER. Surely this should be if the caller is
of type player?

The attached patch gives the correct behaviour. Most noticably, it affects the
probe spell, which fails to report monster health with the existing code.

-- 

Neil Muller

*** server/input.c.orig	Mon May 22 23:47:55 2000
--- server/input.c	Mon May 22 23:48:23 2000
***************
*** 117,127 ****
      new_draw_info(NDI_UNIQUE, 0,op,"It is likely less powerful than you.");
    else
      new_draw_info(NDI_UNIQUE, 0,op,"It is probably as powerful as you.");
    if(mon->attacktype&AT_ACID)
      new_draw_info(NDI_UNIQUE, 0,op,"You seem to smell an acrid odor.");
!   if(tmp->type!=PLAYER)
      return;
    switch((mon->stats.hp+1)*4/(at->clone.stats.hp+1)) { /* From 1-4 */
    case 1:
      new_draw_info(NDI_UNIQUE, 0,op,"It is in a bad shape.");
      break;
--- 117,127 ----
      new_draw_info(NDI_UNIQUE, 0,op,"It is likely less powerful than you.");
    else
      new_draw_info(NDI_UNIQUE, 0,op,"It is probably as powerful as you.");
    if(mon->attacktype&AT_ACID)
      new_draw_info(NDI_UNIQUE, 0,op,"You seem to smell an acrid odor.");
!   if(op->type!=PLAYER)
      return;
    switch((mon->stats.hp+1)*4/(at->clone.stats.hp+1)) { /* From 1-4 */
    case 1:
      new_draw_info(NDI_UNIQUE, 0,op,"It is in a bad shape.");
      break;