| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| | | |
| | | |
| | | |
| | | | |
Also do it anonomously so as not to raise the alarms and upset the powers to be.
|
| | | |
| | | |
| | | |
| | | | |
Some whitespace got shunted around a bit too.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The main advantage is that we can do the privileged setAccessible during creation and
then freely invoke via the authorized method handle.
As per the javadocs:
Access checks are applied in the factory methods of Lookup, when a method handle is created.
his is a key difference from the Core Reflection API, since java.lang.reflect.Method.invoke
performs access checking against every caller, on every call.
The performance boost is just a bonus.
|
| |/ /
| | |
| | |
| | |
| | |
| | | |
Oracle (or maybe it was still Sun) warns that we should prevent from exposing access we gain to
privileged functionality like the unsafe etc. Before Oracle decides to restrict us even more lets
make an effort at least.
|
|\ \ \
| | | |
| | | | |
Unit Test: CtField.make referencing 'this'.
|
| |/ /
| | |
| | |
| | |
| | | |
As per question raised in #161 passing 'this' reference for CtField value new object instance.
Also added assert for testing referncing 'this' for newly added field.
|
|\ \ \
| | | |
| | | | |
Fix for #152 inner class modifiers.
|
| |/ /
| | |
| | |
| | |
| | | |
Applied fix for #152 "Wrong modifiers returned for nested classes" as suggested by @pietrobraione.
Includes unit tests for all access modifiers and static.
|
|/ / |
|
|\ \
| |/
|/| |
Source walk/Spring clean/Parameterize/Enhance/Squash warnings et.al.
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| | |
With the unit tests warnings were squashed by suppressing them with SuppressWarnings annotations, instead of
reparing the issues as was done with the source. Reasons being...
* The tests might be relying on the types as they were defined.
* If the tests were written to test against the correct forms they would've/should've stipulated those.
* Even unused members might be important to tests so we attempt to change as little as possible.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The following were applied during multiple itterations through the source.
* Parameterize raw types.
* Mark unused members.
* Annotate override and deprecated methods.
* Convert loops to enhance for loop.
* Remove redundant else statements.
* Widening collection type references.
* Optimize code for improved readability.
* Squash compiler warnings.
* Identify smells.
|
| |
| |
| |
| | |
Changed to org.junit.Assert.
|
| |
| |
| |
| | |
Changed to org.junit.Assert.
|
| |
| |
| |
| | |
To honour the Object.equals contract.
|
| |
| |
| |
| | |
To honour the Object.equals contract.
|
| |
| |
| |
| | |
To honour the Object.equals contract.
|
| |
| |
| |
| | |
To honour the Object.equals contract.
|
| |
| |
| |
| | |
Changed to org.junit.Assert.
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
There were minimal changes:
Some tests had their own runners these were discontinued as IDEs/maven run the tests anyway.
Removed the main methods for these.
2 tests were dependent on the running order so added the test sorter.
All tests succeed on junit 4 with legacy support.
|
| |
| |
| |
| | |
In order to squash the mvn build warnings.
|
| |
| |
| |
| | |
In order to squash the mvn build warnings.
|
|/
|
|
| |
Move the test output folder 'runtests' to target or build directory so that mvn clean can clean that.
|
|
|
|
| |
ClassClassPath when the JVM is earlier than 9.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
be attached after the JVM starts
|
| |
|
| |
|
| |
|
|
|
|
| |
Javassist uses java.lang.invoke.MethodHandles.Lookup to load a class.
|
|\
| |
| | |
A fix for older JDK8 versions: https://bugs.openjdk.java.net/browse/JDK-8041920
|
| |
| |
| |
| | |
A fix for older JDK8 versions: https://bugs.openjdk.java.net/browse/JDK-8041920
This caused random failures a little time after startup. Reported now so that when we upgrade javassist for JDK9 then we would not have to fork this class.
|
|/
|
|
| |
jigsaw
|
|
|
|
|
|
|
|
| |
Due to the specification change of jigsaw, ClassLoader#defineClass()
cannot be invoked through reflection API. As a workaround,
--add-opens java.base/java.lang=ALL-UNNAMED
must be given to the JVM. The ant target "runtest9" adds this jvm
argument.
|
|
|
|
|
|
| |
See #ClassFilesAsResources in JSR 376. We don't need to hack the module system to read a class file in java.lang.
http://openjdk.java.net/projects/jigsaw/spec/issues/
|
|
|
|
| |
create a proxy object due to access controls by jigsaw.
|
| |
|
| |
|
|\ |
|
| |\ |
|