Skip to contentSkip to conversation

BlobBasedConfig now does not print stacktrace to stderr if blob is not found by path

PATCH New Constantine Plotnikov 1 participant 1 message Jun 15, 2016

lore
Share & tools
lore

Constantine Plotnikov

author
#0 lore raw

PATCH 1/1 BlobBasedConfig now does not print stacktrace to stderr if blob is not found by path

Prior to this patch, if the blob is not found inside commit by path, an exception stack trace was printed to stderr. This patch suppress this behavior by throwing FileNotFoundException instead of generic IOException in order to make it more aligned with FileBasedConfig.

Signed-off-by Constantine Plotnikov <[email protected]>
---
 .../src/org/spearce/jgit/lib/BlobBasedConfig.java  |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)
org.spearce.jgit/src/org/spearce/jgit/lib/BlobBasedConfig.java +2 -1
index 1f44994..8763c6c 100644
@@ -37,6 +37,7 @@
 package org.spearce.jgit.lib;
 
 import java.io.ByteArrayInputStream;
+import java.io.FileNotFoundException;
 import java.io.IOException;
 import java.io.InputStream;
 import java.util.concurrent.Callable;

@@ -118,7 +119,7 @@ this(base, new Callable<byte[]>() {
 				final Repository r = commit.getRepository();
 				final TreeWalk tree = TreeWalk.forPath(r, path, treeId);
 				if (tree == null) {
-					throw new IOException("Entry not found by path: " + path);
+					throw new FileNotFoundException("Entry not found by path: " + path);
 				}
 				ObjectId blobId = tree.getObjectId(0);
 				ObjectLoader loader = tree.getRepository().openBlob(blobId);
Signature · 1 line
1.6.1.2