Skip to Content.
Sympa Menu

sm-sorcery-bugs - [SM-Sorcery-Bugs] [Bug 13795] New: alternative spell lookup routine

sm-sorcery-bugs AT lists.ibiblio.org

Subject: Bugs for Sorcery are reported here

List archive

Chronological Thread  
  • 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.

Top of Page