From c58362c1748abceacf35fc6b3d0d53125f7cdf71 Mon Sep 17 00:00:00 2001 From: Jonathan Nieder Date: Tue, 9 Oct 2018 15:29:00 -0700 Subject: Avoid using @Nullable to annotate arrays As described in the javadoc for org.eclipse.jgit.annotations.Nullable: Warning: Please do not use this annotation on arrays. Different annotation processors treat `@Nullable Object[]` differently: some treat it as an array of nullable objects, for consistency with versions of `Nullable` defined with `@Target TYPE_USE`, while others treat it as a nullable array of objects. JGit therefore avoids using this annotation on arrays altogether. See the checker-framework manual[1] for details. [1] http://types.cs.washington.edu/checker-framework/current/checker-framework-manual.html#faq-array-syntax-meaning Change-Id: I14ffcf80adbb8145d797998de2f2fa6ab84c3ae3 Signed-off-by: Jonathan Nieder --- org.eclipse.jgit/src/org/eclipse/jgit/lib/Config.java | 2 -- org.eclipse.jgit/src/org/eclipse/jgit/lib/Repository.java | 1 - 2 files changed, 3 deletions(-) (limited to 'org.eclipse.jgit/src/org/eclipse/jgit/lib') diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/lib/Config.java b/org.eclipse.jgit/src/org/eclipse/jgit/lib/Config.java index 8558e3d0b2..4726975d07 100644 --- a/org.eclipse.jgit/src/org/eclipse/jgit/lib/Config.java +++ b/org.eclipse.jgit/src/org/eclipse/jgit/lib/Config.java @@ -62,7 +62,6 @@ import java.util.Set; import java.util.concurrent.TimeUnit; import java.util.concurrent.atomic.AtomicReference; -import org.eclipse.jgit.annotations.Nullable; import org.eclipse.jgit.errors.ConfigInvalidException; import org.eclipse.jgit.events.ConfigChangedEvent; import org.eclipse.jgit.events.ConfigChangedListener; @@ -1158,7 +1157,6 @@ public class Config { * if something went wrong while reading the config * @since 4.10 */ - @Nullable protected byte[] readIncludedConfig(String relPath) throws ConfigInvalidException { return null; diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/lib/Repository.java b/org.eclipse.jgit/src/org/eclipse/jgit/lib/Repository.java index 2a2699f906..77d268a3bd 100644 --- a/org.eclipse.jgit/src/org/eclipse/jgit/lib/Repository.java +++ b/org.eclipse.jgit/src/org/eclipse/jgit/lib/Repository.java @@ -1981,7 +1981,6 @@ public abstract class Repository implements AutoCloseable { * empty * @throws IOException */ - @Nullable private byte[] readGitDirectoryFile(String filename) throws IOException { File file = new File(getDirectory(), filename); try { -- cgit v1.2.3