aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Cleanup redundant boxing.Andrey Turbanov2021-11-209-14/+14
| | | | | Methods Integer.parseInt/Boolean.parseBoolean should be preferred over Integer.valueOf/Boolean.valueOf/ if final result is primitive. They are generally faster and generate less garbage.
* Merge pull request #96 from kriegaex/release-198Andy Clement2021-11-102-12/+18
|\ | | | | Release version 1.9.8.RC2
| * Set version to 1.9.8-SNAPSHOTAlexander Kriegisch2021-11-1029-29/+29
| | | | | | | | Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * Set version to 1.9.8.RC2V1_9_8_RC2Alexander Kriegisch2021-11-1029-29/+29
| | | | | | | | Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * Bump JDT Core to 1.9.8.RC2Alexander Kriegisch2021-11-101-1/+1
| | | | | | | | | | | | | | Fixes JDT Core bug https://bugs.eclipse.org/bugs/show_bug.cgi?id=576093 and related AspectJ issue #95. Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * Revert "Bump com.thoughtworks.xstream:xstream to 1.4.18"Alexander Kriegisch2021-10-111-1/+2
| | | | | | | | | | | | | | | | Reason: 1.4.18 causes exceptions during deployment. This reverts commit 87fc651c50dd13b46c0cd869af2150ffa8b94a5f. Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * Improve README-198.htmlAlexander Kriegisch2021-10-111-10/+15
|/ | | | | | Adjust both content and document structure. Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
* Bump com.thoughtworks.xstream:xstream to 1.4.18Andy Clement2021-10-081-1/+1
|
* Merge pull request #93 from kriegaex/java-17Andy Clement2021-10-0850-202/+575
|\ | | | | Java 17
| * Set version to 1.9.8-SNAPSHOTAlexander Kriegisch2021-10-0829-29/+29
| |
| * Release version to 1.9.8.RC1V1_9_8_RC1Alexander Kriegisch2021-10-0829-29/+29
| | | | | | | | Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * Add AspectJ 1.9.8 release notesAlexander Kriegisch2021-10-083-55/+161
| | | | | | | | Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * Bump JDT Core to 1.9.8.RC1Alexander Kriegisch2021-10-081-2/+1
| | | | | | | | Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * Merge branch 'master' into java-17Alexander Kriegisch2021-10-082-4/+12
| |\ | |/ |/|
* | Merge pull request #91 from kriegaex/nexus-staging-new-workaroundAndy Clement2021-09-282-4/+12
|\ \ | | | | | | Add JDK 16+ workaround for Nexus Staging Maven Plugin
| * | Add JDK 16+ workaround for Nexus Staging Maven PluginAlexander Kriegisch2021-09-282-4/+12
|/ / | | | | | | | | | | | | | | | | Plugin version 1.6.8 does not work on JDK 16+ without special MAVEN_OPTS opening certain modules, because the XStream version used needs it. One workaround is to use a more recent XStream version. TODO: remove plugin dependency after OSSRH-66257, NEXUS-26993 are fixed. Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * Fix Java17PreviewFeaturesTests (locale-specific floating-point output)Alexander Kriegisch2021-10-083-4/+6
| | | | | | | | | | | | | | The test worked on my local workstation with German locale, but not on GitHub with English locale. Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * Remove Java 8 from GitHub workflow (ECJ needs Java 11+)Alexander Kriegisch2021-10-081-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Since JDT Core 3.27 (Java 17) and Eclipse 2021-09 (4.21), respectively, ECJ no longer works on JDK 8. Even if we backport JDT Core classes, some of its dependencies contain Java 11 class files, which ultimately also means that ACJ no longer works below JDK 11 due to those transitive dependencies, e.g. org.eclipse.core.resources-3.14.0.jar. For now, I added JDK 14 to the build matrix, i.e. we currently have 11, 14, 17. When JDK 18 is released, we can switch to 11, 17, 18, i.e. keep the two LTS releases plus the latest one. Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * Rename DOM AST class TypePattern to AbstractTypePatternAlexander Kriegisch2021-10-0820-60/+59
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since JDT Core 3.27 (Java 17), there is a name clash, because the new class org.eclipse.jdt.core.dom.TypePattern (JEP 406) gets relocated to org.aspectj.org.eclipse.jdt.core.dom.TypePattern during shading. Fortunately, this made tests like AjASTTest and AjAST5Test fail with rather nasty errors like: java.lang.VerifyError: Bad return type (...) Type 'org/aspectj/org/eclipse/jdt/core/dom/TypePattern' (...) is not assignable to 'org/aspectj/org/eclipse/jdt/core/dom/Pattern' (...) TODO: Update AJDT references to the renamed class in the following classes after refreshing the AspectJ sources: - ExtraPackageReferenceFinder - ExtraTypeReferenceFinder This also means, that for Eclipse 2021-09 (4.21) we need a new AJDT update site, because simply deploying to the 4.19 one would probably lead to problems in the IDE. Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * Bump GitHub CI build from JDK 17-ea to 17, remove 16Alexander Kriegisch2021-10-021-1/+1
| | | | | | | | Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * Update ECJ version, activate Java 17 preview features testsAlexander Kriegisch2021-10-024-23/+19
| | | | | | | | | | | | | | After JDT Core (ECJ) was updated to the final Java 17 feature set, the tests now pass as expected. Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * Add Sonatype OSSRH snapshot (plugin) repositoriesAlexander Kriegisch2021-10-021-0/+28
| | | | | | | | Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * Activate Java 17 build in GitHub workflowAlexander Kriegisch2021-09-071-1/+1
| | | | | | | | Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * Add + activate some Java 17 testsAlexander Kriegisch2021-09-079-64/+243
| | | | | | | | | | | | | | | | | | | | | | | | | | - Fix one fault sanity test configuration - Deactivate Java 16 preview tests (no longer supported by Java 17 compiler) - Test sealed classes as final on Java 17 (no longer preview) - Add tests for JEP 406, pattern matching for switch (preview). At present, the beta 17 branch of JDT Core does not handle the tested features and expected compile errors correctly yet, so I had to temporarily deactivate test execution, only printing TODO messages. Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * Test comment cosmeticsAlexander Kriegisch2021-09-0711-9/+16
| | | | | | | | Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * In ITD processing, use setter instead of assigning Scope directlyAlexander Kriegisch2021-09-073-3/+6
| | | | | | | | | | | | | | | | | | | | | | | | Change calls like pre.scope.parent = newParent; to this pattern: // Use setter in order to also update member 'compilationUnitScope' pre.scope.setParent(newParent); This should fix lots of failing tests after updating JDT Core. Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * Add AJDT support for printing switch expressionsAlexander Kriegisch2021-09-071-7/+32
| | | | | | | | Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * Add Sonatype OSSRH snapshots repositoriesAlexander Kriegisch2021-09-071-0/+28
| | | | | | | | | | | | | | | | | | Otherwise - JDT Core 1.9.8-SNAPSHOT (classic OSSRH snapshot URL) - AspectJ Maven Plugin 1.13-SNAPSHOT (new OSSRH snapshot URL) cannot be found Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * Bump JDT Core from 1.9.7.M2 to 1.9.8-SNAPSHOTAlexander Kriegisch2021-09-071-1/+2
| | | | | | | | | | | | | | | | | | | | Not Java 17 compatible yet, but refreshed from main branch for latest Java 16 updates in compiler (mostly about records). Let's see if CI is still green before trying to upgrade JDT Core to Java 17 beta. Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * Bump ASM 9.1 to 9.2 (Java 17+18 support)Alexander Kriegisch2021-09-071-1/+1
|/ | | | | | This is a first, preparatory step Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
* Merge pull request #86 from kriegaex/update-download-maven-pluginAndy Clement2021-09-061-2/+2
|\ | | | | Bump download-maven-plugin to 1.6.7
| * Bump download-maven-plugin to 1.6.7Alexander Kriegisch2021-09-051-2/+2
|/ | | | | | | | | After my PR https://github.com/maven-download-plugin/maven-download-plugin/pull/191 was merged and https://github.com/maven-download-plugin/maven-download-plugin/issues/186 was closed, use a new release containing the option misnomer fix in order to make the POM more readable. Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
* Merge pull request #84 from kriegaex/test-issue-83Andy Clement2021-08-092-28/+31
|\ | | | | Stabilise flaky test in Ajc1612Tests
| * Remove logging dots per test step in AjcTest.runTestAlexander Kriegisch2021-08-081-2/+1
| | | | | | | | | | | | | | | | | | | | | | Methods AjcTest.runTest was logging something like "TEST: ...." (4 dots), i.e. one dot per performed test step. Not only did this not add much value, but due to usage of PrintStream.print for line continuation it also messed up test step logging by scattering seemingly random dot characters across test step logs. This looked quite ugly, so I simply removed it. Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * Refactor AnnoBinding test class used by Ajc1612Tests some moreAlexander Kriegisch2021-08-081-20/+22
| | | | | | | | | | | | | | Improve improved general logging, error messages, variable naming and indentation, making the code of class AnnoBinding a bit more readable. Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * Improve flaky test in Ajc1612Tests furtherAlexander Kriegisch2021-08-081-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | By using System.nanoTime() instead of currentTimeMillis(), the flakiness even with the original 10,000 rounds is significantly lower than before. Making my IDE repeat the test until failure, it took on average 150 runs to make it fail. So, the more accurate timing helps. With 100,000 rounds, it was even more stable, but eventually I could make it fail. With 1,000,000 rounds however, even running the test 500x could not make it fail. So for all practical purposes, I think the test is reasonably stable now. Closes #83. Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * Stabilise flaky test in Ajc1612TestsAlexander Kriegisch2021-08-051-2/+4
|/ | | | | | | | | | | By increasing from 10,000 to 1,000,000 rounds, the times compared for performance become considerably longer (but still in the tens or hundreds or milliseconds), decreasing the probability of the test failing due to CPU load or some other random effect. Closes #83. Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
* Merge pull request #82 from kriegaex/readme-dev-toolsAndy Clement2021-08-042-0/+99
|\ | | | | Add "Setting up an AspectJ development environment" guide
| * Add "Setting up an AspectJ development environment" guideAlexander Kriegisch2021-08-052-0/+99
|/ | | | Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
* Merge pull request #80 from kriegaex/release-1.9.8.M1Andy Clement2021-07-260-0/+0
|\ | | | | Release 1.9.8.M1 featuring `--release N` fix
| * Set version to 1.9.8-SNAPSHOTAlexander Kriegisch2021-07-2629-29/+29
| |
| * Set version to 1.9.8.M1V1_9_8_M1Alexander Kriegisch2021-07-2629-29/+29
|/
* Merge pull request #77 from SmallGiantGames/bug415838-testAndy Clement2021-06-302-1/+108
|\ | | | | Test for 1.9.6 concurrency bug in LocalVariableTable.unpack()
| * minor improvementsDmitry Mikhaylov2021-06-292-21/+29
| |
| * test for failing synchronization in LocalVariableTable.unpackDmitry Mikhaylov2021-06-282-5/+104
| |
* | Merge pull request #73 from kriegaex/ajc-release-optionAndy Clement2021-06-2815-10/+531
|\ \ | | | | | | Enable AJC compilation with `--release`
| * | Add integration test for '--release N' compiler optionAlexander Kriegisch2021-06-264-0/+101
| | | | | | | | | | | | | | | | | | Relates to #70 Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * | Add AspectJ 1.9.7 and Java 17 test skeletonsAlexander Kriegisch2021-06-2611-2/+409
| | | | | | | | | | | | Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * | AjBuildConfig: simplify null classpath removal from path listAlexander Kriegisch2021-06-261-7/+2
| | | | | | | | | | | | | | | | | | Use removeIf(Objects::isNull) instead of old-fashioned iterator loop. Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
| * | Quick fix for AJC '--release' optionAlexander Kriegisch2021-06-262-1/+19
|/ / | | | | | | | | | | Work in progress, relates to #70 Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>