Merge pull request #279 from csobrinho/patch-1

Fix a bottleneck. If the jar entries is big, List.contains is O(n) an…

Fix a bottleneck. If the jar entries is big, List.contains is O(n) and very slow!

This is a continuation of #271 to speed up the 3.25/26 release. More details on the issue.

for release 3.26.0-GA

adds CtClass#isKotlin() for PR #276

fixes a bug of JvstTest5#testRedundantInsertAfter() for Issue #275

adds CtBehavior#insertAfter(String,boolean,boolean) for Issue #275

fixes Issue #271

updates Readme.html

Merge pull request #267 from sam-ma/master

Fix #265 javassist.CannotCompileException: [source error] the called …

adds ClassFileWriter#addDynamicInfo

fixes typos included in PR #272 (Issue #270)

Merge pull request #272 from richbolen/JASSIST-270

JASSIST-270:  Add support for the new Dynamic constant (17) created in java 11

Remove unused file

Address feedback from PR

Incorporate feedback from the PR

Incorporate feedback from the PR

Incorporate feedback from the PR

Add support for the new Dynamic constant (17) created in java 11

fixed Issue #263

Fix #265 javassist.CannotCompileException: [source error] the called constructor is private

Merge pull request #264 from backpaper0/fix-typo

Fix typo

Fix typo

for release 3.25.0-GA

for release 3.25.0-GA

Merge pull request #253 from sam-ma/master

Fix #252 make instrumentation works on JDK11 for the inner class which has access to the private constructor of the host class

Merge pull request #254 from soul2zimate/backwards-compatible-fix

Add back FactoryHelper.toClass(ClassFile cf, ClassLoader loader, Prot…

Add back FactoryHelper.toClass(ClassFile cf, ClassLoader loader, ProtectionDomain domain) for backward compatibility in minor release.

Fix #252 make instrumentation works on JDK11 for the inner class which has access to the private constructor of the host class

adds javadoc comments for PR #250

Merge pull request #250 from skybber/master

Control ctxClassLoader usage per thread

Control ctxClassLoader usage per thread

Merge pull request #247 from NingZhang-e/issue246

Clean Java doc to fix #246

Fix Issue #246 via clean java doc.

Merge pull request #3 from jboss-javassist/master

Sync with base/master.

updates Readme.html and rebuilds javassist.jar

Merge pull request #244 from shifujun/redirectMethodCallToStatic

add a useful CodeConverter method:redirectMethodCallToStatic

add a new CodeConverter method:redirectMethodCallToStatic

fixes the bug reported in Issue #241.
It makes $r available in the code given to CtBehavior#insertBefore().

adds src/test/test5/SwitchCase.java. It also adds the javadoc comments.

Merge pull request #243 from michalkurka/michalk_fix-j11-race

JAVASSIST-242: Demonstrates a race condition in DefineClassHelper + proposed fix

Proposed fixes in DefineClassHelper & DefinePackageHelper

Don't try to clean-up after making a protected-final method accessible.
No synchronization with concurrent access can cause errors.

JAVASSIST-242: Demonstrates a race condition in DefineClassHelper

changes Readme.html

Merge pull request #231 from jboss-javassist/test/switch-case

Supporting switch-case statements with string constants.

updates Readme.html

enables switch-case with string constants

updates for relase 3.24.1

fixes another bug (supporting jdk.*) reported in #228

Merge pull request #2 from jboss-javassist/master

Sync with master /jboss-javassist/javassist/.

fixes a bug reported in #228

modifies build.xml to support JDK 7.

Merge pull request #230 from JCgH4164838Gh792C124B5/localJavassistBranch

Re-enable JDK7 runtime compatibility for Javassist 3.x

Re-enable JDK7 runtime compatibility for Javassist 3.x (until source changes preclude it).
pom.xml changes:
- lower compile source/target to 1.7 (from 1.8, introduced with Javassist 3.24.0)

Merge pull request #1 from jboss-javassist/master

Sync with jboss-javassist/javassist.

fixes bugs in javadoc comments

adds javassist.jar for test code

updates for release 3.24.0-GA

fixes typos in javadocs.

avoids ConcurrentModificationException in 3.23.1-GA reported in Issue #224.

stops a potential leak in JarClassPath openClassfile().  Issues #222.

adds a method taking Lookup to java.util.proxy.

adds Automatic-Module-Name to manifest

modifies javassist.util.proxy.DefinePackageHelper to be compatible to Java 11.
It also adds a javadoc comment to CtClass#toClass() to recommend the users to use #toClass(Class) or #toClass(Lookup).

changes CtClass#toClass() and ClassPool#toClass() etc. to support Java 11,
in other words, java.lang.invoke.MethodHandles.Lookup.

does refactoring for javassist.util.proxy.

changes the version number

adds test cases for NestHostAttribute and NestMembersAttribute.

modifies NestHostAttribute and NestMembersAttribute and adds test code

Merge pull request #218 from dmitri-gb/master

Add support for Java 11 NestHost and NestMembers attributes (JEP-181)

adds a comment for the fix to JIRA JASSIST-267

Merge pull request #210 from NingZhang-Ericsson/JASSIST-267

fixed JASSIST-267 (new )

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

Fix JASSIST-267
Clean code according to comments.

fixed JASSIST-267

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.