Browse Source

Merge "reftable: remove assert calls"

next
Jonathan Tan 5 years ago committed by Gerrit Code Review @ Eclipse.org
parent
commit
d4404fb438
  1. 4
      org.eclipse.jgit/src/org/eclipse/jgit/internal/storage/dfs/DfsReftableDatabase.java
  2. 5
      org.eclipse.jgit/src/org/eclipse/jgit/internal/storage/reftable/ReftableDatabase.java

4
org.eclipse.jgit/src/org/eclipse/jgit/internal/storage/dfs/DfsReftableDatabase.java

@ -163,7 +163,9 @@ public class DfsReftableDatabase extends DfsRefDatabase {
* if tables cannot be opened. * if tables cannot be opened.
*/ */
protected DfsReftableStack stack() throws IOException { protected DfsReftableStack stack() throws IOException {
assert getLock().isLocked(); if (!getLock().isLocked()) {
throw new IllegalStateException("most hold lock to access stack"); //$NON-NLS-1$
}
DfsObjDatabase odb = getRepository().getObjectDatabase(); DfsObjDatabase odb = getRepository().getObjectDatabase();
if (ctx == null) { if (ctx == null) {

5
org.eclipse.jgit/src/org/eclipse/jgit/internal/storage/reftable/ReftableDatabase.java

@ -138,7 +138,10 @@ public abstract class ReftableDatabase {
* on I/O problems * on I/O problems
*/ */
private Reftable reader() throws IOException { private Reftable reader() throws IOException {
assert lock.isLocked(); if (!lock.isLocked()) {
throw new IllegalStateException(
"must hold lock to access merged table"); //$NON-NLS-1$
}
if (mergedTables == null) { if (mergedTables == null) {
mergedTables = openMergedReftable(); mergedTables = openMergedReftable();
} }

Loading…
Cancel
Save