Add support for Java 11 NestHost and NestMembers attributes

fixes issue #209 (NPE in Exception)

Merge pull request #207 from andresluuk/patch-2

Custom URL stream handler in ByteArrayClassPath

Custom URL stream handler in ByteArrayClassPath

Otherwise a normal streamhandler lookup is made and if javassist is used early on server startup protses this can lead to a deadlock on rare cases on WebSpheare.
For example. A new StreamHandlerFactory is registered but not jet fully initialized. Now javassist is used at the same time but form another thread and it will be the first one to invoke the new StreamHandlerFactory and now it can deadlock with the other thread:

Snipets from my deadlock:
3XMTHREADINFO      "main" J9VMThread:0x000000000175EC00, omrthread_t:0x00007F4120008100, java/lang/Thread:0x00000000624290B8, state:B, prio=5
3XMTHREADBLOCK     Blocked on: org/osgi/util/tracker/ServiceTracker$Tracked@0x00000000627F0A90 Owned by: "other-thread" (J9VMThread:0x0000000001B26300, java/lang/Thread:0x00000000629F1458)
3XMHEAPALLOC             Heap bytes allocated since last GC cycle=0 (0x0)
3XMTHREADINFO3           Java callstack:
4XESTACKTRACE                at org/osgi/util/tracker/ServiceTracker.getServiceReferences(ServiceTracker.java:525)
4XESTACKTRACE                at org/eclipse/osgi/framework/internal/protocol/StreamHandlerFactory.createInternalURLStreamHandler(StreamHandlerFactory.java:162)
4XESTACKTRACE                at org/eclipse/osgi/framework/internal/protocol/StreamHandlerFactory.createURLStreamHandler(StreamHandlerFactory.java:118)
4XESTACKTRACE                at java/net/URL.getURLStreamHandler(URL.java:1154)
4XESTACKTRACE                at java/net/URL.<init>(URL.java:611(Compiled Code))
4XESTACKTRACE                at java/net/URL.<init>(URL.java:502(Compiled Code))
4XESTACKTRACE                at java/net/URL.<init>(URL.java:451)
4XESTACKTRACE                at javassist/ByteArrayClassPath.find(JRebel:92)
...
4XESTACKTRACE                at java/lang/ClassLoader.loadClassHelper(ClassLoader.java:924(Compiled Code))
5XESTACKTRACE                   (entered lock: org/eclipse/core/launcher/Main$StartupClassLoader@0x0000000062F3AA88, entry count: 1)

3XMTHREADINFO      "other-thread" J9VMThread:0x0000000001B26300, omrthread_t:0x00007F4120A5D060, java/lang/Thread:0x00000000629F1458, state:B, prio=5
3XMTHREADBLOCK     Blocked on: org/eclipse/core/launcher/Main$StartupClassLoader@0x0000000062F3AA88 Owned by: "main" (J9VMThread:0x000000000175EC00, java/lang/Thread:0x00000000624290B8)
3XMHEAPALLOC             Heap bytes allocated since last GC cycle=0 (0x0)
3XMTHREADINFO3           Java callstack:
4XESTACKTRACE                at org/osgi/util/tracker/AbstractTracked.size(AbstractTracked.java:356)
4XESTACKTRACE                at org/osgi/util/tracker/ServiceTracker.getServiceReferences(ServiceTracker.java:526)
5XESTACKTRACE                   (entered lock: org/osgi/util/tracker/ServiceTracker$Tracked@0x00000000627F0A90, entry count: 1)
4XESTACKTRACE                at org/eclipse/osgi/framework/internal/protocol/StreamHandlerFactory.createInternalURLStreamHandler(StreamHandlerFactory.java:162)
4XESTACKTRACE                at org/eclipse/osgi/framework/internal/protocol/StreamHandlerFactory.createURLStreamHandler(StreamHandlerFactory.java:118)
4XESTACKTRACE                at java/net/URL.getURLStreamHandler(URL.java:1154)
4XESTACKTRACE                at java/net/URL.<init>(URL.java:611(Compiled Code))
4XESTACKTRACE                at java/net/URL.<init>(URL.java:502(Compiled Code))
4XESTACKTRACE                at java/net/URL.<init>(URL.java:451)
4XESTACKTRACE                at java/net/URI.toURL(URI.java:1100)
4XESTACKTRACE                at com/ibm/jsse2/av.a(av.java:92)
4XESTACKTRACE                at com/ibm/jsse2/av.engineInit(av.java:7)
4XESTACKTRACE                at javax/net/ssl/TrustManagerFactory.init(TrustManagerFactory.java:25)

updates for release 3.23.1

Merge pull request #204 from beikov/issues/171

[#171] Allow running on JDK 8 and possibly on JDK 7 as well. Fixes #171

[#171] Allow running on JDK 8 and possibly on JDK 7 as well. Fixes #171

updates for release 3.23.0-GA

fixes a minor bug in a test case.

Merge pull request #198 from beikov/JASSIST-271

JASSIST-271 Allow running on Java 11 by falling back to ClassLoader.defineClass for Java > 10

Merge pull request #168 from cmelchior/cm/bug/classpath-close

[WIP] Fix leaking file handlers

JASSIST-271 Allow running on Java 11 by falling back to ClassLoader.defineClass for Java > 10

fixes a bug reported in issue #155 (Javassist Incorrect Code when wrapping with try/catch)

adds Modifier#isVarArgs()

adds Modifier#isVarArgs()

changes getDefinedPackage back to getPackage when Java is < 9.

Merge pull request #159 from nickl-/scoped-pool

Fixes to javassist.scopepool

Merge pull request #158 from nickl-/secure-privileged

Secure privileged

Merge pull request #166 from nickl-/test-ctfield-make-with-this

Unit Test: CtField.make referencing 'this'.

Merge pull request #167 from nickl-/innerclass-modifiers

Fix for #152 inner class modifiers.

updates build.xml to be compatible to pom.xml

Add missing unit test

Fix merge mistake

Fix #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.

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.

Fixes to javassist.scopepool

Applied parameters and squashed all compiler warnings.
Changed loops to enhanced for where appropriate.
Fixed SoftValueHashMap:
Made it a raw type.
Overwritten implementation of the `get` method correctly unwrapped the reference but the methods
`containsValue` and `entrySet` needed to be fixed.
Implemented the remaining outstanding methods from the `java.util.Map` contract `values`, `putAll`
and `keySet`.
Dropped `AbstractMap` inheritance as we were not inheriting anything anymore.
Changed internal map to `ConcurrentHashMap' to be atomic and parallel safe.

Found original unit tests from jboss retro which are now updated to reflect the javassist
versions of the scoped pool classes. Added tests for the SoftValueHashMap including
test which overloads the heap to ensure the cache cleans itself appropriately.

Define parameters for returned iterator.

Some functional unit tests proofs.

Proves that you cannot access restricted functionality then goes ahead and
mock test all 3 exposures JAVA_9, JAVA_7+ and OTHERS by changing the
private static final class referenced default chosen on initialise.

Some code comments to fill in gaps.

Even now Oracle still not happy.

Runtime still throws warnings illegal actions even after all that, there's
just no logic to it. We can use Unsafe but don't touch setAccessible. O well
might as well enjoy Unsafe while we still can.

Wrapped the Unsafe and added method cache with varargs method for invoke calls.
Can still do a lot with it but it does what it needs to for now.

Refactor definePackage as well.

Yes it is deprecated but seems the right thing to do

Give helper private constructor fwiw.

Wont stop you from getting an instance but it makes it more trouble at least.
Also reduce the visibility of the enum method.

Add caller class checking.

The only thing remotely inforceable is caller class checking. You cannot
bypass an exception with setAccesible. Oracle went and removed
Reflection.getCallerClass() completely in favour of StackWalker. At least
we can share the SecurityManage ClassContext, SecurityActions seems the
appropriate venue for our stack trace peek to support the older versions

MethodHandles for JDK7+.

Java 7 and 8 does support method handles so we can cater for them. When doing
repeated invokes to the same method, keeping a reference to the Method handle
is much faster than reflection. Also the API is cleaner not having to Object[]
args etc. Worth the effort...

Refactored SecActs methods and retain 1.6 source.

Remembered that 1.6 won't know about MethodHandles so must use reflection methods for them still.

Add sun.misc.Unsafe privileged retrieval to SecActions.

Also do it anonomously so as not to raise the alarms and upset the powers to be.

Type check and paramatized rawtypes for SecurityActions.

Some whitespace got shunted around a bit too.

Add getMethodHandle to SecurityActions.

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.

Make an effort to secure privileged use.

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.

Cleanup

Merge branch 'master' into cm/bug/classpath-close

# Conflicts:
# pom.xml
# src/main/javassist/ByteArrayClassPath.java
# src/main/javassist/ClassClassPath.java
# src/main/javassist/ClassPoolTail.java
# src/main/javassist/LoaderClassPath.java
# src/main/javassist/URLClassPath.java

Remove close method on ClassPath. Add unit tests for new behaviour.

Update test README

Ignore IntelliJ files

Merge pull request #160 from nickl-/squash-warnings

Source walk/Spring clean/Parameterize/Enhance/Squash warnings et.al.

Organize imports to disolve ambiguity.

Squash compiler warnings in tests.

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.

Source walk/Spring clean/Parameterize/Enhance et.al.

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.

Fix deperecated junit.framework.Assert.

Changed to org.junit.Assert.

Fix deperecated junit.framework.Assert.

Changed to org.junit.Assert.

Implement hashCode for Annotation.

To honour the Object.equals contract.

Implement hashCode for Type.

To honour the Object.equals contract.

Implement hashCode for MultiType.

To honour the Object.equals contract.

Implement hashCode for MultiArrayType.

To honour the Object.equals contract.

Fix deperecated junit.framework.Assert.

Changed to org.junit.Assert.

Suppress deprecation Applet.

Suppress deprecation ClassPool.makePackage

Fix deprecation getPackage to getDefinedPackage

Added default serialVersionUID.

Upgrade junit from ver 3 to 4.

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.

Add version to maven-bundle-plugin.

In order to squash the mvn build warnings.

Configure source encoding as UTF-8.

In order to squash the mvn build warnings.

Move runtests to build directory.

Move the test output folder 'runtests' to target or build directory so that mvn clean can clean that.

changes ClassPool#appendSystemPath().  The new implementation appends ClassClassPath when the JVM is earlier than 9.

fixes a bug of getAnnotations().  Some hidden annotations were not ignored.

upgrade maven-javadoc-plugin to 3.0.0-M1 to support publishing of 3.22.0-GA

removes JvstTest4#testJIRA150b because it is obsolete.

3.22.0-GA release

updates ConstPool to support Module and Package

modifies CtClass#setModifiers() to correctly support inner classes.

delete testHotswap() because java 9 does not seem to allow a java agent to be attached after the JVM starts

to support Java 9-ea+171

modifies the version number

adds javassist.util.HotSwapAgent

adds javassist.util.proxy.ProxyFactory#onlyPublicMethods.  If true, Javassist uses java.lang.invoke.MethodHandles.Lookup to load a class.

Merge pull request #130 from andresluuk/patch-1

A fix for older JDK8 versions: https://bugs.openjdk.java.net/browse/JDK-8041920

modifies to use sun.misc.Unsafe#defineClass so that Javassist can run with jigsaw

Update ClassFile.java

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.

updates the test code to run on Java 9 EA+149.
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.

removes javassist.ModuleClassPath.
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/

implements better annotation handling since Javassist may fail to
create a proxy object due to access controls by jigsaw.

updates Readme.html.

release 3.22.0-CR1

Merge branch 'test/java9-jigsaw'

modifies javadoc typos.

fixes typos in javadoc comments

updates files for release 3.21.0-GA

Merge branch 'master' into test/java9-jigsaw

improves the backward compatibility of LoaderClassPath.
LoaderClassPath is modified to show the same behavior in both Java 8 and 9.

enables removal of attributes

Merge pull request #100 from oreissig/remove-annotation

removal of annotations

Merge pull request #98 from PeterFeicht/master

Fix NullPointerException in ControlFlow constructor

allows adding a default method to an interface type.

implement removal of annotations

Fix NullPointerException in ControlFlow constructor.

An NPE would be thrown by the constructor when called with a method
without code.
Also clarify the documentation of a few methods.

fixes a bug in ClassClassPath.  It was reported in Issue #97.

adds javassist.jar because the unit tests fail without it.

updates pom.xml for building on Java 9.

fixes calls to a deprecated method in Java 9.

modifies a test case.  Now all the test cases run successfully with Java 9 EA.