Bug #9452: Refining methods that should be private

[Jan Lelis]

Are refinements also meant to add private methods? This is what I tried:

module R
  refine Object do
    def m
      puts "Success!"
    end

    private(:m)
  end
end

using R

m # success
42.m # success (= not private)

However, I can get near the desired functionality by defining a private method first:

class Object
  private
  def m
  end
end

module R
  refine Object do
    def m
      puts "Success!"
    end
  end
end

using R
m # success
42.m # no success (= it is private)

It calls the right code. But requires global core ext.

[Shugo Maeda]

• Assignee set to Shugo Maeda

[Shugo Maeda]

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

Applied in changeset r44931.

---

• vminsnhelper.c (vmcall_method): should check ci->me->flag of
  a refining method in case the method is private.
  [Bug #9452]

• vmmethod.c (makemethodentryrefined): set me->flag of a refined
  method entry to NOEXPUBLIC in case the original method is private
  and it is refined as a public method. The original flag is stored
  in me->def->body.origme, so it's OK to make a refined method
  entry public. [Bug #9452]

• test/ruby/test_refinement.rb: related tests.

[Tomoyuki Chikanaga]

• 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

[Tomoyuki Chikanaga]

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

r44931 was backported to ruby20_0 at r44967.

[Yui NARUSE]

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

r45107.