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: 29 Dec 2005 23:30:48 -0000

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





------- Additional Comments From acedit AT armory.com 2005-12-29 17:30 -------
most argument parsing happens in these 'wierd while loops'...I dont know why
you
think theres a problem with them.

It is true that the first loop is pointless if theres only one argument, but
as
soon as there was another we'd just have to add it back again. The second loop
allows both arguments to be used in either order, your fix makes no sense to
me.
If I specify --no-optionals -c spell, then it would ignore -c (actually it
looks
like it would treat that as the spell), theres a few other things I could do
which would work fine with the existing loop but behave inconsistently without
the loop. I really dont see why this is better.

Its somewhat useful to check for spell existence. Perhaps should make all gaze
options check for this and have a single patch fix all of them at once.

Lastly, I know its hard, but one problem per bug please. This is technically
two
completely seperate issues.

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