Skip to content

[#5288] Overwrite parent class with module wrapper#5487

Merged
kares merged 1 commit intojruby:masterfrom
alexis779:issue_5288
Dec 1, 2018
Merged

[#5288] Overwrite parent class with module wrapper#5487
kares merged 1 commit intojruby:masterfrom
alexis779:issue_5288

Conversation

@alexis779
Copy link
Contributor

Fixes Module#refine regression when using Module#include to inject functions.

Compare latest refinement commit with CRuby rb_using_refinement.

RubyModule#setSuperClass call was dropped. We add it back again.

@kares
Copy link
Member

kares commented Dec 1, 2018

nice work @alexis779, thanks!

@kares kares added this to the JRuby 9.2.5.0 milestone Dec 1, 2018
@kares kares merged commit 1bf79da into jruby:master Dec 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants