prism 0.13.0 → 0.15.0

data/lib/prism/node_ext.rb

@@ -52,4 +52,48 @@ module Prism
       o
     end
   end
+
+  class ConstantReadNode < Node
+    # Returns the list of parts for the full name of this constant. For example: [:Foo]
+    def full_name_parts
+      [name]
+    end
+
+    # Returns the full name of this constant. For example: "Foo"
+    def full_name
+      name.name
+    end
+  end
+
+  class ConstantPathNode < Node
+    # Returns the list of parts for the full name of this constant path. For example: [:Foo, :Bar]
+    def full_name_parts
+      parts = [child.name]
+      current = parent
+
+      while current.is_a?(ConstantPathNode)
+        parts.unshift(current.child.name)
+        current = current.parent
+      end
+
+      parts.unshift(current&.name || :"")
+    end
+
+    # Returns the full name of this constant path. For example: "Foo::Bar"
+    def full_name
+      full_name_parts.join("::")
+    end
+  end
+
+  class ConstantPathTargetNode < Node
+    # Returns the list of parts for the full name of this constant path. For example: [:Foo, :Bar]
+    def full_name_parts
+      (parent&.full_name_parts || [:""]).push(child.name)
+    end
+
+    # Returns the full name of this constant path. For example: "Foo::Bar"
+    def full_name
+      full_name_parts.join("::")
+    end
+  end
 end

data/lib/prism/parse_result.rb

@@ -117,7 +117,7 @@ module Prism
     end
 
     def pretty_print(q)
-      q.text("(#{start_line},#{start_column})-(#{end_line},#{end_column}))")
+      q.text("(#{start_line},#{start_column})-(#{end_line},#{end_column})")
     end
 
     def ==(other)
@@ -137,7 +137,7 @@ module Prism
     end
 
     def self.null
-      new(0, 0)
+      new(nil, 0, 0)
     end
   end
 
@@ -166,6 +166,32 @@ module Prism
     end
   end
 
+  # This represents a magic comment that was encountered during parsing.
+  class MagicComment
+    attr_reader :key_loc, :value_loc
+
+    def initialize(key_loc, value_loc)
+      @key_loc = key_loc
+      @value_loc = value_loc
+    end
+
+    def key
+      key_loc.slice
+    end
+
+    def value
+      value_loc.slice
+    end
+
+    def deconstruct_keys(keys)
+      { key_loc: key_loc, value_loc: value_loc }
+    end
+
+    def inspect
+      "#<Prism::MagicComment @key=#{key.inspect} @value=#{value.inspect}>"
+    end
+  end
+
   # This represents an error that was encountered during parsing.
   class ParseError
     attr_reader :message, :location
@@ -206,18 +232,19 @@ module Prism
   # the AST, any comments that were encounters, and any errors that were
   # encountered.
   class ParseResult
-    attr_reader :value, :comments, :errors, :warnings, :source
+    attr_reader :value, :comments, :magic_comments, :errors, :warnings, :source
 
-    def initialize(value, comments, errors, warnings, source)
+    def initialize(value, comments, magic_comments, errors, warnings, source)
       @value = value
       @comments = comments
+      @magic_comments = magic_comments
       @errors = errors
       @warnings = warnings
       @source = source
     end
 
     def deconstruct_keys(keys)
-      { value: value, comments: comments, errors: errors, warnings: warnings }
+      { value: value, comments: comments, magic_comments: magic_comments, errors: errors, warnings: warnings }
     end
 
     def success?

data/lib/prism/pattern.rb

@@ -10,7 +10,7 @@ module Prism
   #     in ConstantPathNode[ConstantReadNode[name: :Prism], ConstantReadNode[name: :Pattern]]
   #     end
   #
-  # the pattern is the `ConstantPathNode[...]` expression.
+  # the pattern is the <tt>ConstantPathNode[...]</tt> expression.
   #
   # The pattern gets compiled into an object that responds to #call by running
   # the #compile method. This method itself will run back through Prism to

data/lib/prism/serialize.rb

@@ -21,7 +21,7 @@ end
 module Prism
   module Serialize
     MAJOR_VERSION = 0
-    MINOR_VERSION = 13
+    MINOR_VERSION = 15
     PATCH_VERSION = 0
 
     def self.load(input, serialized)
@@ -62,9 +62,10 @@ module Prism
 
       def load_metadata
         comments = load_varint.times.map { Comment.new(Comment::TYPES.fetch(load_varint), load_location) }
+        magic_comments = load_varint.times.map { MagicComment.new(load_location, load_location) }
         errors = load_varint.times.map { ParseError.new(load_embedded_string, load_location) }
         warnings = load_varint.times.map { ParseWarning.new(load_embedded_string, load_location) }
-        [comments, errors, warnings]
+        [comments, magic_comments, errors, warnings]
       end
 
       def load_tokens
@@ -83,14 +84,14 @@ module Prism
       def load_tokens_result
         tokens = load_tokens
         encoding = load_encoding
-        comments, errors, warnings = load_metadata
+        comments, magic_comments, errors, warnings = load_metadata
 
         if encoding != @encoding
           tokens.each { |token,| token.value.force_encoding(encoding) }
         end
 
         raise "Expected to consume all bytes while deserializing" unless @io.eof?
-        Prism::ParseResult.new(tokens, comments, errors, warnings, @source)
+        Prism::ParseResult.new(tokens, comments, magic_comments, errors, warnings, @source)
       end
 
       def load_nodes
@@ -104,17 +105,17 @@ module Prism
         @encoding = load_encoding
         @input = input.force_encoding(@encoding).freeze
 
-        comments, errors, warnings = load_metadata
+        comments, magic_comments, errors, warnings = load_metadata
 
         @constant_pool_offset = io.read(4).unpack1("L")
         @constant_pool = Array.new(load_varint, nil)
 
-        [load_node, comments, errors, warnings]
+        [load_node, comments, magic_comments, errors, warnings]
       end
 
       def load_result
-        node, comments, errors, warnings = load_nodes
-        Prism::ParseResult.new(node, comments, errors, warnings, @source)
+        node, comments, magic_comments, errors, warnings = load_nodes
+        Prism::ParseResult.new(node, comments, magic_comments, errors, warnings, @source)
       end
 
       private
@@ -151,13 +152,14 @@ module Prism
       end
 
       def load_string
-        case io.getbyte
+        type = io.getbyte
+        case type
         when 1
           input.byteslice(load_varint, load_varint).force_encoding(encoding)
         when 2
           load_embedded_string
         else
-          raise
+          raise "Unknown serialized string type: #{type}"
         end
       end
 
@@ -223,7 +225,7 @@ module Prism
         when 9 then
           AssocSplatNode.new(load_optional_node, load_location, location)
         when 10 then
-          BackReferenceReadNode.new(location)
+          BackReferenceReadNode.new(load_required_constant, location)
         when 11 then
           BeginNode.new(load_optional_location, load_optional_node, load_optional_node, load_optional_node, load_optional_node, load_optional_location, location)
         when 12 then
@@ -239,13 +241,13 @@ module Prism
         when 17 then
           BreakNode.new(load_optional_node, load_location, location)
         when 18 then
-          CallAndWriteNode.new(load_optional_node, load_optional_location, load_optional_location, load_optional_location, load_optional_node, load_optional_location, load_varint, load_string, load_string, load_location, load_node, location)
+          CallAndWriteNode.new(load_optional_node, load_optional_location, load_optional_location, load_varint, load_required_constant, load_required_constant, load_location, load_node, location)
         when 19 then
-          CallNode.new(load_optional_node, load_optional_location, load_optional_location, load_optional_location, load_optional_node, load_optional_location, load_optional_node, load_varint, load_string, location)
+          CallNode.new(load_optional_node, load_optional_location, load_optional_location, load_optional_location, load_optional_node, load_optional_location, load_optional_node, load_varint, load_required_constant, location)
         when 20 then
-          CallOperatorWriteNode.new(load_optional_node, load_optional_location, load_optional_location, load_optional_location, load_optional_node, load_optional_location, load_varint, load_string, load_string, load_required_constant, load_location, load_node, location)
+          CallOperatorWriteNode.new(load_optional_node, load_optional_location, load_optional_location, load_varint, load_required_constant, load_required_constant, load_required_constant, load_location, load_node, location)
         when 21 then
-          CallOrWriteNode.new(load_optional_node, load_optional_location, load_optional_location, load_optional_location, load_optional_node, load_optional_location, load_varint, load_string, load_string, load_location, load_node, location)
+          CallOrWriteNode.new(load_optional_node, load_optional_location, load_optional_location, load_varint, load_required_constant, load_required_constant, load_location, load_node, location)
         when 22 then
           CapturePatternNode.new(load_node, load_node, load_location, location)
         when 23 then
@@ -342,150 +344,156 @@ module Prism
         when 68 then
           InNode.new(load_node, load_optional_node, load_location, load_optional_location, location)
         when 69 then
-          InstanceVariableAndWriteNode.new(load_required_constant, load_location, load_location, load_node, location)
+          IndexAndWriteNode.new(load_optional_node, load_optional_location, load_location, load_optional_node, load_location, load_optional_node, load_varint, load_location, load_node, location)
         when 70 then
-          InstanceVariableOperatorWriteNode.new(load_required_constant, load_location, load_location, load_node, load_required_constant, location)
+          IndexOperatorWriteNode.new(load_optional_node, load_optional_location, load_location, load_optional_node, load_location, load_optional_node, load_varint, load_required_constant, load_location, load_node, location)
         when 71 then
-          InstanceVariableOrWriteNode.new(load_required_constant, load_location, load_location, load_node, location)
+          IndexOrWriteNode.new(load_optional_node, load_optional_location, load_location, load_optional_node, load_location, load_optional_node, load_varint, load_location, load_node, location)
         when 72 then
-          InstanceVariableReadNode.new(load_required_constant, location)
+          InstanceVariableAndWriteNode.new(load_required_constant, load_location, load_location, load_node, location)
         when 73 then
-          InstanceVariableTargetNode.new(load_required_constant, location)
+          InstanceVariableOperatorWriteNode.new(load_required_constant, load_location, load_location, load_node, load_required_constant, location)
         when 74 then
-          InstanceVariableWriteNode.new(load_required_constant, load_location, load_node, load_location, location)
+          InstanceVariableOrWriteNode.new(load_required_constant, load_location, load_location, load_node, location)
         when 75 then
-          IntegerNode.new(load_varint, location)
+          InstanceVariableReadNode.new(load_required_constant, location)
         when 76 then
-          InterpolatedMatchLastLineNode.new(load_location, Array.new(load_varint) { load_node }, load_location, load_varint, location)
+          InstanceVariableTargetNode.new(load_required_constant, location)
         when 77 then
-          InterpolatedRegularExpressionNode.new(load_location, Array.new(load_varint) { load_node }, load_location, load_varint, location)
+          InstanceVariableWriteNode.new(load_required_constant, load_location, load_node, load_location, location)
         when 78 then
-          InterpolatedStringNode.new(load_optional_location, Array.new(load_varint) { load_node }, load_optional_location, location)
+          IntegerNode.new(load_varint, location)
         when 79 then
-          InterpolatedSymbolNode.new(load_optional_location, Array.new(load_varint) { load_node }, load_optional_location, location)
+          InterpolatedMatchLastLineNode.new(load_location, Array.new(load_varint) { load_node }, load_location, load_varint, location)
         when 80 then
-          InterpolatedXStringNode.new(load_location, Array.new(load_varint) { load_node }, load_location, location)
+          InterpolatedRegularExpressionNode.new(load_location, Array.new(load_varint) { load_node }, load_location, load_varint, location)
         when 81 then
-          KeywordHashNode.new(Array.new(load_varint) { load_node }, location)
+          InterpolatedStringNode.new(load_optional_location, Array.new(load_varint) { load_node }, load_optional_location, location)
         when 82 then
-          KeywordParameterNode.new(load_required_constant, load_location, load_optional_node, location)
+          InterpolatedSymbolNode.new(load_optional_location, Array.new(load_varint) { load_node }, load_optional_location, location)
         when 83 then
-          KeywordRestParameterNode.new(load_optional_constant, load_optional_location, load_location, location)
+          InterpolatedXStringNode.new(load_location, Array.new(load_varint) { load_node }, load_location, location)
         when 84 then
-          LambdaNode.new(Array.new(load_varint) { load_required_constant }, load_location, load_location, load_location, load_optional_node, load_optional_node, location)
+          KeywordHashNode.new(Array.new(load_varint) { load_node }, location)
         when 85 then
-          LocalVariableAndWriteNode.new(load_location, load_location, load_node, load_required_constant, load_varint, location)
+          KeywordParameterNode.new(load_required_constant, load_location, load_optional_node, location)
         when 86 then
-          LocalVariableOperatorWriteNode.new(load_location, load_location, load_node, load_required_constant, load_required_constant, load_varint, location)
+          KeywordRestParameterNode.new(load_optional_constant, load_optional_location, load_location, location)
         when 87 then
-          LocalVariableOrWriteNode.new(load_location, load_location, load_node, load_required_constant, load_varint, location)
+          LambdaNode.new(Array.new(load_varint) { load_required_constant }, load_location, load_location, load_location, load_optional_node, load_optional_node, location)
         when 88 then
-          LocalVariableReadNode.new(load_required_constant, load_varint, location)
+          LocalVariableAndWriteNode.new(load_location, load_location, load_node, load_required_constant, load_varint, location)
         when 89 then
-          LocalVariableTargetNode.new(load_required_constant, load_varint, location)
+          LocalVariableOperatorWriteNode.new(load_location, load_location, load_node, load_required_constant, load_required_constant, load_varint, location)
         when 90 then
-          LocalVariableWriteNode.new(load_required_constant, load_varint, load_location, load_node, load_location, location)
+          LocalVariableOrWriteNode.new(load_location, load_location, load_node, load_required_constant, load_varint, location)
         when 91 then
-          MatchLastLineNode.new(load_location, load_location, load_location, load_string, load_varint, location)
+          LocalVariableReadNode.new(load_required_constant, load_varint, location)
         when 92 then
-          MatchPredicateNode.new(load_node, load_node, load_location, location)
+          LocalVariableTargetNode.new(load_required_constant, load_varint, location)
         when 93 then
-          MatchRequiredNode.new(load_node, load_node, load_location, location)
+          LocalVariableWriteNode.new(load_required_constant, load_varint, load_location, load_node, load_location, location)
         when 94 then
-          MatchWriteNode.new(load_node, Array.new(load_varint) { load_required_constant }, location)
+          MatchLastLineNode.new(load_location, load_location, load_location, load_string, load_varint, location)
         when 95 then
-          MissingNode.new(location)
+          MatchPredicateNode.new(load_node, load_node, load_location, location)
         when 96 then
-          ModuleNode.new(Array.new(load_varint) { load_required_constant }, load_location, load_node, load_optional_node, load_location, load_required_constant, location)
+          MatchRequiredNode.new(load_node, load_node, load_location, location)
         when 97 then
-          MultiTargetNode.new(Array.new(load_varint) { load_node }, load_optional_location, load_optional_location, location)
+          MatchWriteNode.new(load_node, Array.new(load_varint) { load_required_constant }, location)
         when 98 then
-          MultiWriteNode.new(Array.new(load_varint) { load_node }, load_optional_location, load_optional_location, load_location, load_node, location)
+          MissingNode.new(location)
         when 99 then
-          NextNode.new(load_optional_node, load_location, location)
+          ModuleNode.new(Array.new(load_varint) { load_required_constant }, load_location, load_node, load_optional_node, load_location, load_required_constant, location)
         when 100 then
-          NilNode.new(location)
+          MultiTargetNode.new(Array.new(load_varint) { load_node }, load_optional_location, load_optional_location, location)
         when 101 then
-          NoKeywordsParameterNode.new(load_location, load_location, location)
+          MultiWriteNode.new(Array.new(load_varint) { load_node }, load_optional_location, load_optional_location, load_location, load_node, location)
         when 102 then
-          NumberedReferenceReadNode.new(load_varint, location)
+          NextNode.new(load_optional_node, load_location, location)
         when 103 then
-          OptionalParameterNode.new(load_required_constant, load_location, load_location, load_node, location)
+          NilNode.new(location)
         when 104 then
-          OrNode.new(load_node, load_node, load_location, location)
+          NoKeywordsParameterNode.new(load_location, load_location, location)
         when 105 then
-          ParametersNode.new(Array.new(load_varint) { load_node }, Array.new(load_varint) { load_node }, load_optional_node, Array.new(load_varint) { load_node }, Array.new(load_varint) { load_node }, load_optional_node, load_optional_node, location)
+          NumberedReferenceReadNode.new(load_varint, location)
         when 106 then
-          ParenthesesNode.new(load_optional_node, load_location, load_location, location)
+          OptionalParameterNode.new(load_required_constant, load_location, load_location, load_node, location)
         when 107 then
-          PinnedExpressionNode.new(load_node, load_location, load_location, load_location, location)
+          OrNode.new(load_node, load_node, load_location, location)
         when 108 then
-          PinnedVariableNode.new(load_node, load_location, location)
+          ParametersNode.new(Array.new(load_varint) { load_node }, Array.new(load_varint) { load_node }, load_optional_node, Array.new(load_varint) { load_node }, Array.new(load_varint) { load_node }, load_optional_node, load_optional_node, location)
         when 109 then
-          PostExecutionNode.new(load_optional_node, load_location, load_location, load_location, location)
+          ParenthesesNode.new(load_optional_node, load_location, load_location, location)
         when 110 then
-          PreExecutionNode.new(load_optional_node, load_location, load_location, load_location, location)
+          PinnedExpressionNode.new(load_node, load_location, load_location, load_location, location)
         when 111 then
-          ProgramNode.new(Array.new(load_varint) { load_required_constant }, load_node, location)
+          PinnedVariableNode.new(load_node, load_location, location)
         when 112 then
-          RangeNode.new(load_optional_node, load_optional_node, load_location, load_varint, location)
+          PostExecutionNode.new(load_optional_node, load_location, load_location, load_location, location)
         when 113 then
-          RationalNode.new(load_node, location)
+          PreExecutionNode.new(load_optional_node, load_location, load_location, load_location, location)
         when 114 then
-          RedoNode.new(location)
+          ProgramNode.new(Array.new(load_varint) { load_required_constant }, load_node, location)
         when 115 then
-          RegularExpressionNode.new(load_location, load_location, load_location, load_string, load_varint, location)
+          RangeNode.new(load_optional_node, load_optional_node, load_location, load_varint, location)
         when 116 then
-          RequiredDestructuredParameterNode.new(Array.new(load_varint) { load_node }, load_location, load_location, location)
+          RationalNode.new(load_node, location)
         when 117 then
-          RequiredParameterNode.new(load_required_constant, location)
+          RedoNode.new(location)
         when 118 then
-          RescueModifierNode.new(load_node, load_location, load_node, location)
+          RegularExpressionNode.new(load_location, load_location, load_location, load_string, load_varint, location)
         when 119 then
-          RescueNode.new(load_location, Array.new(load_varint) { load_node }, load_optional_location, load_optional_node, load_optional_node, load_optional_node, location)
+          RequiredDestructuredParameterNode.new(Array.new(load_varint) { load_node }, load_location, load_location, location)
         when 120 then
-          RestParameterNode.new(load_optional_constant, load_optional_location, load_location, location)
+          RequiredParameterNode.new(load_required_constant, location)
         when 121 then
-          RetryNode.new(location)
+          RescueModifierNode.new(load_node, load_location, load_node, location)
         when 122 then
-          ReturnNode.new(load_location, load_optional_node, location)
+          RescueNode.new(load_location, Array.new(load_varint) { load_node }, load_optional_location, load_optional_node, load_optional_node, load_optional_node, location)
         when 123 then
-          SelfNode.new(location)
+          RestParameterNode.new(load_optional_constant, load_optional_location, load_location, location)
         when 124 then
-          SingletonClassNode.new(Array.new(load_varint) { load_required_constant }, load_location, load_location, load_node, load_optional_node, load_location, location)
+          RetryNode.new(location)
         when 125 then
-          SourceEncodingNode.new(location)
+          ReturnNode.new(load_location, load_optional_node, location)
         when 126 then
-          SourceFileNode.new(load_string, location)
+          SelfNode.new(location)
         when 127 then
-          SourceLineNode.new(location)
+          SingletonClassNode.new(Array.new(load_varint) { load_required_constant }, load_location, load_location, load_node, load_optional_node, load_location, location)
         when 128 then
-          SplatNode.new(load_location, load_optional_node, location)
+          SourceEncodingNode.new(location)
         when 129 then
-          StatementsNode.new(Array.new(load_varint) { load_node }, location)
+          SourceFileNode.new(load_string, location)
         when 130 then
-          StringConcatNode.new(load_node, load_node, location)
+          SourceLineNode.new(location)
         when 131 then
-          StringNode.new(load_varint, load_optional_location, load_location, load_optional_location, load_string, location)
+          SplatNode.new(load_location, load_optional_node, location)
         when 132 then
-          SuperNode.new(load_location, load_optional_location, load_optional_node, load_optional_location, load_optional_node, location)
+          StatementsNode.new(Array.new(load_varint) { load_node }, location)
         when 133 then
-          SymbolNode.new(load_optional_location, load_optional_location, load_optional_location, load_string, location)
+          StringConcatNode.new(load_node, load_node, location)
         when 134 then
-          TrueNode.new(location)
+          StringNode.new(load_varint, load_optional_location, load_location, load_optional_location, load_string, location)
         when 135 then
-          UndefNode.new(Array.new(load_varint) { load_node }, load_location, location)
+          SuperNode.new(load_location, load_optional_location, load_optional_node, load_optional_location, load_optional_node, location)
         when 136 then
-          UnlessNode.new(load_location, load_node, load_optional_node, load_optional_node, load_optional_location, location)
+          SymbolNode.new(load_optional_location, load_optional_location, load_optional_location, load_string, location)
         when 137 then
-          UntilNode.new(load_location, load_optional_location, load_node, load_optional_node, load_varint, location)
+          TrueNode.new(location)
         when 138 then
-          WhenNode.new(load_location, Array.new(load_varint) { load_node }, load_optional_node, location)
+          UndefNode.new(Array.new(load_varint) { load_node }, load_location, location)
         when 139 then
-          WhileNode.new(load_location, load_optional_location, load_node, load_optional_node, load_varint, location)
+          UnlessNode.new(load_location, load_node, load_optional_node, load_optional_node, load_optional_location, location)
         when 140 then
-          XStringNode.new(load_location, load_location, load_location, load_string, location)
+          UntilNode.new(load_location, load_optional_location, load_node, load_optional_node, load_varint, location)
         when 141 then
+          WhenNode.new(load_location, Array.new(load_varint) { load_node }, load_optional_node, location)
+        when 142 then
+          WhileNode.new(load_location, load_optional_location, load_node, load_optional_node, load_varint, location)
+        when 143 then
+          XStringNode.new(load_location, load_location, load_location, load_string, location)
+        when 144 then
           YieldNode.new(load_location, load_optional_location, load_optional_node, load_optional_location, location)
         end
       end

data/lib/prism/visitor.rb

@@ -248,6 +248,15 @@ module Prism
     # Visit a InNode node
     alias visit_in_node visit_child_nodes
 
+    # Visit a IndexAndWriteNode node
+    alias visit_index_and_write_node visit_child_nodes
+
+    # Visit a IndexOperatorWriteNode node
+    alias visit_index_operator_write_node visit_child_nodes
+
+    # Visit a IndexOrWriteNode node
+    alias visit_index_or_write_node visit_child_nodes
+
     # Visit a InstanceVariableAndWriteNode node
     alias visit_instance_variable_and_write_node visit_child_nodes
 

data/prism.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |spec|
   spec.name = "prism"
-  spec.version = "0.13.0"
+  spec.version = "0.15.0"
   spec.authors = ["Shopify"]
   spec.email = ["ruby@shopify.com"]
 
@@ -29,6 +29,7 @@ Gem::Specification.new do |spec|
     "docs/fuzzing.md",
     "docs/heredocs.md",
     "docs/mapping.md",
+    "docs/prism.png",
     "docs/ripper.md",
     "docs/ruby_api.md",
     "docs/serialization.md",
@@ -46,7 +47,6 @@ Gem::Specification.new do |spec|
     "include/prism/pack.h",
     "include/prism/parser.h",
     "include/prism/regexp.h",
-    "include/prism/unescape.h",
     "include/prism/util/pm_buffer.h",
     "include/prism/util/pm_char.h",
     "include/prism/util/pm_constant_pool.h",
@@ -92,7 +92,6 @@ Gem::Specification.new do |spec|
     "src/regexp.c",
     "src/serialize.c",
     "src/token_type.c",
-    "src/unescape.c",
     "src/util/pm_buffer.c",
     "src/util/pm_char.c",
     "src/util/pm_constant_pool.c",

data/src/diagnostic.c

@@ -192,6 +192,7 @@ static const char* const diagnostic_messages[PM_DIAGNOSTIC_ID_LEN] = {
     [PM_ERR_NUMBERED_PARAMETER_NOT_ALLOWED]     = "Numbered parameters are not allowed alongside explicit parameters",
     [PM_ERR_NUMBERED_PARAMETER_OUTER_SCOPE]     = "Numbered parameter is already used in outer scope",
     [PM_ERR_OPERATOR_MULTI_ASSIGN]              = "Unexpected operator for a multiple assignment",
+    [PM_ERR_OPERATOR_WRITE_ARGUMENTS]           = "Unexpected operator after a call with arguments",
     [PM_ERR_OPERATOR_WRITE_BLOCK]               = "Unexpected operator after a call with a block",
     [PM_ERR_PARAMETER_ASSOC_SPLAT_MULTI]        = "Unexpected multiple `**` splat parameters",
     [PM_ERR_PARAMETER_BLOCK_MULTI]              = "Multiple block parameters; only one block is allowed",
@@ -265,7 +266,7 @@ pm_diagnostic_message(pm_diagnostic_id_t diag_id) {
 // Append an error to the given list of diagnostic.
 bool
 pm_diagnostic_list_append(pm_list_t *list, const uint8_t *start, const uint8_t *end, pm_diagnostic_id_t diag_id) {
-    pm_diagnostic_t *diagnostic = (pm_diagnostic_t *) malloc(sizeof(pm_diagnostic_t));
+    pm_diagnostic_t *diagnostic = (pm_diagnostic_t *) calloc(sizeof(pm_diagnostic_t), 1);
     if (diagnostic == NULL) return false;
 
     *diagnostic = (pm_diagnostic_t) { .start = start, .end = end, .message = pm_diagnostic_message(diag_id) };