Minor API fixes.
Xavier Ducrohet [Fri, 15 Mar 2013 20:27:49 +0000 (13:27 -0700)]
Change-Id: Ib0d2419d7c83a14dbb041ac3722f09e17cb89bd2

gradle/src/main/groovy/com/android/build/gradle/BasePlugin.groovy
gradle/src/main/groovy/com/android/build/gradle/internal/dependency/ManifestDependencyImpl.java
gradle/src/main/groovy/com/android/build/gradle/internal/dependency/SymbolFileProviderImpl.java

index a337811..2d7fddc 100644 (file)
@@ -14,8 +14,8 @@
  * limitations under the License.
  */
 package com.android.build.gradle
-
 import com.android.SdkConstants
+import com.android.annotations.NonNull
 import com.android.build.gradle.internal.ApplicationVariant
 import com.android.build.gradle.internal.BadPluginException
 import com.android.build.gradle.internal.LoggerWrapper
@@ -58,7 +58,6 @@ import com.android.builder.SourceProvider
 import com.android.builder.VariantConfiguration
 import com.android.builder.dependency.AndroidDependency
 import com.android.builder.dependency.JarDependency
-import com.android.builder.dependency.ManifestDependency
 import com.android.builder.signing.SigningConfig
 import com.android.utils.ILogger
 import com.google.common.collect.ArrayListMultimap
@@ -91,7 +90,6 @@ import static com.android.builder.BuilderConstants.INSTRUMENTATION_RESULTS
 import static com.android.builder.BuilderConstants.INSTRUMENTATION_TEST
 import static com.android.builder.BuilderConstants.INSTRUMENTATION_TESTS
 import static com.android.builder.BuilderConstants.REPORTS
-
 /**
  * Base class for all Android plugins
  */
@@ -1115,6 +1113,7 @@ public abstract class BasePlugin {
         return null
     }
 
+    @NonNull
     protected List<ManifestDependencyImpl> getManifestDependencies(
             List<AndroidDependency> libraries) {
 
@@ -1122,13 +1121,14 @@ public abstract class BasePlugin {
 
         for (AndroidDependency lib : libraries) {
             // get the dependencies
-            List<ManifestDependency> children = getManifestDependencies(lib.dependencies)
+            List<ManifestDependencyImpl> children = getManifestDependencies(lib.dependencies)
             list.add(new ManifestDependencyImpl(lib.manifest, children))
         }
 
         return list
     }
 
+    @NonNull
     protected static List<SymbolFileProviderImpl> getTextSymbolDependencies(
             List<AndroidDependency> libraries) {
 
index 0d19817..3af2ff9 100644 (file)
@@ -16,6 +16,7 @@
 
 package com.android.build.gradle.internal.dependency;
 
+import com.android.annotations.NonNull;
 import com.android.builder.dependency.ManifestDependency;
 import org.gradle.api.tasks.InputFile;
 import org.gradle.api.tasks.Nested;
@@ -28,27 +29,31 @@ import java.util.List;
  */
 public class ManifestDependencyImpl implements ManifestDependency{
 
-    private File manifest;
-    private List<? extends ManifestDependency> dependencies;
+    private final File manifest;
+    private final List<ManifestDependencyImpl> dependencies;
 
-    public ManifestDependencyImpl(File manifest, List<? extends ManifestDependency> dependencies) {
+    public ManifestDependencyImpl(@NonNull File manifest,
+                                  @NonNull List<ManifestDependencyImpl> dependencies) {
         this.manifest = manifest;
         this.dependencies = dependencies;
     }
 
     @InputFile
     @Override
+    @NonNull
     public File getManifest() {
         return manifest;
     }
 
     @Override
+    @NonNull
     public List<? extends ManifestDependency> getManifestDependencies() {
         return dependencies;
     }
 
     @Nested
+    @NonNull
     public List<ManifestDependencyImpl> getManifestDependenciesForInput() {
-        return (List<ManifestDependencyImpl>) dependencies;
+        return dependencies;
     }
 }
index 65334dc..8fa7bb3 100644 (file)
@@ -16,6 +16,7 @@
 
 package com.android.build.gradle.internal.dependency;
 
+import com.android.annotations.NonNull;
 import com.android.builder.dependency.SymbolFileProvider;
 import org.gradle.api.tasks.InputFile;
 
@@ -26,22 +27,24 @@ import java.io.File;
  */
 public class SymbolFileProviderImpl implements SymbolFileProvider {
 
-    private File manifest;
-    private File symbolFile;
+    private final File manifest;
+    private final File symbolFile;
 
-    public SymbolFileProviderImpl(File manifest, File symbolFile) {
+    public SymbolFileProviderImpl(@NonNull File manifest, @NonNull File symbolFile) {
         this.manifest = manifest;
         this.symbolFile = symbolFile;
     }
 
     @InputFile
     @Override
+    @NonNull
     public File getManifest() {
         return manifest;
     }
 
     @InputFile
     @Override
+    @NonNull
     public File getSymbolFile() {
         return symbolFile;
     }