Skip to Content.
Sympa Menu

sm-sorcery-bugs - [SM-Sorcery-Bugs] [Bug 10299] gaze depends and gaze dependencies do bad argument checking

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 10299] gaze depends and gaze dependencies do bad argument checking
  • Date: 30 Dec 2005 09:57:50 -0000

http://bugs.sourcemage.org/show_bug.cgi?id=10299


jakakranjc AT email.si changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #4544 is|0 |1
obsolete| |




------- Additional Comments From jakakranjc AT email.si 2005-12-30 03:57 -------
Created an attachment (id=4548)
--> (http://bugs.sourcemage.org/attachment.cgi?id=4548&action=view)
removed the second argument check change

Uh, indeed, my mistake. I will shorten the patch.

Most gaze functions are very sensitive regarding the order of their arguments.
I was thinking of adding a function right at the parse stage of gaze, to check
for the arguments anywhere in $@, remove them and pass them on separately to
the functions along with the untuched parameters.

Many of the functions check for spell existance, not directly with
codex_does_spell_exist, but just checking for null after searches etc. This
could be unified in the previously mentioned function. It could then be used
elsewhere too. What do you think?

--
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.




Archive powered by MHonArc 2.6.24.

Top of Page