Viewing Issue Simple Details Jump to Notes ] Wiki ] View Advanced ] Issue History ] Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0008006 [DCSS] Patches minor have not tried 2014-01-20 01:48 2014-01-20 18:43
Reporter Bodrick View Status public  
Assigned To neil
Priority normal Resolution done  
Status resolved   Product Branch 0.14 ancient branch
Summary 0008006: Added Angry indicator to the % screen.
Description This replaces the clarity indicator if you have a *Rage item or berserkitis, but don't have clarity or stasis.
Additional Information
Tags No tags attached.
Attached Files ? file icon 0001-Added-Angry-indicator-to-the-screen.patch [^] (1,491 bytes) 2014-01-20 01:48 [Show Content]

- Relationships

-  Notes
(0025030)
Cedor (reporter)
2014-01-20 10:02

was working on this some days ago, so I have some pure technical comments : instead of testing you.clarity() and you.stasis(), you should call it into some variables (as you'll do it later with rclar and stasis) and use theses variables instead of methods calling (it's faster and less ressources consuming)
(0025036)
neil (administrator)
2014-01-20 18:43

In trunk (0.14-a0-2048-g1234845), thanks! In subsequent commits I applied the changes Cedor suggested, and changed the label to "Rnd*Rage" following a discussion in ##crawl-dev.

- Issue History
Date Modified Username Field Change
2014-01-20 01:48 Bodrick New Issue
2014-01-20 01:48 Bodrick File Added: 0001-Added-Angry-indicator-to-the-screen.patch
2014-01-20 10:02 Cedor Note Added: 0025030
2014-01-20 15:00 Cedor Issue Monitored: Cedor
2014-01-20 18:43 neil Note Added: 0025036
2014-01-20 18:43 neil Status new => resolved
2014-01-20 18:43 neil Fixed in Branch => 0.14 development branch
2014-01-20 18:43 neil Resolution open => done
2014-01-20 18:43 neil Assigned To => neil


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