Starting work on tabulapdf#81

jazzido · jazzido · commit 7ce6e3607cc4 · 2016-06-30T11:42:33.000-04:00
diff --git a/src/main/java/technology/tabula/Ruling.java b/src/main/java/technology/tabula/Ruling.java
@@ -399,7 +399,8 @@ public static List<Ruling> collapseOrientedRulings(List<Ruling> lines, int expan
         if (lines.size() == 0) {
             return rv;
         }
-        Collections.sort(lines, new Comparator<Ruling>() {
+        
+        QuickSort.sort(lines, new Comparator<Ruling>() {
             @Override
             public int compare(Ruling a, Ruling b) {
                 return (int) (!Utils.feq(a.getPosition(), b.getPosition()) ? a.getPosition() - b.getPosition() : a.getStart() - b.getStart());
diff --git a/src/test/java/technology/tabula/TestSpreadsheetExtractor.java b/src/test/java/technology/tabula/TestSpreadsheetExtractor.java
@@ -576,7 +576,18 @@ public void testRealLifeRTL() throws IOException {
         // these (commented-out) tests reflect the theoretical correct answer,
         // which is not currently possible because of the two problems listed above
         // assertEquals("&#1605;&#1585;&#1581;&#1576;&#1575;&#1611;",                       table.getRows().get(0).get(0).getText()); // really ought to be &#1611;, but this is forgiveable for now
-
+    }
+    
+    @Test
+    public void testStableSort() throws IOException {
+    	Page page = UtilsForTesting.getPage("src/test/resources/technology/tabula/rulings_sort.pdf", 1);
+        SpreadsheetExtractionAlgorithm sea = new SpreadsheetExtractionAlgorithm();
+        List<Table> tables = (List<Table>) sea.extract(page);
+        // assertEquals(1, tables.size());
+        Table table = tables.get(0);
+        
+        System.out.println(table);
+        
     }
 
 }
diff --git a/src/test/resources/technology/tabula/rulings_sort.pdf b/src/test/resources/technology/tabula/rulings_sort.pdf
