diff options
author | David Pursehouse | 2017-03-10 06:57:36 +0000 |
---|---|---|
committer | Matthias Sohn | 2017-03-12 09:11:23 +0000 |
commit | 4ca05a1c7692cabb24f263e663f6dace6b9bda7b (patch) | |
tree | 25eb6d6ba0f601f4137f6108f76a3e7174ff7a0c | |
parent | 9dc02d0b641a4c2d2a06578acc8a32dfebddcef5 (diff) | |
download | egit-4ca05a1c7692cabb24f263e663f6dace6b9bda7b.tar.gz egit-4ca05a1c7692cabb24f263e663f6dace6b9bda7b.tar.xz egit-4ca05a1c7692cabb24f263e663f6dace6b9bda7b.zip |
RepositoriesViewContentProvider: Remove redundant null-checks
Repository#getWorkTree() is annotated @NonNull, so the null-check
on its return value is redundant.
Change-Id: Icb533322553f0ba5d7a0cadea0965cb824f5a421
Signed-off-by: David Pursehouse <david.pursehouse@gmail.com>
-rw-r--r-- | org.eclipse.egit.ui/src/org/eclipse/egit/ui/internal/repository/RepositoriesViewContentProvider.java | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/org.eclipse.egit.ui/src/org/eclipse/egit/ui/internal/repository/RepositoriesViewContentProvider.java b/org.eclipse.egit.ui/src/org/eclipse/egit/ui/internal/repository/RepositoriesViewContentProvider.java index 7b1e29d653..163d1151e1 100644 --- a/org.eclipse.egit.ui/src/org/eclipse/egit/ui/internal/repository/RepositoriesViewContentProvider.java +++ b/org.eclipse.egit.ui/src/org/eclipse/egit/ui/internal/repository/RepositoriesViewContentProvider.java @@ -327,7 +327,7 @@ public class RepositoriesViewContentProvider implements ITreeContentProvider, if (node.getRepository().isBare()) return children.toArray(); File workingDir = repo.getWorkTree(); - if (workingDir == null || !workingDir.exists()) + if (!workingDir.exists()) return children.toArray(); File[] childFiles = workingDir.listFiles(); @@ -568,7 +568,7 @@ public class RepositoriesViewContentProvider implements ITreeContentProvider, if (node.getRepository().isBare()) return false; File workingDir = repo.getWorkTree(); - if (workingDir == null || !workingDir.exists()) + if (!workingDir.exists()) return false; return workingDir.listFiles().length > 0; default: |