]> source.dussan.org Git - jgit.git/commitdiff
TreeWalk: Remove CorruptObjectException from addTree(AbstractTreeIterator) 79/64779/1
authorShawn Pearce <spearce@spearce.org>
Wed, 20 Jan 2016 15:41:08 +0000 (07:41 -0800)
committerShawn Pearce <spearce@spearce.org>
Wed, 20 Jan 2016 15:41:08 +0000 (07:41 -0800)
This form of addTree() does not parse any objects and cannot throw the
declared checked exception.  Callers are being forced to try-catch
CorruptObjectException that cannot occur when the iterator instance
has already been constructed.

Change-Id: Id338035302903bab81569d1576eab063eee0885a

org.eclipse.jgit/src/org/eclipse/jgit/treewalk/TreeWalk.java

index 83fada4f95404c0a6f045c71c0ffa074e3ae023f..5cd713da787f480dca3dc40f3a91db434c68112f 100644 (file)
@@ -574,18 +574,13 @@ public class TreeWalk implements AutoCloseable, AttributesProvider {
         * @param p
         *            an iterator to walk over. The iterator should be new, with no
         *            parent, and should still be positioned before the first entry.
-        *            The tree which the iterator operates on must have the same root
-        *            as other trees in the walk.
-        *
+        *            The tree which the iterator operates on must have the same
+        *            root as other trees in the walk.
         * @return position of this tree within the walker.
-        * @throws CorruptObjectException
-        *             the iterator was unable to obtain its first entry, due to
-        *             possible data corruption within the backing data store.
         */
-       public int addTree(final AbstractTreeIterator p)
-                       throws CorruptObjectException {
-               final int n = trees.length;
-               final AbstractTreeIterator[] newTrees = new AbstractTreeIterator[n + 1];
+       public int addTree(AbstractTreeIterator p) {
+               int n = trees.length;
+               AbstractTreeIterator[] newTrees = new AbstractTreeIterator[n + 1];
 
                System.arraycopy(trees, 0, newTrees, 0, n);
                newTrees[n] = p;