Reinier Zwitserloot
2eabd8485d
[delombok] [prettyprinter] add support for the compact record constructor
2021-03-23 04:58:20 +01:00
Reinier Zwitserloot
5e2a90836d
[javac] defer removal of import lombok.x; nodes until delombok
...
Our test harness setup for JDK15 and up doesn't work with annotations on record 'fields' (`record X(@RightHere String foo)`).
2021-03-22 11:42:09 +01:00
Reinier Zwitserloot
3400f66fa7
[deps] pushing ASM to 9.1, and ASM opcode compat levels to ASM9, as we need the support for records
2021-03-22 11:38:56 +01:00
Reinier Zwitserloot
bcd05a01b1
Merge branch 'records'
...
# Conflicts:
# src/core/lombok/eclipse/handlers/EclipseHandlerUtil.java
# src/core/lombok/eclipse/handlers/HandleBuilder.java
# src/core/lombok/eclipse/handlers/HandleData.java
# src/core/lombok/eclipse/handlers/HandleNonNull.java
# src/core/lombok/eclipse/handlers/HandleSuperBuilder.java
# src/core/lombok/javac/handlers/HandleBuilder.java
# src/core/lombok/javac/handlers/HandleNonNull.java
# src/core/lombok/javac/handlers/HandleSuperBuilder.java
# test/core/src/lombok/RunTestsViaEcj.java
2021-03-22 08:03:42 +01:00
Reinier Zwitserloot
ade900b82b
[testing] fixed an error (I probably introduced this in the typofix earlier) in a test file.
2021-03-21 20:24:40 +01:00
Reinier Zwitserloot
8e23c10a99
Merge pull request #2725 from md-5/master
...
[fixes #2724 ] Increase priority of Builder annotation removal
2021-03-21 05:24:46 +01:00
Reinier Zwitserloot
fb455f4634
[changelog] reflecting fixes #2701 and fixes #2704 in the changelog
2021-03-21 05:22:41 +01:00
Reinier Zwitserloot
2d2f67ea4a
[trivial] [typofix]
2021-03-21 05:19:20 +01:00
Reinier Zwitserloot
aeaa544a55
Merge pull request #2771 from janrieke/copyMoreJacksonAnnotations
...
Add more Jackson annotations to the copy-to-setter list
2021-03-21 05:12:42 +01:00
Reinier Zwitserloot
e9e16a355f
Merge pull request #2772 from janrieke/superBuilderNPEwithSingular
...
[fixes #2701 ] fix NPE on customized SuperBuilder with Singular
2021-03-21 05:12:12 +01:00
Reinier Zwitserloot
38faf6d350
Merge pull request #2770 from janrieke/superBuilderNPEinConstructorExists
...
[fixes #2704 ] SuperBuilder: avoid NPE on existing constructors
2021-03-21 05:09:13 +01:00
Reinier Zwitserloot
5ef5c77198
Merge pull request #2780 from md-5/delombok-16
...
[jdk16] Add lombok opens in single location
2021-03-21 04:59:49 +01:00
md_5
8f6d2d142c
[jdk16] Add lombok opens in single location
...
Fixes awhitford/lombok.maven#47
2021-03-20 14:11:21 +11:00
md_5
70951d4dd7
[ fixes #2724 ] Increase priority of Builder annotation removal
...
Otherwise they may run after `HandleDelegate`. This is problematic as
`HandleDelegate` parses the compulation unit and `@Builder` may contain imports
(eg `AccessLevel`) that were already removed by other processors, eg `@Getter`.
2021-03-20 13:18:50 +11:00
Reinier Zwitserloot
c6076730de
[ecj] [bug-workarounds]
...
The last comment in a line is lost if the array handed to the scanner is exactly sized to the input to process.
2021-03-19 05:07:23 +01:00
Roel Spilker
fccf57a2a5
fix test stub dependencies for ecj
2021-03-19 04:13:37 +01:00
Reinier Zwitserloot
d28027310f
[changelog]
2021-03-19 03:36:32 +01:00
Roel Spilker
334167bd22
[jdk16] Suddenly, Stream has a default method toList :-)
2021-03-19 03:35:59 +01:00
Reinier Zwitserloot
251b7541b7
[build] broke the distribution.
2021-03-19 03:28:31 +01:00
Reinier Zwitserloot
88f8cdb3a2
[build] remove some errors in eclipse.
...
The ant task we use for streaming test reporting in the ant script needs deps we don't wanna include in eclipse, so I moved it to a src dir that we don't include in eclipse by default.
2021-03-19 03:17:39 +01:00
Reinier Zwitserloot
937a4dfca2
[jdk16] fix delombok (and the test suite, which depends on it) for jdk16.
2021-03-19 03:17:39 +01:00
Reinier Zwitserloot
5241235b99
[testing] add a test.javac16
2021-03-19 03:17:39 +01:00
Reinier Zwitserloot
e69d0b143e
[testing] fix vm-finder on macs.
2021-03-19 03:17:38 +01:00
Reinier Zwitserloot
9498849d69
[testing] Fixing running older tests.
...
the new formatter is nice, but needed to be compile with 1.6, otherwise running junit on older VMs doesn't work.
2021-03-19 03:17:38 +01:00
Roel Spilker
9b211af8be
[jdk16] JCBindingPattern now has a field names var instead of separate fields
2021-03-19 03:08:10 +01:00
Roel Spilker
2ed5077cae
[jdk16] TypeSymbol.isLocal has been renamed
2021-03-19 02:48:16 +01:00
Reinier Zwitserloot
420d09f99d
[testing] restore the printing of the actual/expected when tests fail.
2021-03-19 02:38:58 +01:00
Reinier Zwitserloot
ccd6a4e895
[testing] made the test script a bit more consistent
2021-03-19 02:20:44 +01:00
Reinier Zwitserloot
dcb7efcc0e
[testing] improve the output of ant test.
2021-03-19 02:14:24 +01:00
Reinier Zwitserloot
0c6a243c8e
[build] quality-of-life improvements for lombok dev on windows
...
On windows, Eclipse keeps annotation processors open, which means putting spiProcessor in 'build' is annoying, as you can't delete open files on windows. Fixed by moving it to dist
2021-03-19 02:14:24 +01:00
Reinier Zwitserloot
27f3917d89
[testing] now that we've patched lombok to work in j16, no need for the --add-opens lines anymore.
2021-03-19 02:14:24 +01:00
Reinier Zwitserloot
6723525c4a
[testing] unbroke 'ant test.javac8'.
2021-03-19 02:14:24 +01:00
Roel Spilker
cbf930a9b8
[jdk16] forget optimization...
2021-03-19 02:02:36 +01:00
Roel Spilker
a5b7439516
[jdk16] delombok
2021-03-19 01:11:48 +01:00
Roel Spilker
76144935e2
[jdk16] docker
...
[jdk16] delombok
2021-03-19 00:54:25 +01:00
Reinier Zwitserloot
e18ae4328e
[testing] fixing a typo in a test file
2021-03-18 23:19:07 +01:00
Reinier Zwitserloot
15a28add9d
[build] the new spi processor didn't handle inner types used as SPI targets.
2021-03-18 23:18:54 +01:00
Reinier Zwitserloot
24c1a1ed54
[typo] oh dear, I broke the build again.
2021-03-18 21:33:59 +01:00
Reinier Zwitserloot
c04941f11c
[build] Further adjustments to the 'ant intellij' target.
...
As I don't use intellij, no idea if it works. Feedback welcome!
2021-03-17 12:50:28 +01:00
Reinier Zwitserloot
58693486f0
[build] Fixed the ant eclipse and ant intellij targets.
2021-03-17 12:46:34 +01:00
Reinier Zwitserloot
9806e5cca4
[ fixes #2681 ] [jdk16] support jdk16
2021-03-16 17:39:56 +01:00
Reinier Zwitserloot
476a43b970
[build] sped up the build considerable.
...
We used to use the mango SPI processor, but this had two major issues:
* ecj wouldn't run annotation processors, requiring a separate proc:only compile run with javac
* mangoSPI couldn't do incremental compilation, so this required a full rebuild just to generate SPI files, every time.
Addressed by updating to IPP40 which does support explicit annotation processors even for ecj, and
adding our own SPI generating annotation processor to the build which can deal with incrementals.
There are some limits; removing a `@Provides` annotation on an existing type will need a full clean to pick that up, for example.
These limits seem quite exotic, though.
2021-03-16 17:09:13 +01:00
Reinier Zwitserloot
a49aec582b
[testing] ant test.javac6 now passes. yay.
2021-03-14 03:48:05 +01:00
Reinier Zwitserloot
58ff1482e3
[testing] improve information given when position validator tests fail
2021-03-13 20:14:51 +01:00
Reinier Zwitserloot
fb10b185bd
[testing] also ignore unset positions for the pretty print tester.
2021-03-13 18:20:14 +01:00
Reinier Zwitserloot
c979ee9ddf
[testing] now easier to identify what failed, by name of test case
...
test case names now include idempotent/javac/ecj/prettyprint as a prefix to the test case filename.
2021-03-13 18:19:35 +01:00
Reinier Zwitserloot
e8cd744d3b
[superbuilder] toBuilder + singular + javac6 was broken
...
superbuilder generated the node `x.pluralName(plural == null ? Collections.emptyList() : plural)`.
java6 is not smart enough to infer the appropriate generics for the emptyList() call.
Fixed by always specifying the needed type explicitly: Collections.<String>emptyList();
2021-03-13 04:53:44 +01:00
Reinier Zwitserloot
c1bf4ec3b6
[trivial] just fixing line endings - no other changes
2021-03-13 04:31:51 +01:00
Reinier Zwitserloot
bf0de8b678
[testing] marking down all jackson related tests as j8+ only
...
The jackson dep we need for testing is too new a class file to run on j6, stubbing it out too much effort.
Odds we find a bug specifically because this stuff fails on java6/7 are low, and I'm okay with supporting
6/7 a little less well.
2021-03-13 04:25:24 +01:00
Reinier Zwitserloot
90ec4cb0d0
[testing] javac6 fails some of the idempotency tests because certain nodes aren't initialized with a position.
...
The odds that the 'check if positions of every node is set to something useful' check is not going to find any
actual bugs in the idempotency phase, so, just turn it off entirely there.
2021-03-13 04:23:22 +01:00