Merge pull request #4 from mmiller-max/mm/fix-variable-location

Fix variable location

Author: mmiller-max

Project.toml

@@ -1,7 +1,7 @@
 name = "GraphQLParser"
 uuid = "0ae10fbf-af58-4883-b66b-ff0ac82d20dd"
 authors = ["Mal Miller and contributors"]
-version = "0.1.1"
+version = "0.1.2"
 
 [deps]
 Parsers = "69de0a69-1ddd-5017-9359-2bf0b02dc9f0"

src/parser.jl

@@ -117,8 +117,6 @@ function parse_fragment_definition(buf, pos, line, column, len)
 end
 
 function parse_variable_definitions(buf, pos, line, column, len)
-    start_line = line
-    start_column = column
     b = getbyte(buf, pos)
     if b != UInt('(') # redundant check?
         invalid("Variable definitions must start with '('", buf, pos)
@@ -130,6 +128,8 @@ function parse_variable_definitions(buf, pos, line, column, len)
 
     variables_defs = VariableDefinition[]
     while b != UInt(')')
+        start_line = line
+        start_column = column
         if b != UInt('$')
             invalid("Variable name must start with '\$'", buf, pos)
         end