Collection.toArray() call style

There are two styles to convert a collection to an array: either using a pre-sized array (like c.toArray(new String[c.size()])) or using an empty array (like c.toArray(new String[0]).
In older Java versions using pre-sized array was recommended, as the reflection call which is necessary to create an array of proper size was quite slow. However since late updates of OpenJDK 6 this call was intrinsified, making the performance of the empty array version the same and sometimes even better, compared to the pre-sized version. Also passing pre-sized array is dangerous for a concurrent or synchronized collection as a data race is possible between the size and toArray call which may result in extra nulls at the end of the array, if the collection was concurrently shrunk during the operation.

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

Single Map method can be used

Reports common usage patterns of java.util.Map that could be replaced with Java 8 methods: getOrDefault(), computeIfAbsent(), putIfAbsent(), merge(), or replaceAll().

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

Manual array copy

Reports the manual copying of array contents which may be replaced by calls to System.arraycopy().

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

Collections.sort() can be replaced with List.sort()

Reports calls to Collections.sort(list, comparator) which could be replaced with list.sort(comparator).

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

Replace List.indexOf() with List.contains()

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

Manual array to collection copy

Reports the copying of array contents to a collection where each element is added individually using a for loop. Such constructs may be replaced by a call to Collection.addAll(Arrays.asList()) or Collections.addAll().

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

Fix the serial version UID contribution to work properly

lets have maven fail if those tests fail...

typo

Merge pull request #5 from larsgrefer/remove-old-version-checks

Remove checks for old Java Versions

Merge branch 'master' of github.com:eclipse/org.aspectj into remove-old-version-checks

Test on LTS versions and most recent non LTS

Merge branch 'master' of github.com:eclipse/org.aspectj into remove-old-version-checks

Merge pull request #4 from larsgrefer/feature/github-actions

Fix and improve the CI Jobs

Merge branch 'master' of github.com:eclipse/org.aspectj into feature/github-actions

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

Fix up tests and reduce verbosity on J11

Attempting to get J11 build happy at github

Making tests behave on 11

Merge branch 'feature/simplify-ajdoc' of github.com:larsgrefer/org.aspectj into feature/github-actions

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

Simplify checks for Java 8 to true

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

Always run javadoc using the ToolProvider API

`com.sun.tools.javadoc.Main` isn't available in recent Java versions (13+)

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

Trying to reduce test verbosity

Remove checks for old Java Versions

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

Adjust ASM version number used to work properly with Records (J14)

Fix compilation of JavadocRunner for Java 13+

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

Fix JRockitAgentTest for Java 9 and 10

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

Test more Java versions

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

Print used tool versions

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

Fix compilation on Java 11

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

Merge pull request #3 from larsgrefer/feature/github-actions

Create a simple CI Job using GitHub Actions

Merge pull request #2 from larsgrefer/feature/diamond-operator

Use the diamond operator where possible

Fix Javadoc generation on Java 11

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

Fix Testcase

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

Create maven.yml

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

Use the diamond operator where possible

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

Merge pull request #1 from larsgrefer/feature/java5

Update the code to Java 5 features

Merge branch 'master' into feature/java5

Add a debug flag to workaround issue until we can sort it (565713)

Unnecessary unboxing

Reports "unboxing", e.g. explicit unwrapping of wrapped primitive values. Unboxing is unnecessary under Java 5 and newer, and can be safely removed.

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

Unnecessary boxing

Reports explicit boxing, i.e. wrapping of primitive values in objects. Explicit manual boxing is unnecessary under Java 5 and newer, and can be safely removed.

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

'String.indexOf()' expression is replaceable with 'contains()'

Reports any String.indexOf() expressions which can be replaced with a call to the String.contains() method available in Java 5 and newer.

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

'while' loop replaceable with enhanced 'for' loop

Reports while loops which iterate over collections, and can be replaced with an enhanced for loop (i.e. foreach iteration syntax).

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

'for' loop replaceable with enhanced 'for' loop

Reports for loops which iterate over collections or arrays, and can be replaced with an enhanced for loop (i.e. the foreach iteration syntax).

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

Polish

Fix serialVersionUID initialization for java9+

Move initialization of static final serialVersionUID field to clinit, since as of java9+, the runtime won't allow setting final fields outside clinit

Signed-off-by: Eli Finkel <eyfinkel@gmail.com>

Rev to 1.9.7.BUILD-SNAPSHOT

Fixed a misplaced <para> element in the Docbook documentation for ajc

AspectJ 1.9.6 final bits

Update ASM and fix packaging for matcher artifact

polish and new test for 550705

558995 testcode

Fix 550705: tricky intermittent verify error

Polish

Added basic ThreadLocalAwareRepository - groundwork for 561819

Fix for 558995: NullPointerException at org.aspectj.weaver.ReferenceType.getWeaverState

few more bits to ignore

Reorganise + comment .gitignore, ignore IntelliJ IDEA projects

Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>

fix tests running on windows

This caused checkRtJar to return an error, which led to some tests to fail

Signed-off-by: Eli Finkel <eyfinkel@gmail.com>

Exclude some linux created debug files

Fix support for -14 on command line - was missing from JDT

Corrected links inside poms

polish

Include JDTCore for Java14

polish

polish

polish

Move to 1.9.6.BUILD-SNAPSHOT version

1.9.5 release versions in poms

polish

1.9.5 readme

polish

organize imports

Debug for 551732

fix comment

Fix 550494

552724: fix typo

first textblock test for Java13

Java 13 support

Update to an ASM for Java13

Fix Bug 551084 - NullPointerException at org.aspectj.ajdt.internal.core.builder.AjBuildManager.checkRtJar

Fix Bug 387122 - threadlocal in weavingadaptor is potential memory leak

543657: tweak to use current data rather than last known packed data

Moved tests around for avoiding weaving switch infrastructure method

- dont't weave synthetic enum helper method for switch
- add test for switch on enum with around all advice

Signed-off-by: Semyon Danilov <samvimes@yandex.ru>

make test a bit more reliable

Fix Bug 550290 - Lack of TypeSafeEnum#hashCode may lead to non-deterministic bytecode

Fix Bug 550696 - ClassCastException during load time weaving: InstructionHandle cannot be cast to BranchHandle

polish

polish

Fix 548785: Check level is >=1.8 when looking to skip inlining around advice lambdas

Fix 548860: don't pass lowercased string through

Fix 547808: npe regression for multi @Around

Update to 1.9.5.BUILD-SNAPSHOT in poms

polish 1.9.4

1.9.4 POMS

Final bits of AspectJ 1.9.4

Fix 546303: correct entries in weaver manifest for ltw

pushed versions to 1.9.4.BUILD-SNAPSHOT

polish for 1.9.3

Updated with Java12 support