sm-sorcery-bugs AT lists.ibiblio.org
Subject: Bugs for Sorcery are reported here
List archive
[SM-Sorcery-Bugs] [Bug 7294] New: spell_installed problem
- From: bugzilla-daemon AT metalab.unc.edu
- To: sm-sorcery-bugs AT lists.ibiblio.org
- Subject: [SM-Sorcery-Bugs] [Bug 7294] New: spell_installed problem
- Date: Mon, 30 Aug 2004 13:54:59 -0400
http://bugs.sourcemage.org/show_bug.cgi?id=7294
Summary: spell_installed problem
Product: Sorcery
Version: Devel
Platform: Other
OS/Version: other
Status: NEW
Severity: normal
Priority: P2
Component: subroutines
AssignedTo: sm-sorcery-bugs AT lists.ibiblio.org
ReportedBy: v.merkatz AT gmx.net
CC: eric AT sandall.us
spell_installed returns false for held spells, which isn't what several spells
that use it expect.
I see three possible solutions:
1. add spell_held to libapi so spells can use spell_installed || spell_held
2. add a function that does spell_installed || spell_held in one call
3. make spell_installed return true for held spells
I would prefer 3., since that would mean no spells need changing and for the
few
cases where code needs to do different stuff based on installed/held status
spell_held can be used in addition.
2. would also work, though that would need some changes to spells, 1. isn't
really good, since no spell cares whether something is installed or held, they
just care if the software is available.
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
- [SM-Sorcery-Bugs] [Bug 7294] New: spell_installed problem, bugzilla-daemon, 08/30/2004
Archive powered by MHonArc 2.6.24.