New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
public org.jruby.Runtime.Profile.Builtin::MethodData #3383
public org.jruby.Runtime.Profile.Builtin::MethodData #3383
Conversation
this makes building custom tooling easier http://pastebin.com/XhNYdMtD
"InvocationSet is not public in org.jruby.runtime.profile.builtin; cannot be accessed from outside package"
Thanks to changing visiblity I can compile my modified copies of all default ProfilePrinters. |
e801fed
to
9d04c55
Compare
originally those package-protected classes were hidden on purpose as they were not polished. |
b2cbfa1
to
5693183
Compare
@kares was this closed by mistake? |
@headius seems like it might have been ... re-opening (@BanzaiMan feel free to close with feedback) |
From my perspective I don't need this right now. |
these are still valid (even in 9K's context), although some of the stuff such as making fields public might be seen as internals and thus subject to change. believe there's no reason not to merge. |
this makes building custom tooling easier
http://pastebin.com/XhNYdMtD