aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | Merge pull request #27 from capgen628/patch-1Andy Clement2020-12-021-2/+1
|\ \ | |/ |/| Remove call to System.out
| * Remove call to System.outAnuj Dodhia2020-12-021-2/+1
|/
* Merge pull request #22 from jamesjer/masterAndy Clement2020-09-161-1/+1
|\ | | | | Fix apostrophe in the license
| * Fix apostrophe in the licenseJerry James2020-09-151-1/+1
|/
* Merge pull request #21 from jamesjer/masterAndy Clement2020-09-14108-300/+311
|\ | | | | Fix javadoc errors
| * Typo fixesJerry James2020-09-143-4/+4
| |
| * Fix javadoc references to nonexistent fields, classes, or packagesJerry James2020-09-1420-44/+33
| |
| * Fix misplaced or incorrect javadoc tagsJerry James2020-09-1443-89/+92
| |
| * Fix misplaced or incorrectly nested HTML tagsJerry James2020-09-1429-71/+90
| |
| * Fix incorrect HTML entities in javadoc commentsJerry James2020-09-1440-94/+94
|/
* Merge pull request #19 from jamesjer/masterAndy Clement2020-09-094-6/+6
|\ | | | | Give docbook IDs unique names
| * Give docbook IDs unique namesJerry James2020-09-094-6/+6
|/
* Revert to 1.9.7.BUILD-SNAPSHOTAndy Clement2020-08-2129-30/+30
|
* 1.9.7 milestone 1 publishedAndy Clement2020-08-2134-42/+77
|
* Merge pull request #18 from larsgrefer/feature/github-actionsV1_9_7M1Andy Clement2020-08-161-10/+18
|\ | | | | Split tests in a separate step on GitHub Actions
| * Split tests in a separate step on GitHub ActionsLars Grefer2020-08-171-10/+18
|/ | | | Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>
* debug for test failure under github actionAndy Clement2020-08-162-25/+32
|
* Update to JUnit Jupiter 5.6.2Lars Grefer2020-08-161-10/+32
| | | | | | 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/modifiersAndy Clement2020-08-1693-2550/+2550
|\ | | | | Cleanup unnecessary modifiers
| * Remove unnecessary interface modifiersLars Grefer2020-08-1791-2548/+2548
| | | | | | | | Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>
| * Remove unnecessary enum modifiersLars Grefer2020-08-172-2/+2
| | | | | | | | Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>
* | debug for test failure under github actionAndy Clement2020-08-161-0/+2
|/
* debug for test failure under github actionAndy Clement2020-08-161-10/+11
|
* polishAndy Clement2020-08-1615-253/+268
|
* Merge pull request #10 from larsgrefer/cleanup/pomAndy Clement2020-08-1624-31/+0
|\ | | | | Cleanup the Maven pom.xml files
| * Remove non-existing dependencyLars Grefer2020-08-151-8/+0
| | | | | | | | Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>
| * Remove project.parent.relative path as ../pom.xml is already the defaultLars Grefer2020-08-1523-23/+0
| | | | | | | | Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>
* | Reduce test output for incremental comp testsAndy Clement2020-08-155-50/+60
| |
* | Merge pull request #13 from larsgrefer/feature/collection-performanceAndy Clement2020-08-15128-491/+340
|\ \ | | | | | | Improve performance and readability of collection handling
| * \ Merge branch 'master' into feature/collection-performanceAndy Clement2020-08-15148-438/+361
| |\ \ | |/ / |/| |
* | | Merge pull request #15 from larsgrefer/update/maven/3.6.3Andy Clement2020-08-155-189/+242
|\ \ \ | | | | | | | | Update to Maven 3.6.3
| * | | Update to Maven 3.6.3Lars Grefer2020-08-165-189/+242
| | | | | | | | | | | | | | | | Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>
* | | | Merge pull request #12 from larsgrefer/cleanup/unused-codeAndy Clement2020-08-15135-155/+2
|\ \ \ \ | | | | | | | | | | Cleanup unused imports
| * | | | Cleanup unused importsLars Grefer2020-08-16135-155/+2
| |/ / / | | | | | | | | | | | | Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>
* | | | Merge pull request #11 from larsgrefer/feature/editorconfigAndy Clement2020-08-151-0/+20
|\ \ \ \ | |/ / / |/| | | Add a basic .editorconfig file
| * | | Add a basic .editorconfig fileLars Grefer2020-08-151-0/+20
| | | | | | | | | | | | | | | | Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>
* | | | Update to JUnit 4.13Lars Grefer2020-08-1510-35/+37
| | | | | | | | | | | | | | | | Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>
* | | | Merge pull request #7 from larsgrefer/patch-2Andy Clement2020-08-151-2/+2
|\ \ \ \ | | | | | | | | | | Use the Maven Wrapper for GitHub Actions
| * | | | Use the Maven Wrapper for GitHub ActionsLars Grefer2020-08-151-2/+2
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>
* | | | | Reduce test output for incremental comp testsAndy Clement2020-08-155-49/+32
| | | | |
* | | | | Fix test on J14Andy Clement2020-08-152-18/+28
| |_|_|/ |/| | |
| | | * Weaken Collection declarationsLars Grefer2020-08-1580-199/+193
| | | | | | | | | | | | | | | | | | | | | | | | 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>
| | | * Redundant Collection.addAll() callLars Grefer2020-08-1528-78/+41
| | | | | | | | | | | | | | | | | | | | | | | | 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 styleLars Grefer2020-08-1529-40/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 usedLars Grefer2020-08-154-30/+6
| | | | | | | | | | | | | | | | | | | | | | | | 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 copyLars Grefer2020-08-156-69/+29
| | | | | | | | | | | | | | | | | | | | | | | | 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()Lars Grefer2020-08-1512-17/+17
| | | | | | | | | | | | | | | | | | | | | | | | 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()Lars Grefer2020-08-151-1/+1
| | | | | | | | | | | | | | | | Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>
| | | * Manual array to collection copyLars Grefer2020-08-1511-72/+20
| |_|/ |/| | | | | | | | | | | | | | 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 properlyAndy Clement2020-08-141-126/+126
| | |