Code

separate invariant check from main logic for clarity
authormental <mental@users.sourceforge.net>
Sat, 12 Aug 2006 15:34:39 +0000 (15:34 +0000)
committermental <mental@users.sourceforge.net>
Sat, 12 Aug 2006 15:34:39 +0000 (15:34 +0000)
share/extensions/simplepath.rb

index 466a75d10faf6df068c4a0a103acf9c5fe77e002..3d9b7b372142f823d406cb25db5a487d9475f16c 100755 (executable)
@@ -78,9 +78,9 @@ def parsePath(d)
     lastCommand = nil
 
     lexPath(d) do |token, isCommand|
-        if command
-            raise 'Invalid number of parameters' if isCommand
-        else
+        raise 'Invalid number of parameters' if command and isCommand
+
+        unless command
             if isCommand
                 raise 'Invalid path, must begin with moveto.' \
                   unless lastCommand or token.upcase == 'M'