Axel Howind

javassist creates invalid class files for Java records when injecting static fields and adding static initializer code

1

# Description of problem When injecting a field and a static initializer into a Java record class, Javassist generates invalid class files. I have seen this for several record classes...

add module-info instead of automatic module name

2

This PR adds a named module instead of an automatic module name. This is required to jlink compatibility.

Hi, six weeks ago my fix for #244 was merged. Could we do a new release? #244 was tagged as "enhancement", but it really fixes a bug. It does not...

NP_NONNULL_RETURN_VIOLATION: false positive when using String.formatted()

4

Running Spotbugs 4.5.0 on this method: public @NotNull String version() { return "%d.%d.%d%s".formatted(major, minor, patchLevel, !suffix.isEmpty() ? "-"+suffix : ""); } yields: > NP_NONNULL_RETURN_VIOLATION: Method may return null, but is...

Update plugin for Java 21+ compatibility

1

The plugin does not work with current stable Java. I have forked the plugin to be able to build my projects again and will provide a PR. Either my PR...

fix #153 (JDK 21+ compatibility)

6

This PR fixes #153: - update ASM to 9.7 (already "knows" about Java 23 and will probably work with that version once it's released) - update Gradle wrapper to 8.9...

release Java 23 compatible version

1

The Plugin currently (version 3.0.1) fails to work on projects with JDK 23. There's already an unmerged PR that addresses this: https://github.com/beryx/badass-jlink-plugin/pull/271 I add this new issue to describe a...

- fix invalid bitwise operations: 1

Run static analysis on merge/commit

5

I think we should add some static analysis tool that detects common programming errors to the Github pipeline. There are several options free of charge for Open Source projects, including...

- declare the class explicitly as final (it was effectively final before because it only has a private constructor) - use a ConcurrentHashMap to avoid the need for explicit synchronization...