[ruby/prism] Only accept noblock in Ruby 4.1+

ruby/prism@fb88c72dce

Author: kddnewton

prism/prism.c

@@ -13949,7 +13949,7 @@ parse_parameters(
                 pm_token_t operator = parser->previous;
                 pm_node_t *param;
 
-                if (accept1(parser, PM_TOKEN_KEYWORD_NIL)) {
+                if (parser->version >= PM_OPTIONS_VERSION_CRUBY_4_1 && accept1(parser, PM_TOKEN_KEYWORD_NIL)) {
                     param = (pm_node_t *) pm_no_block_parameter_node_create(parser, &operator, &parser->previous);
                 } else {
                     pm_token_t name = {0};

test/prism/errors/3.3-4.0/noblock.txt

@@ -0,0 +1,6 @@
+def foo(&nil)
+         ^~~ unexpected 'nil'; expected a `)` to close the parameters
+            ^ unexpected ')', expecting end-of-input
+            ^ unexpected ')', ignoring it
+end
+

test/prism/fixtures/4.1/noblock.txt

@@ -0,0 +1,4 @@
+def foo(&nil)
+end
+
+-> (&nil) {}

test/prism/locals_test.rb

@@ -30,7 +30,7 @@ class LocalsTest < TestCase
       "command_method_call_2.txt",
 
       # https://bugs.ruby-lang.org/issues/21669
-      "4.1/void_value.txt"
+      "4.1/void_value.txt",
     ]
 
     Fixture.each_for_current_ruby(except: except) do |fixture|
@@ -207,7 +207,7 @@ def prism_locals(source)
               end
             end
 
-            if params.block
+            if params.block.is_a?(BlockParameterNode)
               sorted << (params.block.name || :&)
             end
 

test/prism/ruby/ripper_test.rb

@@ -40,6 +40,9 @@ class RipperTest < TestCase
     # https://bugs.ruby-lang.org/issues/21669
     incorrect << "4.1/void_value.txt"
 
+    # https://bugs.ruby-lang.org/issues/19979
+    incorrect << "4.1/noblock.txt"
+
     # Skip these tests that we haven't implemented yet.
     omitted_sexp_raw = [
       "bom_leading_space.txt",