Project

General

Profile

Bug #9308

def's return value causes errors

Added by Benjamin Kammerl over 2 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
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

Duplicated by Ruby trunk - Bug #9376: Passing do/end block inside decorated method fails, while passing {} block works. Closed 01/07/2014
Duplicated by Ruby trunk - Bug #9416: 'private def' results in syntax errors in the method Closed 01/15/2014

Associated revisions

Revision 44449
Added by Nobuyoshi Nakada over 2 years ago

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. [Bug #9308]

Revision 44449
Added by Nobuyoshi Nakada over 2 years ago

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. [Bug #9308]

Revision 44449
Added by Nobuyoshi Nakada over 2 years ago

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. [Bug #9308]

History

#1 Updated by Nobuyoshi Nakada over 2 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. [Bug #9308]

#2 [ruby-core:59614] Updated by Nobuyoshi Nakada over 2 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.

#3 [ruby-core:59621] Updated by Usaku NAKAMURA over 2 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.

#4 Updated by Nobuyoshi Nakada over 2 years ago

  • Duplicated by Bug #9416: 'private def' results in syntax errors in the method added

#5 [ruby-core:60191] Updated by Usaku NAKAMURA over 2 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.

#6 [ruby-core:60655] Updated by Tomoyuki Chikanaga about 2 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.

#7 [ruby-core:60951] Updated by Yui NARUSE about 2 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.

Also available in: Atom PDF