Project

General

Profile

Actions

Bug #9308

closed

def's return value causes errors

Added by phortx (Benjamin Kammerl) over 10 years ago. Updated about 10 years ago.

Status:
Closed
Assignee:
-
Target version:
ruby -v:
ruby 2.1.0dev (2013-11-23 trunk 43807) [x86_64-linux]
[ruby-core:59342]

Description

It seems 2.1 hase some trouble with the "do" keyword in combination with the private/public/protected keywords and the def's return value:

Works: https://gist.github.com/phortx/8146341
Works Not: https://gist.github.com/phortx/8146345

$ ruby --version
ruby 2.1.0dev (2013-11-23 trunk 43807) [x86_64-linux]

$ ruby works.rb
$ ruby works_not.rb
test.rb:5: syntax error, unexpected keyword_do_block, expecting keyword_end
test.rb:12: syntax error, unexpected keyword_end, expecting end-of-input


Related issues 2 (0 open2 closed)

Has duplicate Ruby master - Bug #9376: Passing do/end block inside decorated method fails, while passing {} block works.Closed01/07/2014Actions
Has duplicate Ruby master - Bug #9416: 'private def' results in syntax errors in the methodClosed01/15/2014Actions
Actions #1

Updated by nobu (Nobuyoshi Nakada) about 10 years ago

  • Status changed from Open to Closed
  • % Done changed from 0 to 100

This issue was solved with changeset r44449.
Benjamin, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.


parse.y: save cmdarg_stack in local scope

  • parse.y (local_push_gen, local_pop_gen): save cmdarg_stack to
    isolate command argument state from outer scope.
    [ruby-core:59342] [Bug #9308]

Updated by nobu (Nobuyoshi Nakada) about 10 years ago

  • Priority changed from 5 to Normal
  • Backport changed from 1.9.3: UNKNOWN, 2.0.0: UNKNOWN, 2.1: UNKNOWN to 1.9.3: DONTNEED, 2.0.0: REQUIRED, 2.1: REQUIRED

I set backport to 2.0 to REQUIRED, because this is an old issue but noone has never been hit.
For source-level compatibility, it's better that 2.0 also accepts this syntax.

1.9? It has died almost.

Updated by usa (Usaku NAKAMURA) about 10 years ago

  • Backport changed from 1.9.3: DONTNEED, 2.0.0: REQUIRED, 2.1: REQUIRED to 1.9.3: REQUIRED, 2.0.0: REQUIRED, 2.1: REQUIRED

1.9.3 is not dead... yet.

Actions #4

Updated by nobu (Nobuyoshi Nakada) about 10 years ago

  • Has duplicate Bug #9416: 'private def' results in syntax errors in the method added

Updated by usa (Usaku NAKAMURA) about 10 years ago

  • Backport changed from 1.9.3: REQUIRED, 2.0.0: REQUIRED, 2.1: REQUIRED to 1.9.3: DONE, 2.0.0: REQUIRED, 2.1: REQUIRED

backported to ruby_1_9_3 at r44737.

Updated by nagachika (Tomoyuki Chikanaga) about 10 years ago

  • Backport changed from 1.9.3: DONE, 2.0.0: REQUIRED, 2.1: REQUIRED to 1.9.3: DONE, 2.0.0: DONE, 2.1: REQUIRED

r44449 was backported to ruby_2_0_0 branch at 44898.

Updated by naruse (Yui NARUSE) about 10 years ago

  • Backport changed from 1.9.3: DONE, 2.0.0: DONE, 2.1: REQUIRED to 1.9.3: DONE, 2.0.0: DONE, 2.1: DONE

r45086.

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0Like0Like0Like0