Fix hash emitter on kwrestargs

mbj · Oct 8, 2023 · edafcff · edafcff
1 parent c996bba
commit edafcff
Show file tree

Hide file tree

Showing 6 changed files with 14 additions and 3 deletions.
diff --git a/Changelog.md b/Changelog.md
@@ -1,3 +1,9 @@
+# v0.6.8 2023-10-08
+
+[#348](https://github.com/mbj/unparser/pull/348)
+
+* Fix crash on kwrestarg hash member
+
 # v0.6.8 2023-06-14
 
 [#347](https://github.com/mbj/unparser/pull/347)

diff --git a/Gemfile.lock b/Gemfile.lock
@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    unparser (0.6.8)
+    unparser (0.6.9)
       diff-lcs (~> 1.3)
       parser (>= 3.2.0)
 

diff --git a/config/mutant.yml b/config/mutant.yml
@@ -25,6 +25,7 @@ matcher:
   - 'Unparser::Emitter::Class#local_variable_scope'
   - 'Unparser::Emitter::Def#local_variable_scope'
   - 'Unparser::Emitter::FindPattern#dispatch' # 3.0+ specific
+  - 'Unparser::Emitter::Hash#emit_heredoc_reminder_member' # 3.2+ specific
   - 'Unparser::Emitter::HashPattern#write_symbol_body'
   - 'Unparser::Emitter::LocalVariableRoot*'
   - 'Unparser::Emitter::LocalVariableRoot.included'

diff --git a/lib/unparser/emitter/hash.rb b/lib/unparser/emitter/hash.rb
@@ -25,7 +25,7 @@ def dispatch
       end
 
       def emit_heredoc_reminder_member(node)
-        emitter(node.children.last).emit_heredoc_reminders
+        emitter(node.children.last).emit_heredoc_reminders if n_pair?(node)
       end
 
       def emit_hash_body

diff --git a/test/corpus/literal/since/32.rb b/test/corpus/literal/since/32.rb
@@ -5,3 +5,7 @@ def foo(argument, **)
 def foo(argument, *)
   bar(argument, *)
 end
+
+def foo(**)
+  { default: 1, ** }
+end
diff --git a/unparser.gemspec b/unparser.gemspec
@@ -1,6 +1,6 @@
 Gem::Specification.new do |gem|
   gem.name        = 'unparser'
-  gem.version     = '0.6.8'
+  gem.version     = '0.6.9'
 
   gem.authors     = ['Markus Schirp']
   gem.email       = 'mbj@schirp-dso.com'