Merge branch 'master' into feature/collection-performance

Merge pull request #15 from larsgrefer/update/maven/3.6.3

Update to Maven 3.6.3

Merge pull request #12 from larsgrefer/cleanup/unused-code

Cleanup unused imports

Merge pull request #11 from larsgrefer/feature/editorconfig

Add a basic .editorconfig file

Cleanup unused imports

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

Update to Maven 3.6.3

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

Update to JUnit 4.13

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

Merge pull request #7 from larsgrefer/patch-2

Use the Maven Wrapper for GitHub Actions

Reduce test output for incremental comp tests

Weaken Collection declarations

Reports on declarations of Collection variables made by using the collection class as the type, rather than an appropriate interface.

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

Fix test on J14

Redundant Collection.addAll() call

Reports Collection.addAll() and Map.putAll() calls after instantiation of a collection using a constructor call without arguments. Such constructs can be replaced with a single call to a parametrized constructor which simplifies code. Also for some collections the replacement might be more performant.

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

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...

Use the Maven Wrapper for GitHub Actions

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

Add a basic .editorconfig file

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

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