AALOAD should give null when the input array is null.

See merge request !11

improve the doc comment of ModuleVisitor

See merge request !7

FindBugs: typo in the code that check if a module is declared open

See merge request !6

The copyright header is missing on top of ModuleInfoBndPlugin.java

See merge request !5

Remove the optimizer and the asm-all and asm-debug-all jars.

See merge request !3

Fix the handling of inner classes with EXPAND_ASM_INSNS.

See merge request !4

Resetting the inner class data structures so that they are rebuilt when revisiting the code with a ClassReader does not work, due to the code in visitInnerClass to avoid duplicate entries in this table (based on state stored in the constant pool entries, which are not reset). In fact, thanks to this code to avoid duplicate entries, we can skip resetting the inner class data structures here, and be assured that they will not change when revisiting the class with a ClassReader.

Add a check for the class version and format code. Also don't output a class if it fails verification.

MethodType and ldc MethodHandle

The jar size is no longer an issue for the vast majority of the ASM users, unlike the lack of debug info in the default jars. This changes keeps the debug info in the default jars by removing the optimizer. The asm-debug-all jars is then no longer necessary, and is thus removed.

Similarly, maintaining compatibility with the JDK 1.3 is no longer necessary (it is unsupported since a long time). Instead, this change uses the JDK 1.5 as the new baseline. This is now striclty enforced (the build will fail if a more recent API is used internally), for all jars (previously different jars were using different baseline APIs).

Fix the computation of the Label.DEBUG flag.

See merge request !2

The current code can incorrectly classify a Label as "used only for debug", which can yield incorrect stack map offsets due to skipped labels in MethodWriter.visitLabel (see bug #317791). This happens when a label is found for the first time in a debug attribute (it is then flagged as DEBUG), and then appears again in a stack map frame (these labels are extracted after those from the debug attributes, during the bytecode parsing, but the DEBUG flag, if present, is not removed then).

The fix consists in removing the DEBUG flag, if present, when creating or reusing a label for non-debug attributes.

Update the V1_9 constant name to follow http://openjdk.java.net/jeps/223

See merge request !1

Don't use the classes we want to test for comparing classes in byte array form in AbstractTest, this can hide bugs (ideally we shouldn't use ClassReader either, but this is harder to do). Also fix 3 bugs found thanks to this refactoring.