Make exception messages more verbose and user-friendly. Minor fixes in PdfException constants.

ars18wrw · ars18wrw · commit cfd6418d071f · 2018-03-30T16:43:11.000+03:00
DEVSIX-622
diff --git a/io/src/main/java/com/itextpdf/io/codec/Jbig2SegmentReader.java b/io/src/main/java/com/itextpdf/io/codec/Jbig2SegmentReader.java
@@ -247,7 +247,7 @@ void readSegment(Jbig2Segment s) throws java.io.IOException {
             ra.seek(last);
             Jbig2Page p = pages.get(s.page);
             if (p == null) {
-                throw new com.itextpdf.io.IOException("referring.to.widht.height.of.page.we.havent.seen.yet.1").setMessageParams(s.page);
+                throw new com.itextpdf.io.IOException("Referring to widht or height of a page we haven't seen yet: {0}").setMessageParams(s.page);
             }
 
             p.pageBitmapWidth = page_bitmap_width;
@@ -300,7 +300,7 @@ Jbig2Segment readHeader() throws java.io.IOException {
             }
 
         } else if (count_of_referred_to_segments == 5 || count_of_referred_to_segments == 6) {
-            throw new com.itextpdf.io.IOException("count.of.referred.to.segments.had.bad.value.in.header.for.segment.1.starting.at.2")
+            throw new com.itextpdf.io.IOException("Count of referred-to segments has forbidden value in the header for segment {0} starting at {1}")
                     .setMessageParams(segment_number, ptr);
 
         }
@@ -329,7 +329,7 @@ Jbig2Segment readHeader() throws java.io.IOException {
             segment_page_association = ra.read();
         }
         if (segment_page_association < 0) {
-            throw new com.itextpdf.io.IOException("page.1.invalid.for.segment.2.starting.at.3")
+            throw new com.itextpdf.io.IOException("Page {0} is invalid for segment {1} starting at {2}")
                     .setMessageParams(segment_page_association, segment_number, ptr);
         }
         s.page = segment_page_association;
@@ -369,7 +369,7 @@ void readFileHeader() throws java.io.IOException {
 
         for (int i = 0; i < idstring.length; i++) {
             if (idstring[i] != refidstring[i]) {
-                throw new com.itextpdf.io.IOException("file.header.idstring.not.good.at.byte.1").setMessageParams(i);
+                throw new com.itextpdf.io.IOException("File header idstring is not good at byte {0}").setMessageParams(i);
             }
         }
 
@@ -379,7 +379,7 @@ void readFileHeader() throws java.io.IOException {
         this.number_of_pages_known = (fileheaderflags & 0x2) == 0x0;
 
         if ((fileheaderflags & 0xfc) != 0x0) {
-            throw new com.itextpdf.io.IOException("file.header.flags.bits.2.7.not.0");
+            throw new com.itextpdf.io.IOException("File header flags bits from 2 to 7 should be 0, some not");
         }
 
         if (this.number_of_pages_known) {
diff --git a/io/src/main/java/com/itextpdf/io/font/CidFont.java b/io/src/main/java/com/itextpdf/io/font/CidFont.java
@@ -65,7 +65,7 @@ public class CidFont extends FontProgram {
         initializeCidFontNameAndStyle(fontName);
         Map<String, Object> fontDesc = CidFontProperties.getAllFonts().get(fontNames.getFontName());
         if (fontDesc == null) {
-            throw new IOException("no.such.predefined.font.1").setMessageParams(fontName);
+            throw new IOException("There is no such predefined font: {0}").setMessageParams(fontName);
         }
         initializeCidFontProperties(fontDesc);
     }
diff --git a/io/src/main/java/com/itextpdf/io/font/OpenTypeParser.java b/io/src/main/java/com/itextpdf/io/font/OpenTypeParser.java
@@ -393,27 +393,27 @@ private void initializeSfntTables() throws java.io.IOException {
             int dirIdx = ttcIndex;
             if (dirIdx < 0) {
                 if (fileName != null) {
-                    throw new IOException("the.font.index.for.1.must.be.positive").setMessageParams(fileName);
+                    throw new IOException("The font index for {0} must be positive.").setMessageParams(fileName);
                 } else {
-                    throw new IOException("the.font.index.must.be.positive");
+                    throw new IOException("The font index must be positive.");
                 }
             }
             String mainTag = readStandardString(4);
             if (!mainTag.equals("ttcf")) {
                 if (fileName != null) {
-                    throw new IOException("1.is.not.a.valid.ttc.file").setMessageParams(fileName);
+                    throw new IOException("{0} is not a valid ttc file.").setMessageParams(fileName);
                 } else {
-                    throw new IOException("not.a.valid.ttc.file");
+                    throw new IOException("Not a valid ttc file.");
                 }
             }
             raf.skipBytes(4);
             int dirCount = raf.readInt();
             if (dirIdx >= dirCount) {
                 if (fileName != null) {
-                    throw new IOException("the.font.index.for.1.must.be.between.0.and.2.it.was.3")
+                    throw new IOException("The font index for {0} must be between 0 and {1}. It is {2}.")
                             .setMessageParams(fileName, dirCount - 1, dirIdx);
                 } else {
-                    throw new IOException("the.font.index.must.be.between.0.and.1.it.was.2")
+                    throw new IOException("The font index must be between 0 and {0}. It is {1}.")
                             .setMessageParams(dirCount - 1, dirIdx);
                 }
             }
@@ -424,9 +424,9 @@ private void initializeSfntTables() throws java.io.IOException {
         int ttId = raf.readInt();
         if (ttId != 0x00010000 && ttId != 0x4F54544F) {
             if (fileName != null) {
-                throw new IOException("1.is.not.a.valid.ttf.or.otf.file").setMessageParams(fileName);
+                throw new IOException("{0} is not a valid ttf or otf file.").setMessageParams(fileName);
             } else {
-                throw new IOException("not.a.valid.ttf.or.otf.file");
+                throw new IOException("Not a valid ttf or otf file.");
             }
         }
         int num_tables = raf.readUnsignedShort();
diff --git a/io/src/main/java/com/itextpdf/io/font/Type1Font.java b/io/src/main/java/com/itextpdf/io/font/Type1Font.java
@@ -82,7 +82,7 @@ protected static Type1Font createStandardFont(String name) throws java.io.IOExce
         if (StandardFonts.isStandardFont(name)) {
             return new Type1Font(name, null, null, null);
         } else {
-            throw new IOException("1.is.not.a.standard.type1.font").setMessageParams(name);
+            throw new IOException("{0} is not a standard type1 font.").setMessageParams(name);
         }
     }
 
@@ -313,9 +313,9 @@ protected void process() throws java.io.IOException {
         if (!startKernPairs) {
             String metricsPath = fontParser.getAfmPath();
             if (metricsPath != null) {
-                throw new IOException("missing.startcharmetrics.in.1").setMessageParams(metricsPath);
+                throw new IOException("startcharmetrics is missing in {0}.").setMessageParams(metricsPath);
             } else {
-                throw new IOException("missing.startcharmetrics.in.the.metrics.file");
+                throw new IOException("startcharmetrics is missing in the metrics file.");
             }
         }
         avgWidth = 0;
@@ -378,9 +378,9 @@ protected void process() throws java.io.IOException {
         if (startKernPairs) {
             String metricsPath = fontParser.getAfmPath();
             if (metricsPath != null) {
-                throw new IOException("missing.endcharmetrics.in.1").setMessageParams(metricsPath);
+                throw new IOException("endcharmetrics is missing in {0}.").setMessageParams(metricsPath);
             } else {
-                throw new IOException("missing.endcharmetrics.in.the.metrics.file");
+                throw new IOException("endcharmetrics is missing in the metrics file.");
             }
         }
 
@@ -435,18 +435,18 @@ long record = ((long)firstUni << 32) + secondUni;
         } else if (!endOfMetrics) {
             String metricsPath = fontParser.getAfmPath();
             if (metricsPath != null) {
-                throw new IOException("missing.endfontmetrics.in.1").setMessageParams(metricsPath);
+                throw new IOException("endfontmetrics is missing in {0}.").setMessageParams(metricsPath);
             } else {
-                throw new IOException("missing.endfontmetrics.in.the.metrics.file");
+                throw new IOException("endfontmetrics is missing in the metrics file.");
             }
         }
 
         if (startKernPairs) {
             String metricsPath = fontParser.getAfmPath();
             if (metricsPath != null) {
-                throw new IOException("missing.endkernpairs.in.1").setMessageParams(metricsPath);
+                throw new IOException("endkernpairs is missing in {0}.").setMessageParams(metricsPath);
             } else {
-                throw new IOException("missing.endkernpairs.in.the.metrics.file");
+                throw new IOException("endkernpairs is missing in the metrics file.");
             }
         }
         raf.close();
diff --git a/io/src/main/java/com/itextpdf/io/font/Type1Parser.java b/io/src/main/java/com/itextpdf/io/font/Type1Parser.java
@@ -94,7 +94,7 @@ public RandomAccessFileOrArray getMetricsFile() throws java.io.IOException {
                 String resourcePath = FontResources.AFMS + afmPath + ".afm";
                 resource = ResourceUtil.getResourceStream(resourcePath);
                 if (resource == null) {
-                    throw new IOException("1.not.found.as.resource").setMessageParams(resourcePath);
+                    throw new IOException("{0} was not found as resource.").setMessageParams(resourcePath);
                 }
                 ByteArrayOutputStream stream = new ByteArrayOutputStream();
                 int read;
@@ -131,14 +131,14 @@ public RandomAccessFileOrArray getMetricsFile() throws java.io.IOException {
                 try {
                     Pfm2afm.convert(rf, ba);
                 } catch (Exception ignored) {
-                    throw new IOException("invalid.afm.or.pfm.font.file");
+                    throw new IOException("Invalid afm or pfm font file.");
                 } finally {
                     rf.close();
                 }
                 return new RandomAccessFileOrArray(sourceFactory.createSource(ba.toByteArray()));
             }
         } else {
-            throw new IOException("invalid.afm.or.pfm.font.file");
+            throw new IOException("Invalid afm or pfm font file.");
         }
     }
 
diff --git a/io/src/main/java/com/itextpdf/io/font/cmap/CMapByteCid.java b/io/src/main/java/com/itextpdf/io/font/cmap/CMapByteCid.java
@@ -120,7 +120,7 @@ private void encodeSequence(byte[] seq, char cid) {
             int one = seq[idx] & 0xff;
             char c = plane[one];
             if (c != 0 && (c & 0x8000) == 0)
-                throw new IOException("inconsistent.mapping");
+                throw new IOException("Inconsistent mapping.");
             if (c == 0) {
                 planes.add(new char[256]);
                 c = (char)(planes.size() - 1 | 0x8000);
@@ -132,7 +132,7 @@ private void encodeSequence(byte[] seq, char cid) {
         int one = seq[size] & 0xff;
         char c = plane[one];
         if ((c & 0x8000) != 0)
-            throw new IOException("inconsistent.mapping");
+            throw new IOException("Inconsistent mapping.");
         plane[one] = cid;
     }
 }
diff --git a/io/src/main/java/com/itextpdf/io/font/cmap/CMapContentParser.java b/io/src/main/java/com/itextpdf/io/font/cmap/CMapContentParser.java
@@ -101,13 +101,13 @@ public CMapObject readDictionary() throws java.io.IOException {
         Map<String, CMapObject> dic = new HashMap<>();
         while (true) {
             if (!nextValidToken())
-                throw new IOException("unexpected.end.of.file");
+                throw new IOException("Unexpected end of file.");
             if (tokeniser.getTokenType() == TokenType.EndDic)
                 break;
             if (tokeniser.getTokenType() == TokenType.Other && "def".equals(tokeniser.getStringValue()))
                 continue;
             if (tokeniser.getTokenType() != TokenType.Name)
-                throw new IOException("dictionary.key.1.is.not.a.name").setMessageParams(tokeniser.getStringValue());
+                throw new IOException("Dictionary key {0} is not a name.").setMessageParams(tokeniser.getStringValue());
             String name = tokeniser.getStringValue();
             CMapObject obj = readObject();
             if (obj.isToken()) {
diff --git a/kernel/src/main/java/com/itextpdf/kernel/PdfException.java b/kernel/src/main/java/com/itextpdf/kernel/PdfException.java
@@ -113,7 +113,9 @@ public class PdfException extends RuntimeException {
     public static final String CannotRemoveDocumentRootTag = "Cannot remove document root tag.";
     public static final String CannotRemoveMarkedContentReferenceBecauseItsPageWasAlreadyFlushed = "Cannot remove marked content reference, because its page has been already flushed.";
     public static final String CannotRemoveTagBecauseItsParentIsFlushed = "Cannot remove tag, because its parent is flushed.";
+    @Deprecated
     public static final String CannotSetDataToPdfstreamWhichWasCreatedByInputStream = "Cannot set data to PdfStream which was created by InputStream.";
+    public static final String CannotSetDataToPdfStreamWhichWasCreatedByInputStream = "Cannot set data to PdfStream which was created by InputStream.";
     public static final String CannotSetEncryptedPayloadToDocumentOpenedInReadingMode = "Cannot set encrypted payload to a document opened in read only mode.";
     public static final String CannotSetEncryptedPayloadToEncryptedDocument = "Cannot set encrypted payload to an encrypted document.";
     public static final String CannotSplitDocumentThatIsBeingWritten = "Cannot split document that is being written.";
diff --git a/kernel/src/main/java/com/itextpdf/kernel/colors/Color.java b/kernel/src/main/java/com/itextpdf/kernel/colors/Color.java
@@ -148,7 +148,7 @@ public static Color makeColor(PdfColorSpace colorSpace, float[] colorValue) {
             unknownColorSpace = true;
         }
         if (unknownColorSpace) {
-            throw new PdfException("unknown.color.space");
+            throw new PdfException("Unknown color space.");
         }
         return c;
     }
diff --git a/kernel/src/main/java/com/itextpdf/kernel/font/PdfTrueTypeFont.java b/kernel/src/main/java/com/itextpdf/kernel/font/PdfTrueTypeFont.java
@@ -71,7 +71,7 @@ public class PdfTrueTypeFont extends PdfSimpleFont<TrueTypeFont> {
         this.embedded = embedded;
         FontNames fontNames = ttf.getFontNames();
         if (embedded && !fontNames.allowEmbedding()) {
-            throw new PdfException("1.cannot.be.embedded.due.to.licensing.restrictions")
+            throw new PdfException("{0} cannot be embedded due to licensing restrictions.")
                     .setMessageParams(fontNames.getFontName());
         }
         if ((encoding == null || encoding.length() == 0) && ttf.isFontSpecific()) {
diff --git a/kernel/src/main/java/com/itextpdf/kernel/font/PdfType0Font.java b/kernel/src/main/java/com/itextpdf/kernel/font/PdfType0Font.java
@@ -138,7 +138,7 @@ public class PdfType0Font extends PdfFont {
     PdfType0Font(CidFont font, String cmap) {
         super();
         if (!CidFontProperties.isCidFont(font.getFontNames().getFontName(), cmap)) {
-            throw new PdfException("font.1.with.2.encoding.is.not.a.cjk.font")
+            throw new PdfException("Font {0} with {1} encoding is not a cjk font.")
                     .setMessageParams(font.getFontNames().getFontName(), cmap);
         }
         this.fontProgram = font;
@@ -391,7 +391,7 @@ public GlyphLine createGlyphLine(String content) {
                 }
             }
         } else {
-            throw new PdfException("font.has.no.suitable.cmap");
+            throw new PdfException("Font has no suitable cmap.");
         }
 
         return new GlyphLine(glyphs);
@@ -432,7 +432,7 @@ public int appendGlyphs(String text, int from, int to, List<Glyph> glyphs) {
                 return appendUniGlyphs(text, from, to, glyphs);
             }
         } else {
-            throw new PdfException("font.has.no.suitable.cmap");
+            throw new PdfException("Font has no suitable cmap.");
         }
     }
 
@@ -500,7 +500,7 @@ public int appendAnyGlyph(String text, int from, List<Glyph> glyphs) {
                 glyphs.add(getGlyph(ch));
             }
         } else {
-            throw new PdfException("font.has.no.suitable.cmap");
+            throw new PdfException("Font has no suitable cmap.");
         }
         return process;
     }
diff --git a/kernel/src/main/java/com/itextpdf/kernel/font/PdfType3Font.java b/kernel/src/main/java/com/itextpdf/kernel/font/PdfType3Font.java
@@ -306,7 +306,7 @@ public boolean containsGlyph(int unicode) {
     public void flush() {
         ensureUnderlyingObjectHasIndirectReference();
         if (((Type3Font) getFontProgram()).getNumberOfGlyphs() < 1) {
-            throw new PdfException("no.glyphs.defined.fo r.type3.font");
+            throw new PdfException("No glyphs defined for type3 font.");
         }
 
         PdfDictionary charProcs = new PdfDictionary();
diff --git a/kernel/src/main/java/com/itextpdf/kernel/font/Type3Glyph.java b/kernel/src/main/java/com/itextpdf/kernel/font/Type3Glyph.java
@@ -187,7 +187,7 @@ private void writeMetrics(float wx, float llx, float lly, float urx, float ury,
     @Override
     public PdfXObject addImage(ImageData image, float a, float b, float c, float d, float e, float f, boolean inlineImage) {
         if (!isColor && (!image.isMask() || !(image.getBpc() == 1 || image.getBpc() > 0xff))) {
-            throw new PdfException("not.colorized.typed3.fonts.only.accept.mask.images");
+            throw new PdfException("Not colorized type3 fonts accept only mask images.");
         }
         return super.addImage(image, a, b, c, d, e, f, inlineImage);
     }
diff --git a/kernel/src/main/java/com/itextpdf/kernel/pdf/PdfStream.java b/kernel/src/main/java/com/itextpdf/kernel/pdf/PdfStream.java
@@ -296,7 +296,7 @@ public void setData(byte[] bytes, boolean append) {
             throw new PdfException(PdfException.CannotOperateWithFlushedPdfStream);
         }
         if (inputStream != null) {
-            throw new PdfException(PdfException.CannotSetDataToPdfstreamWhichWasCreatedByInputStream);
+            throw new PdfException(PdfException.CannotSetDataToPdfStreamWhichWasCreatedByInputStream);
         }
 
         boolean outputStreamIsUninitialized = outputStream == null;
diff --git a/kernel/src/test/java/com/itextpdf/kernel/pdf/PdfFontTest.java b/kernel/src/test/java/com/itextpdf/kernel/pdf/PdfFontTest.java
@@ -1361,7 +1361,7 @@ public void createWrongAfm1() throws IOException, InterruptedException {
         } catch (com.itextpdf.io.IOException e) {
             message = e.getMessage();
         }
-        Assert.assertEquals("invalid.afm.or.pfm.font.file", message);
+        Assert.assertEquals("Invalid afm or pfm font file.", message);
     }
 
     @Test

Original file line number	Diff line number	Diff line change
`@@ -247,7 +247,7 @@ void readSegment(Jbig2Segment s) throws java.io.IOException {`
`247`	`247`	`ra.seek(last);`
`248`	`248`	`Jbig2Page p = pages.get(s.page);`
`249`	`249`	`if (p == null) {`
`250`		`- throw new com.itextpdf.io.IOException("referring.to.widht.height.of.page.we.havent.seen.yet.1").setMessageParams(s.page);`
	`250`	`+ throw new com.itextpdf.io.IOException("Referring to widht or height of a page we haven't seen yet: {0}").setMessageParams(s.page);`
`251`	`251`	`}`
`252`	`252`
`253`	`253`	`p.pageBitmapWidth = page_bitmap_width;`
`@@ -300,7 +300,7 @@ Jbig2Segment readHeader() throws java.io.IOException {`
`300`	`300`	`}`
`301`	`301`
`302`	`302`	`} else if (count_of_referred_to_segments == 5 \|\| count_of_referred_to_segments == 6) {`
`303`		`- throw new com.itextpdf.io.IOException("count.of.referred.to.segments.had.bad.value.in.header.for.segment.1.starting.at.2")`
	`303`	`+ throw new com.itextpdf.io.IOException("Count of referred-to segments has forbidden value in the header for segment {0} starting at {1}")`
`304`	`304`	`.setMessageParams(segment_number, ptr);`
`305`	`305`
`306`	`306`	`}`
`@@ -329,7 +329,7 @@ Jbig2Segment readHeader() throws java.io.IOException {`
`329`	`329`	`segment_page_association = ra.read();`
`330`	`330`	`}`
`331`	`331`	`if (segment_page_association < 0) {`
`332`		`- throw new com.itextpdf.io.IOException("page.1.invalid.for.segment.2.starting.at.3")`
	`332`	`+ throw new com.itextpdf.io.IOException("Page {0} is invalid for segment {1} starting at {2}")`
`333`	`333`	`.setMessageParams(segment_page_association, segment_number, ptr);`
`334`	`334`	`}`
`335`	`335`	`s.page = segment_page_association;`
`@@ -369,7 +369,7 @@ void readFileHeader() throws java.io.IOException {`
`369`	`369`
`370`	`370`	`for (int i = 0; i < idstring.length; i++) {`
`371`	`371`	`if (idstring[i] != refidstring[i]) {`
`372`		`- throw new com.itextpdf.io.IOException("file.header.idstring.not.good.at.byte.1").setMessageParams(i);`
	`372`	`+ throw new com.itextpdf.io.IOException("File header idstring is not good at byte {0}").setMessageParams(i);`
`373`	`373`	`}`
`374`	`374`	`}`
`375`	`375`
`@@ -379,7 +379,7 @@ void readFileHeader() throws java.io.IOException {`
`379`	`379`	`this.number_of_pages_known = (fileheaderflags & 0x2) == 0x0;`
`380`	`380`
`381`	`381`	`if ((fileheaderflags & 0xfc) != 0x0) {`
`382`		`- throw new com.itextpdf.io.IOException("file.header.flags.bits.2.7.not.0");`
	`382`	`+ throw new com.itextpdf.io.IOException("File header flags bits from 2 to 7 should be 0, some not");`
`383`	`383`	`}`
`384`	`384`
`385`	`385`	`if (this.number_of_pages_known) {`
Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,7 @@ public class CidFont extends FontProgram {`
`65`	`65`	`initializeCidFontNameAndStyle(fontName);`
`66`	`66`	`Map<String, Object> fontDesc = CidFontProperties.getAllFonts().get(fontNames.getFontName());`
`67`	`67`	`if (fontDesc == null) {`
`68`		`- throw new IOException("no.such.predefined.font.1").setMessageParams(fontName);`
	`68`	`+ throw new IOException("There is no such predefined font: {0}").setMessageParams(fontName);`
`69`	`69`	`}`
`70`	`70`	`initializeCidFontProperties(fontDesc);`
`71`	`71`	`}`
Original file line number	Diff line number	Diff line change
`@@ -82,7 +82,7 @@ protected static Type1Font createStandardFont(String name) throws java.io.IOExce`
`82`	`82`	`if (StandardFonts.isStandardFont(name)) {`
`83`	`83`	`return new Type1Font(name, null, null, null);`
`84`	`84`	`} else {`
`85`		`- throw new IOException("1.is.not.a.standard.type1.font").setMessageParams(name);`
	`85`	`+ throw new IOException("{0} is not a standard type1 font.").setMessageParams(name);`
`86`	`86`	`}`
`87`	`87`	`}`
`88`	`88`
`@@ -313,9 +313,9 @@ protected void process() throws java.io.IOException {`
`313`	`313`	`if (!startKernPairs) {`
`314`	`314`	`String metricsPath = fontParser.getAfmPath();`
`315`	`315`	`if (metricsPath != null) {`
`316`		`- throw new IOException("missing.startcharmetrics.in.1").setMessageParams(metricsPath);`
	`316`	`+ throw new IOException("startcharmetrics is missing in {0}.").setMessageParams(metricsPath);`
`317`	`317`	`} else {`
`318`		`- throw new IOException("missing.startcharmetrics.in.the.metrics.file");`
	`318`	`+ throw new IOException("startcharmetrics is missing in the metrics file.");`
`319`	`319`	`}`
`320`	`320`	`}`
`321`	`321`	`avgWidth = 0;`
`@@ -378,9 +378,9 @@ protected void process() throws java.io.IOException {`
`378`	`378`	`if (startKernPairs) {`
`379`	`379`	`String metricsPath = fontParser.getAfmPath();`
`380`	`380`	`if (metricsPath != null) {`
`381`		`- throw new IOException("missing.endcharmetrics.in.1").setMessageParams(metricsPath);`
	`381`	`+ throw new IOException("endcharmetrics is missing in {0}.").setMessageParams(metricsPath);`
`382`	`382`	`} else {`
`383`		`- throw new IOException("missing.endcharmetrics.in.the.metrics.file");`
	`383`	`+ throw new IOException("endcharmetrics is missing in the metrics file.");`
`384`	`384`	`}`
`385`	`385`	`}`
`386`	`386`
`@@ -435,18 +435,18 @@ long record = ((long)firstUni << 32) + secondUni;`
`435`	`435`	`} else if (!endOfMetrics) {`
`436`	`436`	`String metricsPath = fontParser.getAfmPath();`
`437`	`437`	`if (metricsPath != null) {`
`438`		`- throw new IOException("missing.endfontmetrics.in.1").setMessageParams(metricsPath);`
	`438`	`+ throw new IOException("endfontmetrics is missing in {0}.").setMessageParams(metricsPath);`
`439`	`439`	`} else {`
`440`		`- throw new IOException("missing.endfontmetrics.in.the.metrics.file");`
	`440`	`+ throw new IOException("endfontmetrics is missing in the metrics file.");`
`441`	`441`	`}`
`442`	`442`	`}`
`443`	`443`
`444`	`444`	`if (startKernPairs) {`
`445`	`445`	`String metricsPath = fontParser.getAfmPath();`
`446`	`446`	`if (metricsPath != null) {`
`447`		`- throw new IOException("missing.endkernpairs.in.1").setMessageParams(metricsPath);`
	`447`	`+ throw new IOException("endkernpairs is missing in {0}.").setMessageParams(metricsPath);`
`448`	`448`	`} else {`
`449`		`- throw new IOException("missing.endkernpairs.in.the.metrics.file");`
	`449`	`+ throw new IOException("endkernpairs is missing in the metrics file.");`
`450`	`450`	`}`
`451`	`451`	`}`
`452`	`452`	`raf.close();`
Original file line number	Diff line number	Diff line change
`@@ -94,7 +94,7 @@ public RandomAccessFileOrArray getMetricsFile() throws java.io.IOException {`
`94`	`94`	`String resourcePath = FontResources.AFMS + afmPath + ".afm";`
`95`	`95`	`resource = ResourceUtil.getResourceStream(resourcePath);`
`96`	`96`	`if (resource == null) {`
`97`		`- throw new IOException("1.not.found.as.resource").setMessageParams(resourcePath);`
	`97`	`+ throw new IOException("{0} was not found as resource.").setMessageParams(resourcePath);`
`98`	`98`	`}`
`99`	`99`	`ByteArrayOutputStream stream = new ByteArrayOutputStream();`
`100`	`100`	`int read;`
`@@ -131,14 +131,14 @@ public RandomAccessFileOrArray getMetricsFile() throws java.io.IOException {`
`131`	`131`	`try {`
`132`	`132`	`Pfm2afm.convert(rf, ba);`
`133`	`133`	`} catch (Exception ignored) {`
`134`		`- throw new IOException("invalid.afm.or.pfm.font.file");`
	`134`	`+ throw new IOException("Invalid afm or pfm font file.");`
`135`	`135`	`} finally {`
`136`	`136`	`rf.close();`
`137`	`137`	`}`
`138`	`138`	`return new RandomAccessFileOrArray(sourceFactory.createSource(ba.toByteArray()));`
`139`	`139`	`}`
`140`	`140`	`} else {`
`141`		`- throw new IOException("invalid.afm.or.pfm.font.file");`
	`141`	`+ throw new IOException("Invalid afm or pfm font file.");`
`142`	`142`	`}`
`143`	`143`	`}`
`144`	`144`
Original file line number	Diff line number	Diff line change
`@@ -148,7 +148,7 @@ public static Color makeColor(PdfColorSpace colorSpace, float[] colorValue) {`
`148`	`148`	`unknownColorSpace = true;`
`149`	`149`	`}`
`150`	`150`	`if (unknownColorSpace) {`
`151`		`- throw new PdfException("unknown.color.space");`
	`151`	`+ throw new PdfException("Unknown color space.");`
`152`	`152`	`}`
`153`	`153`	`return c;`
`154`	`154`	`}`
Original file line number	Diff line number	Diff line change
`@@ -71,7 +71,7 @@ public class PdfTrueTypeFont extends PdfSimpleFont<TrueTypeFont> {`
`71`	`71`	`this.embedded = embedded;`
`72`	`72`	`FontNames fontNames = ttf.getFontNames();`
`73`	`73`	`if (embedded && !fontNames.allowEmbedding()) {`
`74`		`- throw new PdfException("1.cannot.be.embedded.due.to.licensing.restrictions")`
	`74`	`+ throw new PdfException("{0} cannot be embedded due to licensing restrictions.")`
`75`	`75`	`.setMessageParams(fontNames.getFontName());`
`76`	`76`	`}`
`77`	`77`	`if ((encoding == null \|\| encoding.length() == 0) && ttf.isFontSpecific()) {`