Skip to content

java support cleanup and improvements#2624

Closed
kares wants to merge 45 commits intojruby:jruby-1_7from
kares:test-java-support-improvements11
Closed

java support cleanup and improvements#2624
kares wants to merge 45 commits intojruby:jruby-1_7from
kares:test-java-support-improvements11

Conversation

@kares
Copy link
Member

@kares kares commented Feb 26, 2015

details at commits .. I'm happy to re-iterate leave some out as needed

kares added 30 commits February 6, 2015 09:34
…ch are internal

also hidden JavaClass' constructor as if it's ever instantiated outside JavaSupport's class-value cache it will start to break things badly
…son typos

... `obj == NEVER` when really unwrapped value was meant to be compared
…ltiple threads)

avoid warnings such as rb:42 warning: already initialized constant JavaUtilConcurrent

probably related to jruby#2014
…ethod

... has a desired side effect of improving "initial" performance
due not re-raising ClassNotFoundExceptions within the Ruby runtime

they also will no longer be reported by the runtime with log.exeptions=true
- only pass context to (internal) getTopLevelProxyOrPackage method
- setupJavaClass does not need a java_class argument (it's not used)
- only intern string on addMethod (although it's likely redundat there as well)
…me up the stack

... it very likely never-ever trimmed anything
also (package) "internalized" JavaClass.getRubyArray as toRubyArray
kares added 15 commits February 6, 2015 09:34
…& also :

- invoke implementation now avoids unnecessary System.arraycopy call
- pre-maturily optimized some from "excessive" getRuntime() invocations
@headius
Copy link
Member

headius commented Feb 26, 2015

This has been merged into test-refactor-ji and I'm continuing some refactoring there. Will merge that branch to jruby-1_7 shortly.

@headius headius closed this Feb 26, 2015
@headius headius added this to the JRuby 1.7.20 milestone Feb 26, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants