Fix MultiProjectIncrementalTests.testAspectPath_pr265693, 2nd try

Do not assume a certain element order for tree nodes if there can be
more than one.

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

Try to fix MultiProjectIncrementalTests.testAspectPath_pr265693

Check if path vs. package name discrepancy makes test fail on Linux. On
Windows it passes. So let's find out if file p/Asp.java vs. pkg.Asp
causes the problems.

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

AjcTestCase cosmetics: surround command in failure report by quotes

This is the same for successful tests in class CompilationResult.

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

Fix resource leak in FileUtil

This made ModuleTests.testBuildModuleAndApplyAspectsFromAspectPath fail
because a file delete job for a module JAR failed after a previous
compile job had called FileUtil.isZipFile(File) in which the opened zip
file was never closed. A try with resources fixes that.

Maybe the corresponding test worked on Linux before, I did not try. I
just know that Linux is more forgiving about deleting open files while
on Windows they are being locked, which makes Windows the better system
to search for open file leaks.

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

AjBuildManager: use try with resources in 2 places

I was debugging something in ModuleTests, trying to find out if there
are resource leaks. They were not in this class but in FileUtil - see
next commit. However, the little refactoring here does not hurt either.

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

Fix path separators in ajc190.xml, make tests run on Windows

Replace ':' by ";" which will be replaced to the platform
separator automatically. A fixed value of ':' does not work on Windows.

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

Merge pull request #32 from kriegaex/installer-windows_10-windows_server

Support Windows 10 and Windows Server 2016/2019 in installer

Support Windows 10 and Windows Server 2016/2019 in installer

Those versions were not detected until now, which lead to bogus Windows
batch files forwarding only 9 Ajc parameters to the Java process via
"%1 %2 %3 %4 %5 %6 %7 %8 %9" instead of "%*".

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

Merge pull request #29 from ebourg/patch-1

Update the URL of BCEL

Update the URL of BCEL

Merge pull request #23 from larsgrefer/feature/java15

Update GitHub Actions from Java 14 to 15

Merge pull request #27 from capgen628/patch-1

Remove call to System.out

Remove call to System.out

Update GitHub Action from Java 14 to 15

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

Merge pull request #22 from jamesjer/master

Fix apostrophe in the license

Fix apostrophe in the license

Merge pull request #21 from jamesjer/master

Fix javadoc errors

Typo fixes

Fix javadoc references to nonexistent fields, classes, or packages

Fix misplaced or incorrect javadoc tags

Fix misplaced or incorrectly nested HTML tags

Fix incorrect HTML entities in javadoc comments

Merge pull request #19 from jamesjer/master

Give docbook IDs unique names

Give docbook IDs unique names

Revert to 1.9.7.BUILD-SNAPSHOT

1.9.7 milestone 1 published

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

Split tests in a separate step on GitHub Actions

Split tests in a separate step on GitHub Actions

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

debug for test failure under github action

Update to JUnit Jupiter 5.6.2

The "old" JUnit-3-style tests will be executed by the vintage-engine

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

Merge pull request #16 from larsgrefer/cleanup/modifiers

Cleanup unnecessary modifiers

debug for test failure under github action

Remove unnecessary interface modifiers

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

Remove unnecessary enum modifiers

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

debug for test failure under github action

polish

Merge pull request #10 from larsgrefer/cleanup/pom

Cleanup the Maven pom.xml files

Reduce test output for incremental comp tests

Merge pull request #13 from larsgrefer/feature/collection-performance

Improve performance and readability of collection handling

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>

Remove non-existing dependency

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

Remove project.parent.relative path as ../pom.xml is already the default

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>