Viewing Issue Simple Details Jump to Notes ] Wiki ] View Advanced ] Issue History ] Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0007887 [DCSS] Bug Report trivial always 2013-12-19 14:46 2013-12-28 11:20
Reporter nagdon View Status public  
Assigned To neil
Priority normal Resolution done  
Status closed   Product Branch 0.14 ancient branch
Summary 0007887: two minor bugs in mons_has_incapacitating_ranged_attack()
Description In mons_has_incapacitating_ranged_attack() (mon-util.cc line 3341), in the "switch (get_ammo_brand(*missile))" statement the case of SPMSL_CURARE is handled buggily:
 * it returns true "if (!foe->res_poison())", but res_poison can return negative values meaning poison vulnerability and it should return true also in those cases.
 * the "break" is missing after the case, so in the case of curare missles the function also returns true when the foe can sleep (as this is tested in the next, SPMSL_SLEEP case)

Fortunately this code seems to be only used by M_STABBER flagged monsters (spriggan assassin and enchanter), which are rare.
Additional Information
Tags No tags attached.
Attached Files

- Relationships

-  Notes
(0024858)
neil (administrator)
2013-12-27 20:41

Fixed in trunk (0.14-a0-1659-ga5703a3), thanks!

- Issue History
Date Modified Username Field Change
2013-12-19 14:46 nagdon New Issue
2013-12-27 20:41 neil Note Added: 0024858
2013-12-27 20:41 neil Status new => resolved
2013-12-27 20:41 neil Fixed in Branch => 0.14 development branch
2013-12-27 20:41 neil Resolution open => done
2013-12-27 20:41 neil Assigned To => neil
2013-12-28 11:20 nagdon Status resolved => closed


Mantis 1.1.8[^]
Copyright © 2000 - 2009 Mantis Group
Powered by Mantis Bugtracker