GitLab

Bug 316398 occurs because the method is forgotten in the equality and so objects that should not be merge (because they are related to different methods) are merged.

Bug 316471 occurs when the annotation field of an annotation is of type class and its name is not in the string table.

http://forge.ow2.org/tracker/?group_id=23&func=detail&atid=100023&aid=316405

Check that the set of integers can be put in a range. Modify also the generation of instruction so that if the set of registers does not describe a range, nothing is written. A test case has been added to the regression suite.

A per class structure was not correctly reinitialised. bug #316397 - based on patch by Vipin Aravind.

The old way, we get a multiset of AnnotationSetRefList that we commit to the output. But in fact some are repeated and we should get only one per item. This is a solution. Another is to remove the notion of structural equality. - Bug #316394 - modified version of Vipin Aravind patch

Added corrected reading of default annotations that are not primitive values or arrays of primitive values.
This requires also changes in the generation of the class. 
Some test cases have been added.

Cleaner code + check that the encoded value is not null. Will raise a failure otherwise and will avoid later errors hard to track.

Incorrect check on innerName. The encoded value is created with null if innerName is null (as name is not null).
Likely source of #316369.

The outSize was incorrect for an invoke-static/range (one too large). Dexdump will complain because it checks that the out is compatible with the number of register (as it must be built as a slice, it cannot be bigger than the number or registers).

- Changes to InnerClass annotation: creation of an explicit MemberClass annotation (no more test if name is null, this was wrong for named inner classes)
- InnerClass name computed from the real name of the class. It cannot be recomputed on named InnerClasses.
- Propagation of the MemberClass annotation to the structure of asmdex (tree visitor, logelements, etc.)
- Modification of the test case structure to get more visibility on what is wrong (parameterized test suite)
- Updated Junit.
- Added test case for Inner and Member class covering all the known variants (static or not, named or not, member not inner).

Variation on code by Panxiaobo.
- adds supportCodeItem
to Method.
- use it instead of isInterfaceOrAbstract.
- adds a test case (helloJni.dex) with a
native declaration (sample from the NDK).

visitLabel missing if no debug info (tryCatch
structure visit was also wrong).

Adds package information and fix the remaining javadoc comments that were incorrect for javadoc generation.

Put Version tags in all the constructors as in ASM. Not sure it is worth the pain. Future testing nightmare ahead !

stricter).

stricter).

switch (read an int too much).
Bogus when there are several
sparse switch definition (but
silently catched if only one).