You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ju...@apache.org on 2010/09/03 14:19:52 UTC

svn commit: r992275 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/common/PDPageLabels.java

Author: jukka
Date: Fri Sep  3 12:19:52 2010
New Revision: 992275

URL: http://svn.apache.org/viewvc?rev=992275&view=rev
Log:
PDFBOX-810: Problem in extracting roman page numbers

Fixed as suggested by Karthick G.

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/common/PDPageLabels.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/common/PDPageLabels.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/common/PDPageLabels.java?rev=992275&r1=992274&r2=992275&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/common/PDPageLabels.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/common/PDPageLabels.java Fri Sep  3 12:19:52 2010
@@ -321,11 +321,14 @@ public class PDPageLabels implements COS
             }
         }
 
-        private static final String[][] ROMANS = new String[][]
-        {
-        { "", "i", "ii", "iii", "iv", "v", "vi", "vii", "viii", "ix" },
-        { "x", "xx", "xxx", "xl", "l", "lx", "lxx", "lxxx", "xc" },
-        { "c", "cc", "ccc", "cd", "d", "dc", "dcc", "dccc", "cm" }, };
+        /**
+         * Lookup table used by the {@link #makeRomanLabel(int)} method.
+         */
+        private static final String[][] ROMANS = new String[][] {
+            { "", "i", "ii", "iii", "iv", "v", "vi", "vii", "viii", "ix" },
+            { "", "x", "xx", "xxx", "xl", "l", "lx", "lxx", "lxxx", "xc" },
+            { "", "c", "cc", "ccc", "cd", "d", "dc", "dcc", "dccc", "cm" }
+        };
 
         private static String makeRomanLabel(int pageIndex)
         {