Skip to content
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

Actually use Java7ClassValue when using Java 7+. #1616

Merged
merged 1 commit into from Apr 16, 2014

Conversation

@cky
Copy link
Contributor

@cky cky commented Apr 9, 2014

When f5850a5 was created, the org.jruby.java.proxies.ClassValueProxyCache class was renamed as org.jruby.util.collections.Java7ClassValue. But that class was only referenced via reflection, and so IDE-based refactoring did not catch it, and Java7ClassValue ended up being totally unused.

headius added a commit that referenced this issue Apr 16, 2014
Actually use Java7ClassValue when using Java 7+.
@headius headius merged commit 5ec626b into jruby:master Apr 16, 2014
1 check failed
@headius
Copy link
Member

@headius headius commented Apr 16, 2014

Thanks!

@headius headius added this to the JRuby 9000 milestone Apr 16, 2014
@atambo
Copy link
Member

@atambo atambo commented Apr 16, 2014

This should be included into jruby-1_7 branch as well, right?

@cky cky deleted the actually-use-Java7ClassValue branch Apr 16, 2014
@cky
Copy link
Contributor Author

@cky cky commented Apr 16, 2014

@atambo Yeah, I didn't know which branch I should use as a base when doing pull requests. I know that @enebo does cherry-pick commits into jruby-1_7, but I guess in future, I should just use that branch as the base for pull requests.

@enebo
Copy link
Member

@enebo enebo commented Apr 16, 2014

@cky jruby-1_7 is preferred for 1.8, 1.9+ compat fixes and other non-risky backwards compatible changes. If there is any risk or if public signatures change then master is a better place. It sounds like this could have went to jruby-1_7 as well. Don't sweat it though we are used to moving stuff around across branches...

@atambo
Copy link
Member

@atambo atambo commented Apr 19, 2014

I cherry-picked it to jruby-1_7 here: f1d3620

@atambo atambo added this to the JRuby 1.7.13 milestone Apr 19, 2014
@atambo atambo removed this from the JRuby 9000 milestone Apr 19, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants