Remove duplicate calls to DirCache.unlock on checkout
Calls to unlock the DirCache before throwing an exception were not needed since checkout calls doCheckout wrapped in a try block that calls DirCache.unlock in a finally block. Change-Id: I2b249a784f9e363430e288aad67fcefb7fac0a6e Signed-off-by: Kevin Sawicki <kevin@github.com>
This commit is contained in:
parent
cc4e6109e4
commit
4005f3c693
|
@ -396,10 +396,9 @@ private boolean doCheckout() throws CorruptObjectException, IOException,
|
|||
prescanOneTree();
|
||||
|
||||
if (!conflicts.isEmpty()) {
|
||||
if (failOnConflict) {
|
||||
dc.unlock();
|
||||
if (failOnConflict)
|
||||
throw new CheckoutConflictException(conflicts.toArray(new String[conflicts.size()]));
|
||||
} else
|
||||
else
|
||||
cleanUpConflicts();
|
||||
}
|
||||
|
||||
|
@ -448,10 +447,8 @@ private boolean doCheckout() throws CorruptObjectException, IOException,
|
|||
}
|
||||
|
||||
// commit the index builder - a new index is persisted
|
||||
if (!builder.commit()) {
|
||||
dc.unlock();
|
||||
if (!builder.commit())
|
||||
throw new IndexWriteException();
|
||||
}
|
||||
} finally {
|
||||
objectReader.release();
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue