invariants are usually clearer up front

diff --git a/share/extensions/simplepath.rb b/share/extensions/simplepath.rb
index 7c634fe249b5a5892bd37b6165c0e408b51bfa4c..466a75d10faf6df068c4a0a103acf9c5fe77e002 100755 (executable)
--- a/share/extensions/simplepath.rb
+++ b/share/extensions/simplepath.rb
@@ -78,7 +78,9 @@ def parsePath(d)
     lastCommand = nil
 
     lexPath(d) do |token, isCommand|
-        unless command
+        if command
+            raise 'Invalid number of parameters' if isCommand
+        else
             if isCommand
                 raise 'Invalid path, must begin with moveto.' \
                   unless lastCommand or token.upcase == 'M'
@@ -94,8 +96,6 @@ def parsePath(d)
                 end
             end
             outputCommand = command.upcase
-        else
-            raise 'Invalid number of parameters' if isCommand
         end
 
         unless isCommand