minor updates and refactors

git-svn-id: https://svn.code.sf.net/p/jackcess/code/jackcess/trunk@812 f203690c-595d-4dc9-a70b-905162fa7fd2

1 files changed, 17 insertions, 16 deletions

diff --git a/src/main/java/com/healthmarketscience/jackcess/impl/TableImpl.java b/src/main/java/com/healthmarketscience/jackcess/impl/TableImpl.java
index 240ebf6..1d7e2e0 100644
--- a/src/main/java/com/healthmarketscience/jackcess/impl/TableImpl.java
+++ b/src/main/java/com/healthmarketscience/jackcess/impl/TableImpl.java
@@ -1792,28 +1792,29 @@ public class TableImpl implements Table
       TempPageHolder rowBufferH)
     throws IOException
   {
-      // find last data page (Not bothering to check other pages for free
-      // space.)
+    // find last data page (Not bothering to check other pages for free
+    // space.)
     UsageMap.PageCursor revPageCursor = ownedPages.cursor();
-      revPageCursor.afterLast();
-      while(true) {
-        int tmpPageNumber = revPageCursor.getPreviousPage();
-        if(tmpPageNumber < 0) {
-          break;
-        }
+    revPageCursor.afterLast();
+    while(true) {
+      int tmpPageNumber = revPageCursor.getPreviousPage();
+      if(tmpPageNumber < 0) {
+        break;
+      }
+      // only use if actually listed in free space pages
+      if(!freeSpacePages.containsPageNumber(tmpPageNumber)) {
+        continue;
+      }
       ByteBuffer dataPage = rowBufferH.setPage(ownedPages.getPageChannel(),
                                                tmpPageNumber);
-        if(dataPage.get() == PageTypes.DATA) {
-          // found last data page, only use if actually listed in free space
-          // pages
-        if(freeSpacePages.containsPageNumber(tmpPageNumber)) {
-          return dataPage;
-          }
-        }
+      if(dataPage.get() == PageTypes.DATA) {
+        // found last data page with free space
+        return dataPage;
       }
+    }
 
     return null;
-      }
+  }
     
   /**
    * Updates the table definition after rows are modified.