Merge pull request #3145 from andrykonchin/ak/fix-handling-of-l-cli-o…

…ption Set contains_keywords flag for implicit gets($/, chomp: true) method to handle -l CLI option
ruby · Oct 7, 2024 · c38df36 · c38df36
2 parents d4a93ad + 717e41c
commit c38df36
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/src/prism.c b/src/prism.c
@@ -21907,6 +21907,7 @@ wrap_statements(pm_parser_t *parser, pm_statements_node_t *statements) {
             ));
 
             pm_arguments_node_arguments_append(arguments, (pm_node_t *) keywords);
+            pm_node_flag_set((pm_node_t *) arguments, PM_ARGUMENTS_NODE_FLAGS_CONTAINS_KEYWORDS);
         }
 
         pm_statements_node_t *wrapped_statements = pm_statements_node_create(parser);

diff --git a/test/prism/api/command_line_test.rb b/test/prism/api/command_line_test.rb
@@ -52,6 +52,9 @@ def test_command_line_l
       assert_kind_of CallNode, predicate
       assert_equal :gets, predicate.name
 
+      arguments = predicate.arguments
+      assert arguments.contains_keywords?
+
       arguments = predicate.arguments.arguments
       assert_equal 2, arguments.length
       assert_equal :$/, arguments.first.name