Merge pull request #470 from shifujun/repair_IDEA

Minimal changes make IDEA 2023.2 work out-of-box

Merge pull request #469 from shifujun/insertAuxInitializer_stacksize

Fix insertAuxInitializer may cause inconsistent stack height problem

Merge pull request #468 from vimfun/patch-1

Update tutorial.html: `javassist.util.HotSwapper`

Minimal changes make IDEA 2023.2 work out-of-box

Correct maven-compiler-plugin JDK version for IDEA import Compiler bytecode version.

Remove JvstTestRoot's constructor, otherwise IDEA cannot show Run icon aside testMethod.
Another way is adding suite() static method to those classes.

Now we can clone and open project with IDEA 2023.2. Debug single test case with IDEA is very useful.

Fix insertAuxInitializer may cause inconsistent stack height problem

Usually, constructor only load super class's constructor's init params into stack.
After this() or super() called, stack will be empty. If so, we insertAuxInitializer
right after this() or super() can reuse max stack size if it enough.

But, there is some weird class out there, their constructors load all in-constructor
init field value into stack before this() or super() call. In this case,
after this() or super() call, stack is not empty, even maybe full.

In summary, insertAuxInitializer should increase MaxStack anyway.

Update tutorial.html: `javassist.util.HotSwapper`

The code path is fixed.

Merge pull request #467 from shifujun/453

Remove DUP check in TransformNewClass

Merge pull request #448 from catsalty/patch-2

Fix issue in no-standard [new] instruction replace

Merge pull request #466 from shifujun/pr-fix-lookupMethod

Fix MemberResolver.lookupMethod bug when super class has more precise…

Remove DUP check in TransformNewClass

DUP usually come after NEW, but not necessary.

#453

Fix MemberResolver.lookupMethod bug when super class has more precise match

When onlyExact=false and super class have a more precise match,
it should not return with current class's maybe result.

New added testSuperCall reveals the problem.

Merge pull request #463 from wuwen5/java21-inner-class

fixes Issue 462 (In Java21, the ConstPool API throws NPE for MethodParameters attributes without parameter names)

fixes GitHub Issue 462 (Internal class issues in the Java 21)

Fix issue in no-standard [new] instruction replace

A standard *new* construct bytecode will generate like:

```
new okhttp3/OkHttpClient
dup
invokespecial okhttp3/OkHttpClient <init> ()V
astore 1
```

but a few part of compiler will generate no-standard bytecode like:

```
new okhttp3/OkHttpClient
dup
astore 1
invokespecial okhttp3/OkHttpClient <init> ()V
```
which will cause javassist build stack error.

This commit will fix it

Update README.md

Merge pull request #434 from fanofxiaofeng/patch-1

try to fix trivial link error

try to fix trivial link error

try to fix trivial link error

updates javassist.jar for 3.29.2-GA.

prepares a new release

Merge pull request #427 from dakusui/master-issue-426_includeAutomaticModuleNameInManifestMf

Includes Automatic-Module-Name in MANIFEST.MF

Issue-#426 Format a line in the pom.xml.

Issue-#426 Include Automatic-Module-Name in MANIFEST.MF and bump up the version to 3.29.2-GA

updates README.md

adds Issue #423 to Changes.md

for releasing 3.29.1

prohibits too many items from being added to a constant pool.
the previous commit was wrong.

prohibits too many items from being added to a constant pool

updates Changes.md to note Issue #414

fixes typos in *.md

removes Readme.html and adds Changes.md and Examples.md

prepares for 3.29.0-GA release

removes an unused import declaration as PR 384 mentions.

Merge pull request #382 from tim-hoffman/PR_RemoveStringBuffer

Replace StringBuffer with StringBuilder, other minor String optimizations

Merge pull request #383 from tim-hoffman/PR_ArrayCopyPerformance

Replace array copy loops with System.arraycopy

uncomments test code suggested by PR405

cancels PR#391, which does not pass all the tests

fixes a wrong indentation

Merge pull request #391 from derklaro/fix-method-naming-syntax

Only wrap the method name if the method doesn't come from the same declaring class

does refactoring

fixes a bug in SignatureAttribute.renameClass().

fixs a compilation problem although one test still fails.

Merge pull request #299 from pietrobraione/master

Fix renaming of classes in presence of generic signatures and nested classes.
This pull request has some problems but I'll fix them later.

Merge branch 'master' into master

Merge pull request #278 from ximsfei/master

fixes a bug of ClassFile#renameClass(), some imports could not be successfully renamed.

Merge pull request #390 from derklaro/master

Lower requirement for an extra type in constructor to Java 7-

changes test code because package java.rmi.activation.ActivationGroupDesc does not exist any longer.

deletes src/test/test2/*.class

Merge pull request #399 from carldea/master

Using TCK Tested JDK builds of OpenJDK

Merge pull request #409 from aschleck/patch-1

Remove redundant type check

Remove redundant type check

The same condition is repeated twice (caught by FindBugs.)

Using TCK Tested JDK builds of OpenJDK

The AdoptOpenJDK has been discontinued since July 2021. When using Zulu you get all the latest updated (TCK Tested) builds for all versions of OpenJDK.

Also added are fixed (major) release version(s) such as 11.0.3. This is often a good practice whenever a build/test triggers (push/pull events) this can determine if the latest JDK (11) had failed the build vs something in your code that caused it. For example, when building with JDK 11.0.3 (fixed version) the build passes (green) and JDK 11 fails (red) will mean that the latest JDK (11) was the cause and not your code.



Note: Other distributions such as Temurin do not support archived fixed release prior to Sept. 2021.

Merge pull request #395 from aburaksahin/master

Fix for NullPointerException on detach()

Fix for NullPointerException on detach()

Only wrap the method name if the method doesn't come from the same decl class

Lower requirement for an extra type in constructor to Java 7-

Replace array copy loop with System.arraycopy

Replace remaining StringBuffer uses with StringBuilder

Replace StringBuffer with StringBuilder and other minor String optimizations

fixes GitHub Issue 378 (wrong link for apache license)

updates javassist.Loader to exclude jdk.internal.

release 3.28.0-GA

Add an action badge.

Create maven.yml

updates Readme.html since #305 has been fixed by PR #306

Merge pull request #306 from nschaefe/master

fix issue 305: CodeConverter.replaceArrayAccess leads to java.util.ConcurrentModificationException

updates Readme.html since #363 has been mergeed

Merge pull request #363 from michalkurka/master

Fix a race condition in CtClassType#getClassFile3

updates Readme.html since #350 has been fixed

Merge pull request #351 from sgjesse/issue-350

This fixes Issue #350.
Check for extended frame type when updating StackMapTable offset.

Fix a race condition in CtClassType#getClassFile3

javassist fails to find a class when concurrently running process
compresses the class (converts classfile to raw bytes)

the idea of the fix is to make sure to only update rawClassfile and classfile
under lock in getClassFile3, all other places that modify classfile are
already synchronized

when reading the object state, we need to read under lock both classfile and
rawClassFile otherwise we might get an inconsistent state

fixes a security problem in pom.xml

fixes typos for #357

Merge pull request #357 from eshizhan/master

add functions for getting the parameter names of method

add unit tests for LocalVariableAttribute.variableNameByIndex and MethodParametersAttribute.parameterName

add functions for getting the parameter names of method

Check for extended frame type when updating StackTableOffset offset

Fixes #350

fixes Issue #339

updates the version number

fixes a bug of javassist.bytecode.stackmap, which was reported as Issue #328.

Merge pull request #307 from sarxos/patch-1

Fix typo in TypeChecker

Fix typo in TypeChecker

Should be "bad field access" not "bad filed access".

fix CodeConverter.replaceArrayAccess leads to java.util.ConcurrentModificationException

updates Readme.html to mention how to release.

for release 3.27.0-GA

Fixes to support the full syntax of possible signatures

Fix renaming of classes in presence of generic signatures and nested
classes.

fixes a bug involuved in Pull Request #294

Merge pull request #294 from junwen12221/master

Supports parsing 'byte[] a = new byte[]{};'.  Only parsing is supported.  Javassist reports a not-supported error.

Merge branch 'master' of github.com:jboss-javassist/javassist

fixdes Issue #282

Support parsing 'byte[] a = new byte[]{};'

add unit test

Merge pull request #281 from diptadas/patch-1

Fix instruction printer double constant

Fix instruction printer double constant

updates Readme.html

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.

fixes a bug of ClassFile#renameClass(), some imports could not be successfully renamed

for release 3.26.0-GA

adds CtClass#isKotlin() for PR #276