Issue #8839 has been updated by headius (Charles Nutter).


jballanc (Joshua Ballanco) wrote:
> =begin
> Just to throw my 2ยข in...
> 
> I think the main benefit to returning a symbol from (({def})) is that it enables the use of method decorators. Similarly, I would be in favor of returning the class defined from (({class})) so that we could also build class decorators. A trivial example:
> 
>     def enumerable(klass)
>       klass.send(:include, Enumerable)
>       klass
>     end
>     
>     enumerable
>     class Foo
>       #...
>     end
> 
> =end

=begin
Clever! Though I don't think that would parse the way you want. This would work though, obviously:

  enumerable class Foo ...

=end
----------------------------------------
Feature #8839: Class and module should return the class or module that was opened
https://bugs.ruby-lang.org/issues/8839#change-42807

Author: headius (Charles Nutter)
Status: Assigned
Priority: Normal
Assignee: matz (Yukihiro Matsumoto)
Category: 
Target version: next minor


With the change for https://bugs.ruby-lang.org/issues/3753, "def" forms now return the symbolic name of the method defined. Because class and module bodies just return the last expression in their bodies, this means they will now usually end up returning a symbol for the last method defined. This does not seem useful or correct.

I think class and module should return a reference to the class or module just opened. This would make the return value useful and consistent.


-- 
http://bugs.ruby-lang.org/