From 65ca9cf84854e0547df23d30e4cd0d51c2fb2d4c Mon Sep 17 00:00:00 2001 From: "Vladislav.Soroka" Date: Mon, 1 Apr 2019 08:17:27 +0300 Subject: [PATCH] Gradle: EA-140773 - ISE: DefaultGradleExtensions.$$$reportNull$$$ --- .../gradle/model/DefaultGradleExtensions.java | 34 ++++++++++--------- 1 file changed, 18 insertions(+), 16 deletions(-) diff --git a/plugins/gradle/tooling-extension-impl/src/org/jetbrains/plugins/gradle/model/DefaultGradleExtensions.java b/plugins/gradle/tooling-extension-impl/src/org/jetbrains/plugins/gradle/model/DefaultGradleExtensions.java index b502a1e95df3..56d0ca6125ea 100644 --- a/plugins/gradle/tooling-extension-impl/src/org/jetbrains/plugins/gradle/model/DefaultGradleExtensions.java +++ b/plugins/gradle/tooling-extension-impl/src/org/jetbrains/plugins/gradle/model/DefaultGradleExtensions.java @@ -19,6 +19,7 @@ import org.jetbrains.annotations.NotNull; import org.jetbrains.annotations.Nullable; import java.util.ArrayList; +import java.util.Collections; import java.util.List; /** @@ -26,22 +27,23 @@ import java.util.List; */ public class DefaultGradleExtensions implements GradleExtensions { private static final long serialVersionUID = 1L; + private final List myExtensions; + private final List myConventions; + private final List myGradleProperties; + private final List myTasks; + private final List myConfigurations; private String myParentProjectPath; - @NotNull - private final List myExtensions = new ArrayList(); - @NotNull - private final List myConventions = new ArrayList(); - @NotNull - private final List myGradleProperties = new ArrayList(); - @NotNull - private final List myTasks = new ArrayList(); - - private final List myConfigurations = new ArrayList(); public DefaultGradleExtensions() { + myExtensions = new ArrayList(); + myConventions = new ArrayList(); + myGradleProperties = new ArrayList(); + myTasks = new ArrayList(); + myConfigurations = new ArrayList(); } public DefaultGradleExtensions(GradleExtensions extensions) { + this(); myParentProjectPath = extensions.getParentProjectPath(); for (GradleExtension extension : extensions.getExtensions()) { myExtensions.add(new DefaultGradleExtension(extension)); @@ -51,7 +53,7 @@ public class DefaultGradleExtensions implements GradleExtensions { } for (GradleProperty property : extensions.getGradleProperties()) { myGradleProperties.add(new DefaultGradleProperty(property)); - } + } for (ExternalTask entry : extensions.getTasks()) { myTasks.add(new DefaultExternalTask(entry)); } @@ -73,30 +75,30 @@ public class DefaultGradleExtensions implements GradleExtensions { @NotNull @Override public List getExtensions() { - return myExtensions; + return myExtensions == null ? Collections.emptyList() : myExtensions; } @Override @NotNull public List getConventions() { - return myConventions; + return myConventions == null ? Collections.emptyList() : myConventions; } @NotNull @Override public List getGradleProperties() { - return myGradleProperties; + return myGradleProperties == null ? Collections.emptyList() : myGradleProperties; } @NotNull @Override public List getTasks() { - return myTasks; + return myTasks == null ? Collections.emptyList() : myTasks; } @NotNull @Override public List getConfigurations() { - return myConfigurations; + return myConfigurations == null ? Collections.emptyList() : myConfigurations; } }