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

1.9.3 inclusion

1.9.3.RC1 final bits

remove unnecessary classpath entries