From 747cead7b112a792a98d916ff883337260bd9a39 Mon Sep 17 00:00:00 2001 From: Sebastian Sdorra Date: Sat, 29 Nov 2014 11:58:44 +0100 Subject: [PATCH] improve npe handling on constructor of ImportResult --- .../src/main/java/sonia/scm/repository/ImportResult.java | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/scm-core/src/main/java/sonia/scm/repository/ImportResult.java b/scm-core/src/main/java/sonia/scm/repository/ImportResult.java index 992ac33010..2911a57030 100644 --- a/scm-core/src/main/java/sonia/scm/repository/ImportResult.java +++ b/scm-core/src/main/java/sonia/scm/repository/ImportResult.java @@ -37,6 +37,8 @@ import com.google.common.base.Objects; import com.google.common.collect.ImmutableList; import com.google.common.collect.Lists; +import static com.google.common.base.Preconditions.*; + //~--- JDK imports ------------------------------------------------------------ import java.util.List; @@ -66,8 +68,10 @@ public final class ImportResult public ImportResult(List importedDirectories, List failedDirectories) { - this.importedDirectories = importedDirectories; - this.failedDirectories = failedDirectories; + this.importedDirectories = checkNotNull(importedDirectories, + "list of imported directories is required"); + this.failedDirectories = checkNotNull(failedDirectories, + "list of failed directories is required"); } //~--- methods --------------------------------------------------------------