Feature #13109: `using` in refinements is required to be physically placed before the refined method call

• Status: Rejected
• Assignee: shugo (Shugo Maeda)
• Target version: -

[matsuda (Akira Matsuda)]

When using refinements in one file, the using call needs to be physically placed before the refined method call.

For example, this works:

using Module.new {
  refine Object do
    def foo() p 'hello'; end
  end
}

class Object
  def bar() foo; end
end

Object.new.bar

but this doesn't work:

class Object
  def bar() foo; end
end

using Module.new {
  refine Object do
    def foo() p 'hello'; end
  end
}

Object.new.bar

#=> doesnot_work.rb:2:in `bar': undefined local variable or method `foo' for #<Object:0x007f8f2a0251c8> (NameError)

I know that current refinements has a unique scope called "file scope", but I don't think this is expected behavior.
At least I expect it to work so far as using is evaluated in runtime, no matter where in the file it is.

I confirmed that this reproduces in all stable versions of 2.0, 2.1, 2.2, 2.3, 2.4, and today's trunk (2.5).

[matz (Yukihiro Matsumoto)]

• Tracker changed from Bug to Feature

I consider it's an intended behavior, but there's a room for improvement.
So I made this issue as a feature request.

Matz.

[matz (Yukihiro Matsumoto)]

• Status changed from Open to Rejected

There may be an idea to use refinement without explicit using. But that's out of the scope of this issue.

Matz.

[razum2um (Vlad Bokov)]

matz (Yukihiro Matsumoto) wrote:

There may be an idea to use refinement without explicit using. But that's out of the scope of this issue.

Matz.

Very interesting, @matz (Yukihiro Matsumoto), please, could you refer to the issue of non-explicit using?

[duerst (Martin Dürst)]

razum2um (Vlad Bokov) wrote:

matz (Yukihiro Matsumoto) wrote:

There may be an idea to use refinement without explicit using. But that's out of the scope of this issue.

Matz.

Very interesting, @matz (Yukihiro Matsumoto), please, could you refer to the issue of non-explicit using?

I don't think Matz meant a specific already existing issue. I think it should be proposed as a new, separate issue.

[kernigh (George Koehler)]

Feature #13805 proposed to activate refinements across multiple files without an explicit using in every file.

[duerst (Martin Dürst)]

• Related to Feature #13805: Make refinement scoping to be like that of constants added