sm-sorcery-bugs AT lists.ibiblio.org
Subject: Bugs for Sorcery are reported here
List archive
[SM-Sorcery-Bugs] [Bug 13795] New: alternative spell lookup routine
- From: bugzilla-daemon AT bugs.sourcemage.org
- To: sm-sorcery-bugs AT lists.ibiblio.org
- Subject: [SM-Sorcery-Bugs] [Bug 13795] New: alternative spell lookup routine
- Date: 22 May 2007 05:49:13 -0000
http://bugs.sourcemage.org/show_bug.cgi?id=13795
Summary: alternative spell lookup routine
Product: Sorcery
Version: Untargetted future release
Platform: Other
OS/Version: other
Status: NEW
Severity: normal
Priority: P2
Component: codexroutines
AssignedTo: sm-sorcery-bugs AT lists.ibiblio.org
ReportedBy: acedit AT armory.com
Currently spell lookup is with grep over a spell index.
It seems possible to do spell lookups with glob matching instead. Im
attaching a
patch that demonstrates this. Depending on the platform the performance may
vary
more than the indexed lookup probably would. I consider this experimental.
For me it seems to work in normal usage. I tried some unit tests and couldn't
make it fail. I didnt exhaustively test for conformance with the indexed
version. I did make a pass through the code for any checks done during cache
creation.
However I noticed a difference in behavior when a spellname appears in two
different sections in the same grimoire. e.g.
/var/lib/sorcery/codex/stable/section1/dupe
/var/lib/sorcery/codex/stable/section2/dupe
With the indexed version the first occurence wins. With the new globbing
approach neither of them does, the lookup fails for the grimoire entirely.
--
Configure bugmail: http://bugs.sourcemage.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
- [SM-Sorcery-Bugs] [Bug 13795] New: alternative spell lookup routine, bugzilla-daemon, 05/22/2007
Archive powered by MHonArc 2.6.24.