Bug #21983: RubyVM::AST no way to find &nil

• Status: Closed
• Assignee: -
• Target version: -
• Backport: 3.2: DONTNEED, 3.3: DONTNEED, 3.4: DONTNEED, 4.0: DONTNEED

[kddnewton (Kevin Newton)]

$ ./ruby -e 'pp RubyVM::AbstractSyntaxTree.parse("def foo(&nil); end")'
(SCOPE@1:0-1:18
 tbl: []
 args: nil
 body: (DEFN@1:0-1:18 mid: :foo body: (SCOPE@1:0-1:18 tbl: [] args: (ARGS@1:8-1:12 pre_num: 0 pre_init: nil opt: nil first_post: nil post_num: 0 post_init: nil rest: nil kw: nil kwrest: nil block: nil) body: nil)))
$ ./ruby -e 'pp RubyVM::AbstractSyntaxTree.parse("def foo(); end")' 
(SCOPE@1:0-1:14
 tbl: []
 args: nil
 body: (DEFN@1:0-1:14 mid: :foo body: (SCOPE@1:0-1:14 tbl: [] args: (ARGS@1:8-1:8 pre_num: 0 pre_init: nil opt: nil first_post: nil post_num: 0 post_init: nil rest: nil kw: nil kwrest: nil block: nil) body: nil)))
$

The bodies are exactly the same here.

[ufuk (Ufuk Kayserilioglu)]

I am guessing that the &nil case should read block: false by comparing it to the **nil case:

❯ ruby -e 'pp RubyVM::AbstractSyntaxTree.parse("def foo(**nil); end")'
(SCOPE@1:0-1:19
 tbl: []
 args: nil
 body:
   (DEFN@1:0-1:19
    mid: :foo
    body:
      (SCOPE@1:0-1:19
       tbl: []
       args:
         (ARGS@1:8-1:13
          pre_num: 0
          pre_init: nil
          opt: nil
          first_post: nil
          post_num: 0
          post_init: nil
          rest: nil
          kw: false
          kwrest: false
          block: nil)
       body: nil)))

[nobu (Nobuyoshi Nakada)]

• Status changed from Open to Closed

Applied in changeset git|13926009ba829f42037719b9a163a41a8d67abed.

---

[Bug #21983] Make &nil distinguishable from nonexistent block arg

[nobu (Nobuyoshi Nakada)]

• Backport changed from 3.2: UNKNOWN, 3.3: UNKNOWN, 3.4: UNKNOWN, 4.0: UNKNOWN to 3.2: DONTNEED, 3.3: DONTNEED, 3.4: DONTNEED, 4.0: DONTNEED