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) {