Merge branch 'master' into master

6 files changed, 103 insertions, 23 deletions

diff --git a/test/transform/resource/after-ecj/BuilderSingularToBuilderWithNull.java b/test/transform/resource/after-ecj/BuilderSingularToBuilderWithNull.java
new file mode 100644
index 00000000..7265e17a
--- /dev/null
+++ b/test/transform/resource/after-ecj/BuilderSingularToBuilderWithNull.java
@@ -0,0 +1,57 @@
+import lombok.Singular;
+@lombok.Builder(toBuilder = true) class BuilderSingularToBuilderWithNull {
+  public static @java.lang.SuppressWarnings("all") class BuilderSingularToBuilderWithNullBuilder {
+    private @java.lang.SuppressWarnings("all") java.util.ArrayList<String> elems;
+    @java.lang.SuppressWarnings("all") BuilderSingularToBuilderWithNullBuilder() {
+      super();
+    }
+    public @java.lang.SuppressWarnings("all") BuilderSingularToBuilderWithNullBuilder elem(String elem) {
+      if ((this.elems == null))
+          this.elems = new java.util.ArrayList<String>();
+      this.elems.add(elem);
+      return this;
+    }
+    public @java.lang.SuppressWarnings("all") BuilderSingularToBuilderWithNullBuilder elems(java.util.Collection<? extends String> elems) {
+      if ((this.elems == null))
+          this.elems = new java.util.ArrayList<String>();
+      this.elems.addAll(elems);
+      return this;
+    }
+    public @java.lang.SuppressWarnings("all") BuilderSingularToBuilderWithNullBuilder clearElems() {
+      if ((this.elems != null))
+          this.elems.clear();
+      return this;
+    }
+    public @java.lang.SuppressWarnings("all") BuilderSingularToBuilderWithNull build() {
+      java.util.List<String> elems;
+      switch (((this.elems == null) ? 0 : this.elems.size())) {
+      case 0 :
+          elems = java.util.Collections.emptyList();
+          break;
+      case 1 :
+          elems = java.util.Collections.singletonList(this.elems.get(0));
+          break;
+      default :
+          elems = java.util.Collections.unmodifiableList(new java.util.ArrayList<String>(this.elems));
+      }
+      return new BuilderSingularToBuilderWithNull(elems);
+    }
+    public @java.lang.Override @java.lang.SuppressWarnings("all") java.lang.String toString() {
+      return (("BuilderSingularToBuilderWithNull.BuilderSingularToBuilderWithNullBuilder(elems=" + this.elems) + ")");
+    }
+  }
+  private @Singular java.util.List<String> elems;
+  public static void test() {
+    new BuilderSingularToBuilderWithNull(null).toBuilder();
+  }
+  @java.lang.SuppressWarnings("all") BuilderSingularToBuilderWithNull(final java.util.List<String> elems) {
+    super();
+    this.elems = elems;
+  }
+  public static @java.lang.SuppressWarnings("all") BuilderSingularToBuilderWithNullBuilder builder() {
+    return new BuilderSingularToBuilderWithNullBuilder();
+  }
+  public @java.lang.SuppressWarnings("all") BuilderSingularToBuilderWithNullBuilder toBuilder() {
+    return new BuilderSingularToBuilderWithNullBuilder().elems(((this.elems == null) ? java.util.Collections.emptyList() : this.elems));
+  }
+}
diff --git a/test/transform/resource/after-ecj/BuilderWithToBuilder.java b/test/transform/resource/after-ecj/BuilderWithToBuilder.java
index d304293c..b9cc27dd 100644
--- a/test/transform/resource/after-ecj/BuilderWithToBuilder.java
+++ b/test/transform/resource/after-ecj/BuilderWithToBuilder.java
@@ -74,7 +74,7 @@ import lombok.Builder;
     return new BuilderWithToBuilderBuilder<T>();
   }
   public @java.lang.SuppressWarnings("all") BuilderWithToBuilderBuilder<T> toBuilder() {
-    return new BuilderWithToBuilderBuilder<T>().one(this.mOne).two(this.mTwo).foo(BuilderWithToBuilder.rrr(this)).bars(this.bars);
+    return new BuilderWithToBuilderBuilder<T>().one(this.mOne).two(this.mTwo).foo(BuilderWithToBuilder.rrr(this)).bars(((this.bars == null) ? java.util.Collections.emptyList() : this.bars));
   }
 }
 @lombok.experimental.Accessors(prefix = "m") class ConstructorWithToBuilder<T> {
diff --git a/test/transform/resource/after-ecj/FieldNameConstantsBasic.java b/test/transform/resource/after-ecj/FieldNameConstantsBasic.java
index f77203ba..674dd602 100644
--- a/test/transform/resource/after-ecj/FieldNameConstantsBasic.java
+++ b/test/transform/resource/after-ecj/FieldNameConstantsBasic.java
@@ -1,15 +1,23 @@
 import lombok.experimental.FieldNameConstants;
 import lombok.AccessLevel;
-public @FieldNameConstants class FieldNameConstantsBasic {
-  public static final java.lang.String FIELD_BUT_PRINT_ME_PLEASE = "butPrintMePlease";
-  protected static final java.lang.String FIELD_I_AM_A_DVD_PLAYER = "iAmADvdPlayer";
-  @FieldNameConstants(level = AccessLevel.PROTECTED) String iAmADvdPlayer;
+public @FieldNameConstants(level = AccessLevel.PACKAGE) class FieldNameConstantsBasic {
+  static final @java.lang.SuppressWarnings("all") class Fields {
+    public static final java.lang.String butPrintMePlease = "butPrintMePlease";
+    public static final java.lang.String iAmADvdPlayer = "iAmADvdPlayer";
+    <clinit>() {
+    }
+    private @java.lang.SuppressWarnings("all") Fields() {
+      super();
+    }
+  }
+  String iAmADvdPlayer;
   int $skipMe;
   static double skipMeToo;
+  @FieldNameConstants.Exclude int andMe;
   String butPrintMePlease;
   <clinit>() {
   }
   public FieldNameConstantsBasic() {
     super();
   }
-}
+}
\ No newline at end of file
diff --git a/test/transform/resource/after-ecj/FieldNameConstantsConfigKeys.java b/test/transform/resource/after-ecj/FieldNameConstantsConfigKeys.java
index 44629ee5..7eb97364 100644
--- a/test/transform/resource/after-ecj/FieldNameConstantsConfigKeys.java
+++ b/test/transform/resource/after-ecj/FieldNameConstantsConfigKeys.java
@@ -1,9 +1,14 @@
 public @lombok.experimental.FieldNameConstants class FieldNameConstantsConfigKeys {
-  public static final java.lang.String I_AM_A_DVD_PLAYER_SFX = "iAmADvdPlayer";
-  String iAmADvdPlayer;
-  <clinit>() {
+  public static final @java.lang.SuppressWarnings("all") class Foobar {
+    public static final java.lang.String iAmADvdPlayer = "iAmADvdPlayer";
+    <clinit>() {
+    }
+    private @java.lang.SuppressWarnings("all") Foobar() {
+      super();
+    }
   }
+  String iAmADvdPlayer;
   public FieldNameConstantsConfigKeys() {
     super();
   }
-}
+}
\ No newline at end of file
diff --git a/test/transform/resource/after-ecj/FieldNameConstantsEnum.java b/test/transform/resource/after-ecj/FieldNameConstantsEnum.java
new file mode 100644
index 00000000..053a6e7a
--- /dev/null
+++ b/test/transform/resource/after-ecj/FieldNameConstantsEnum.java
@@ -0,0 +1,23 @@
+import lombok.experimental.FieldNameConstants;
+import lombok.AccessLevel;
+public @FieldNameConstants(onlyExplicitlyIncluded = true,asEnum = true,innerTypeName = "TypeTest") class FieldNameConstantsEnum {
+  public @java.lang.SuppressWarnings("all") enum TypeTest {
+    iAmADvdPlayer(),
+    $dontSkipMe(),
+    alsoDontSkipMe(),
+    private @java.lang.SuppressWarnings("all") TypeTest() {
+      super();
+    }
+    <clinit>() {
+    }
+  }
+  @FieldNameConstants.Include String iAmADvdPlayer;
+  @FieldNameConstants.Include int $dontSkipMe;
+  static @FieldNameConstants.Include double alsoDontSkipMe;
+  int butSkipMe;
+  <clinit>() {
+  }
+  public FieldNameConstantsEnum() {
+    super();
+  }
+}
\ No newline at end of file
diff --git a/test/transform/resource/after-ecj/FieldNameConstantsWeird.java b/test/transform/resource/after-ecj/FieldNameConstantsWeird.java
deleted file mode 100644
index 9958f664..00000000
--- a/test/transform/resource/after-ecj/FieldNameConstantsWeird.java
+++ /dev/null
@@ -1,13 +0,0 @@
-import lombok.experimental.FieldNameConstants;
-import lombok.AccessLevel;
-public @FieldNameConstants class FieldNameConstantsWeird {
-  public static final java.lang.String FIELD_AZ = "A";
-  @FieldNameConstants(level = AccessLevel.NONE) String iAmADvdPlayer;
-  @FieldNameConstants(prefix = "") String X;
-  @FieldNameConstants(suffix = "Z") String A;
-  <clinit>() {
-  }
-  public FieldNameConstantsWeird() {
-    super();
-  }
-}