[trivial] adding a comment to give context to the fix for delombok being overly liberal with applying "var" to things.

This commit is contained in:
Reinier Zwitserloot committed 2019-04-24 14:12:28 +02:00
1 parent ac871308f1
commit 150be0a186
1 file changed
+2
@@ -442,6 +442,8 @@ public class JavacTreeMaker {
private static final MethodId<JCVariableDecl> VarDef = MethodId("VarDef");
public JCVariableDecl VarDef(JCModifiers mods, Name name, JCExpression vartype, JCExpression init) {
JCVariableDecl varDef = invoke(VarDef, mods, name, vartype, init);
// We use 'position of the type is -1' as indicator in delombok that the original node was written using JDK10's 'var' feature, because javac desugars 'var' to the real type and doesn't leave any markers other than the
// node position to indicate that it did so. Unfortunately, that means vardecls we generate look like 'var' to delombok. Adjust the position to avoid this.
if (varDef.vartype != null && varDef.vartype.pos == -1) varDef.vartype.pos = 0;
return varDef;
}