From 217f91f13a6f12410894caa921b82553c4f20d31 Mon Sep 17 00:00:00 2001 From: Kakhnovich Raman Date: Fri, 7 Aug 2026 14:24:09 +0300 Subject: [PATCH 1/4] Remove Exceptions stack trace from logs --- src/main/java/org/verapdf/cos/COSArray.java | 2 +- src/main/java/org/verapdf/cos/COSDocument.java | 8 ++++---- src/main/java/org/verapdf/cos/COSStream.java | 6 +++--- .../java/org/verapdf/cos/filters/COSFilterRegistry.java | 2 +- src/main/java/org/verapdf/cos/visitor/Writer.java | 2 +- src/main/java/org/verapdf/external/ICCProfile.java | 4 ++-- src/main/java/org/verapdf/external/JPEG2000.java | 2 +- src/main/java/org/verapdf/io/Reader.java | 2 +- src/main/java/org/verapdf/parser/PDFParser.java | 6 +++--- src/main/java/org/verapdf/pd/PDDocument.java | 2 +- src/main/java/org/verapdf/pd/colors/PDIndexed.java | 2 +- .../verapdf/pd/encryption/StandardSecurityHandler.java | 2 +- src/main/java/org/verapdf/pd/font/PDCIDFont.java | 4 ++-- src/main/java/org/verapdf/pd/font/PDFont.java | 2 +- src/main/java/org/verapdf/pd/font/cmap/CMap.java | 2 +- src/main/java/org/verapdf/pd/font/cmap/CMapFactory.java | 2 +- src/main/java/org/verapdf/pd/font/cmap/CMapFile.java | 2 +- src/main/java/org/verapdf/pd/font/cmap/PDCMap.java | 6 +++--- .../pd/font/stdmetrics/StandardFontMetricsFactory.java | 4 ++-- .../java/org/verapdf/pd/font/truetype/AdobeGlyphList.java | 2 +- .../java/org/verapdf/pd/font/truetype/PDTrueTypeFont.java | 2 +- src/main/java/org/verapdf/pd/font/type1/PDType1Font.java | 4 ++-- src/main/java/org/verapdf/pd/font/type3/PDType3Font.java | 2 +- .../java/org/verapdf/pd/function/PDType0Function.java | 4 ++-- src/main/java/org/verapdf/pd/images/PDXImage.java | 2 +- src/main/java/org/verapdf/tools/StaticResources.java | 2 +- 26 files changed, 40 insertions(+), 40 deletions(-) diff --git a/src/main/java/org/verapdf/cos/COSArray.java b/src/main/java/org/verapdf/cos/COSArray.java index 3e8cf1e2..75063a9d 100644 --- a/src/main/java/org/verapdf/cos/COSArray.java +++ b/src/main/java/org/verapdf/cos/COSArray.java @@ -297,7 +297,7 @@ public ASInputStream getData(final COSStream.FilterFlags flags) { try { stream.close(); } catch (IOException e) { - LOGGER.log(Level.SEVERE, "Error in closing stream", e); + LOGGER.log(Level.SEVERE, "Error in closing stream"); } } throw any; diff --git a/src/main/java/org/verapdf/cos/COSDocument.java b/src/main/java/org/verapdf/cos/COSDocument.java index b5e22df6..f373e0a2 100644 --- a/src/main/java/org/verapdf/cos/COSDocument.java +++ b/src/main/java/org/verapdf/cos/COSDocument.java @@ -150,7 +150,7 @@ public List getObjects() { result.add(newObj); } catch (IOException e) { LOGGER.log(Level.FINE, "Error while parsing object : " + key.getNumber() + - ' ' + key.getGeneration(), e); + ' ' + key.getGeneration()); } catch (StackOverflowError e) { // TODO: double check this StackOverflow catching throw new LoopedException("Loop in getting object from reader", e); @@ -174,7 +174,7 @@ public List getObjectsByType(ASAtom type) { addObjectWithTypeKeyCheck(result, obj, type); } catch (IOException e) { LOGGER.log(Level.FINE, "Error while parsing object : " + key.getNumber() + - ' ' + key.getGeneration(), e); + ' ' + key.getGeneration()); } } } @@ -205,7 +205,7 @@ public Map getObjectsMap() { result.put(key, newObj); } catch (IOException e) { LOGGER.log(Level.FINE, "Error while parsing object : " + key.getNumber() + - ' ' + key.getGeneration(), e); + ' ' + key.getGeneration()); } } } @@ -357,7 +357,7 @@ public void saveTo(final OutputStream stream) { writeInputIntoOutput(pdf, stream); pdf.close(); } catch (IOException e) { - LOGGER.log(Level.FINE, "Can't write COSDocument to stream", e); + LOGGER.log(Level.FINE, "Can't write COSDocument to stream"); } finally { if (temp != null && !temp.delete()) { temp.deleteOnExit(); diff --git a/src/main/java/org/verapdf/cos/COSStream.java b/src/main/java/org/verapdf/cos/COSStream.java index eba99c69..ee0e8b38 100644 --- a/src/main/java/org/verapdf/cos/COSStream.java +++ b/src/main/java/org/verapdf/cos/COSStream.java @@ -166,7 +166,7 @@ public ASInputStream getData(final FilterFlags filterFlags) { result.reset(); return result; } catch (IOException e) { - LOGGER.log(Level.FINE, "Can't get stream data", e); + LOGGER.log(Level.FINE, "Can't get stream data"); return new ASInFilter(this.stream) { @Override public int read() { @@ -188,7 +188,7 @@ public boolean setData(final ASInputStream stream) { File encodedDataFile = fileWithData.getFile(); return setData(new InternalInputStream(encodedDataFile, true), FilterFlags.RAW_DATA); } catch (IOException e) { - LOGGER.log(Level.FINE, "Can not set data", e); + LOGGER.log(Level.FINE, "Can not set data"); return false; } } @@ -419,7 +419,7 @@ boolean equals(Object obj, List checkedObjects) { return false; } } catch (IOException e) { - LOGGER.log(Level.FINE, "Exception during comparing streams", e); + LOGGER.log(Level.FINE, "Exception during comparing streams"); return false; } return flags == that.flags; diff --git a/src/main/java/org/verapdf/cos/filters/COSFilterRegistry.java b/src/main/java/org/verapdf/cos/filters/COSFilterRegistry.java index 9a6cf63e..2b954d73 100644 --- a/src/main/java/org/verapdf/cos/filters/COSFilterRegistry.java +++ b/src/main/java/org/verapdf/cos/filters/COSFilterRegistry.java @@ -56,7 +56,7 @@ public class COSFilterRegistry { registerFactory(ASAtom.LZW_DECODE, new ASFilterFactory(ASAtom.LZW_DECODE)); registerFactory(ASAtom.RUN_LENGTH_DECODE, new ASFilterFactory(ASAtom.RUN_LENGTH_DECODE)); } catch (VeraPDFParserException e) { - LOGGER.log(Level.FINE, "Trying to register factory twice", e); + LOGGER.log(Level.FINE, "Trying to register factory twice"); } } diff --git a/src/main/java/org/verapdf/cos/visitor/Writer.java b/src/main/java/org/verapdf/cos/visitor/Writer.java index ef6e3528..8ac9bb03 100644 --- a/src/main/java/org/verapdf/cos/visitor/Writer.java +++ b/src/main/java/org/verapdf/cos/visitor/Writer.java @@ -208,7 +208,7 @@ public void visitFromStream(COSStream obj) { try { obj.setIntegerKey(ASAtom.LENGTH, getASInputStreamLength(in)); } catch (IOException e) { - LOGGER.log(Level.FINE, "Can't calculate length of ASInputStream", e); + LOGGER.log(Level.FINE, "Can't calculate length of ASInputStream"); } visitFromDictionary(obj); diff --git a/src/main/java/org/verapdf/external/ICCProfile.java b/src/main/java/org/verapdf/external/ICCProfile.java index c21fbd7d..47935069 100644 --- a/src/main/java/org/verapdf/external/ICCProfile.java +++ b/src/main/java/org/verapdf/external/ICCProfile.java @@ -118,7 +118,7 @@ private void initializeProfileHeader() { parseTags(data); } catch (IOException e) { this.isLooksValid = false; - LOGGER.log(Level.FINE, "Exception during obtaining ICCProfile header", e); + LOGGER.log(Level.FINE, "Exception during obtaining ICCProfile header"); } } @@ -142,7 +142,7 @@ public String getMD5() { md5.update(buffer); md5ByteValue = md5.digest(); } catch (NoSuchAlgorithmException | IOException e) { - LOGGER.log(Level.FINE, "Exception during calculating ICCProfile md5 value", e); + LOGGER.log(Level.FINE, "Exception during calculating ICCProfile md5 value"); md5ByteValue = new byte[0]; return null; } diff --git a/src/main/java/org/verapdf/external/JPEG2000.java b/src/main/java/org/verapdf/external/JPEG2000.java index 361092a4..7fedaecc 100644 --- a/src/main/java/org/verapdf/external/JPEG2000.java +++ b/src/main/java/org/verapdf/external/JPEG2000.java @@ -98,7 +98,7 @@ public static JPEG2000 fromStream(ASInputStream stream) { } } catch (IOException e) { - LOGGER.log(Level.FINE, "IO Exception reading JP2K stream", e); + LOGGER.log(Level.FINE, "IO Exception reading JP2K stream"); } return builder.build(); } diff --git a/src/main/java/org/verapdf/io/Reader.java b/src/main/java/org/verapdf/io/Reader.java index fc002fab..1905e495 100644 --- a/src/main/java/org/verapdf/io/Reader.java +++ b/src/main/java/org/verapdf/io/Reader.java @@ -200,7 +200,7 @@ private boolean docCanBeDecrypted() { } return res; } catch (IOException e) { - LOGGER.log(Level.FINE, "Cannot read object " + this.parser.getEncryption().getObjectKey(), e); + LOGGER.log(Level.FINE, "Cannot read object " + this.parser.getEncryption().getObjectKey()); return false; } } diff --git a/src/main/java/org/verapdf/parser/PDFParser.java b/src/main/java/org/verapdf/parser/PDFParser.java index f5b54e18..e90e186b 100644 --- a/src/main/java/org/verapdf/parser/PDFParser.java +++ b/src/main/java/org/verapdf/parser/PDFParser.java @@ -151,7 +151,7 @@ private COSHeader parseHeader() throws IOException { headerVersion = Float.parseFloat(headerParts[1]); } } catch (NumberFormatException e) { - LOGGER.log(Level.FINE, getErrorMessage("Can't parse the document header"), e); + LOGGER.log(Level.FINE, getErrorMessage("Can't parse the document header")); } result.setVersion(headerVersion); @@ -173,7 +173,7 @@ public boolean isLinearized() { } } } catch (IOException e) { - LOGGER.log(Level.WARNING, "IO error while trying to find first document dictionary", e); + LOGGER.log(Level.WARNING, "IO error while trying to find first document dictionary"); } return false; @@ -186,7 +186,7 @@ public COSObject getLinearizationDictionary() { return linDict; } } catch (IOException e) { - LOGGER.log(Level.WARNING, "IO error while trying to find linearization dictionary", e); + LOGGER.log(Level.WARNING, "IO error while trying to find linearization dictionary"); } return null; } diff --git a/src/main/java/org/verapdf/pd/PDDocument.java b/src/main/java/org/verapdf/pd/PDDocument.java index 8bf5ed99..a510d99e 100644 --- a/src/main/java/org/verapdf/pd/PDDocument.java +++ b/src/main/java/org/verapdf/pd/PDDocument.java @@ -110,7 +110,7 @@ public void close() { } document.getResourceHandler().close(); } catch (IOException e) { - LOGGER.log(Level.FINE, "Error in closing stream", e); + LOGGER.log(Level.FINE, "Error in closing stream"); } document = null; } diff --git a/src/main/java/org/verapdf/pd/colors/PDIndexed.java b/src/main/java/org/verapdf/pd/colors/PDIndexed.java index aa731064..73c7dca3 100644 --- a/src/main/java/org/verapdf/pd/colors/PDIndexed.java +++ b/src/main/java/org/verapdf/pd/colors/PDIndexed.java @@ -111,7 +111,7 @@ public double[] toRGB(double[] value) { } } } catch (IOException e) { - LOGGER.log(Level.WARNING, "Error reading color table for PDIndexed", e); + LOGGER.log(Level.WARNING, "Error reading color table for PDIndexed"); } return getBaseColorSpace().toRGB(colorTable[index]); diff --git a/src/main/java/org/verapdf/pd/encryption/StandardSecurityHandler.java b/src/main/java/org/verapdf/pd/encryption/StandardSecurityHandler.java index 39b202dd..ff0fd337 100644 --- a/src/main/java/org/verapdf/pd/encryption/StandardSecurityHandler.java +++ b/src/main/java/org/verapdf/pd/encryption/StandardSecurityHandler.java @@ -105,7 +105,7 @@ public boolean authenticatePassword(String password) { this.isPasswordCorrect = this.encryptionKey != null; return this.isPasswordCorrect; } catch (GeneralSecurityException e) { - LOGGER.log(Level.FINE, "Caught Security Exception while document decryption", e); + LOGGER.log(Level.FINE, "Caught Security Exception while document decryption"); this.isPasswordCorrect = false; return false; } diff --git a/src/main/java/org/verapdf/pd/font/PDCIDFont.java b/src/main/java/org/verapdf/pd/font/PDCIDFont.java index b076032b..19eb5c0a 100644 --- a/src/main/java/org/verapdf/pd/font/PDCIDFont.java +++ b/src/main/java/org/verapdf/pd/font/PDCIDFont.java @@ -217,7 +217,7 @@ public FontProgram getFontProgram() { this.fontProgram = new CIDFontType2Program(fontData, this.cMap, cidToGIDMap, key); StaticResources.cacheFontProgram(fontProgramID, this.fontProgram); } catch (IOException e) { - LOGGER.log(Level.FINE, "Can't read TrueType font program.", e); + LOGGER.log(Level.FINE, "Can't read TrueType font program."); } } } else if (fontDescriptor.canParseFontFile(ASAtom.FONT_FILE3)) { @@ -256,7 +256,7 @@ public FontProgram getFontProgram() { LOGGER.warning("Invalid subtype of the embedded font stream"); } } catch (Exception e) { - LOGGER.log(Level.FINE, "Can't read font program.", e); + LOGGER.log(Level.FINE, "Can't read font program."); } } else { this.fontProgram = null; diff --git a/src/main/java/org/verapdf/pd/font/PDFont.java b/src/main/java/org/verapdf/pd/font/PDFont.java index 131b6214..4a82d297 100644 --- a/src/main/java/org/verapdf/pd/font/PDFont.java +++ b/src/main/java/org/verapdf/pd/font/PDFont.java @@ -298,7 +298,7 @@ public boolean parseFontProgram() { setSuccessfullyParsed(fontProgram.isSuccessfulParsing()); return fontProgram.isSuccessfulParsing(); } catch (IOException e) { - LOGGER.log(Level.WARNING, "Can't parse font program of font " + getName(), e); + LOGGER.log(Level.WARNING, "Can't parse font program of font " + getName()); setSuccessfullyParsed(false); } } diff --git a/src/main/java/org/verapdf/pd/font/cmap/CMap.java b/src/main/java/org/verapdf/pd/font/cmap/CMap.java index 56dcdaa8..ad5c6d6e 100644 --- a/src/main/java/org/verapdf/pd/font/cmap/CMap.java +++ b/src/main/java/org/verapdf/pd/font/cmap/CMap.java @@ -147,7 +147,7 @@ public int getCodeFromStream(InputStream stream) throws IOException { } } } catch (VeraPDFParserException e) { - LOGGER.log(Level.WARNING, "CMap " + this.name + " has invalid codespace range.", e); + LOGGER.log(Level.WARNING, "CMap " + this.name + " has invalid codespace range."); } if (shortestMatchingCodeSpaceLength == Integer.MAX_VALUE && !codeSpaces.isEmpty()) { byte[] tmp = new byte[previousShortestMatchingCodeSpaceLength - i - 1]; diff --git a/src/main/java/org/verapdf/pd/font/cmap/CMapFactory.java b/src/main/java/org/verapdf/pd/font/cmap/CMapFactory.java index 3b4d934a..8594c922 100644 --- a/src/main/java/org/verapdf/pd/font/cmap/CMapFactory.java +++ b/src/main/java/org/verapdf/pd/font/cmap/CMapFactory.java @@ -60,7 +60,7 @@ static CMap getCMap(String name, ASInputStream cMapStream) { parser.parse(); res = parser.getCMap(); } catch (IOException e) { - LOGGER.log(Level.WARNING, "Can't parse CMap " + name + ", using default", e); + LOGGER.log(Level.WARNING, "Can't parse CMap " + name + ", using default"); res = new CMap(); } catch (PostScriptException e) { LOGGER.log(Level.WARNING, "PostScript exception while parsing CMap " + name); diff --git a/src/main/java/org/verapdf/pd/font/cmap/CMapFile.java b/src/main/java/org/verapdf/pd/font/cmap/CMapFile.java index 94090835..78c49144 100644 --- a/src/main/java/org/verapdf/pd/font/cmap/CMapFile.java +++ b/src/main/java/org/verapdf/pd/font/cmap/CMapFile.java @@ -87,7 +87,7 @@ private void parseCMapFile() { try (ASInputStream data = this.parentStream.getData(COSStream.FilterFlags.DECODE)) { cMap = CMapFactory.getCMap(PDCMap.getCMapID(parentStream), data); } catch (IOException e) { - LOGGER.log(Level.WARNING, "Exception while parsing cmap file", e); + LOGGER.log(Level.WARNING, "Exception while parsing cmap file"); } } } diff --git a/src/main/java/org/verapdf/pd/font/cmap/PDCMap.java b/src/main/java/org/verapdf/pd/font/cmap/PDCMap.java index 2f6a13f8..eef4c94a 100644 --- a/src/main/java/org/verapdf/pd/font/cmap/PDCMap.java +++ b/src/main/java/org/verapdf/pd/font/cmap/PDCMap.java @@ -118,7 +118,7 @@ private CMap getCMapFile(Set keys) { try (ASInputStream cMapStream = this.getObject().getData(COSStream.FilterFlags.DECODE)) { this.cMapFile = CMapFactory.getCMap(getCMapID(), cMapStream); } catch (IOException e) { - LOGGER.log(Level.FINE, "Can't close stream", e); + LOGGER.log(Level.FINE, "Can't close stream"); } } else if (this.getObject().getType() == COSObjType.COS_NAME) { String name = this.getObject().getString(); @@ -128,7 +128,7 @@ private CMap getCMapFile(Set keys) { this.cMapFile = CMapFactory.getCMap(getCMapID(), cMapStream); } } catch (IOException e) { - LOGGER.log(Level.FINE, "Can't close stream", e); + LOGGER.log(Level.FINE, "Can't close stream"); } } else { return null; @@ -249,7 +249,7 @@ private static ASInputStream loadCMap(String cMapName) { } } } catch (IOException e) { - LOGGER.log(Level.FINE, "Error in opening predefined CMap " + cMapName, e); + LOGGER.log(Level.FINE, "Error in opening predefined CMap " + cMapName); return null; } } diff --git a/src/main/java/org/verapdf/pd/font/stdmetrics/StandardFontMetricsFactory.java b/src/main/java/org/verapdf/pd/font/stdmetrics/StandardFontMetricsFactory.java index a6fbed93..60d6941a 100644 --- a/src/main/java/org/verapdf/pd/font/stdmetrics/StandardFontMetricsFactory.java +++ b/src/main/java/org/verapdf/pd/font/stdmetrics/StandardFontMetricsFactory.java @@ -59,7 +59,7 @@ public static StandardFontMetrics getFontMetrics(String fontName) { res = parser.parse(); FONT_METRICS_MAP.put(fontName, res); } catch (IOException e) { - LOGGER.log(Level.FINE, "Can't open file input stream for predefined font file " + DIR_PATH + fontName + EXTENSION, e); + LOGGER.log(Level.FINE, "Can't open file input stream for predefined font file " + DIR_PATH + fontName + EXTENSION); return null; } } @@ -77,7 +77,7 @@ private static ASInputStream load(String fileName) { } } } catch (IOException e) { - LOGGER.log(Level.FINE, "Error in opening predefined font metrics file " + fileName, e); + LOGGER.log(Level.FINE, "Error in opening predefined font metrics file " + fileName); return null; } } diff --git a/src/main/java/org/verapdf/pd/font/truetype/AdobeGlyphList.java b/src/main/java/org/verapdf/pd/font/truetype/AdobeGlyphList.java index 08bbdbbe..8019f97e 100644 --- a/src/main/java/org/verapdf/pd/font/truetype/AdobeGlyphList.java +++ b/src/main/java/org/verapdf/pd/font/truetype/AdobeGlyphList.java @@ -77,7 +77,7 @@ public class AdobeGlyphList { scanner.close(); } } catch (IOException e) { - LOGGER.log(Level.FINE, "Error in opening Adobe Glyph List file", e); + LOGGER.log(Level.FINE, "Error in opening Adobe Glyph List file"); } } diff --git a/src/main/java/org/verapdf/pd/font/truetype/PDTrueTypeFont.java b/src/main/java/org/verapdf/pd/font/truetype/PDTrueTypeFont.java index f3e4a02b..be9fae7e 100644 --- a/src/main/java/org/verapdf/pd/font/truetype/PDTrueTypeFont.java +++ b/src/main/java/org/verapdf/pd/font/truetype/PDTrueTypeFont.java @@ -110,7 +110,7 @@ private void parseTrueTypeFontProgram(ASAtom fontFileType) { } } } catch (Exception e) { - LOGGER.log(Level.FINE, "Can't read TrueType font program.", e); + LOGGER.log(Level.FINE, "Can't read TrueType font program."); } } } diff --git a/src/main/java/org/verapdf/pd/font/type1/PDType1Font.java b/src/main/java/org/verapdf/pd/font/type1/PDType1Font.java index e50fbff6..c8d750cf 100644 --- a/src/main/java/org/verapdf/pd/font/type1/PDType1Font.java +++ b/src/main/java/org/verapdf/pd/font/type1/PDType1Font.java @@ -107,7 +107,7 @@ public Set getDescriptorCharSet() { } return descriptorCharSet; } catch (IOException ex) { - LOGGER.log(Level.FINE, "Can't parse /CharSet entry in Type 1 font descriptor", ex); + LOGGER.log(Level.FINE, "Can't parse /CharSet entry in Type 1 font descriptor"); return Collections.emptySet(); } } @@ -181,7 +181,7 @@ private void parseType1FontProgram(ASAtom fontFileType) { } } } catch (Exception e) { - LOGGER.log(Level.FINE, "Can't read Type 1 font program.", e); + LOGGER.log(Level.FINE, "Can't read Type 1 font program."); } } } diff --git a/src/main/java/org/verapdf/pd/font/type3/PDType3Font.java b/src/main/java/org/verapdf/pd/font/type3/PDType3Font.java index 3b6ab32c..d4f80ac5 100644 --- a/src/main/java/org/verapdf/pd/font/type3/PDType3Font.java +++ b/src/main/java/org/verapdf/pd/font/type3/PDType3Font.java @@ -147,7 +147,7 @@ public float getWidthFromProgram(int code) { parser.parse(); return (float) parser.getWidth(); } catch (IOException e) { - LOGGER.log(Level.FINE, "Can't get width from type 3 char proc", e); + LOGGER.log(Level.FINE, "Can't get width from type 3 char proc"); } } return -1; diff --git a/src/main/java/org/verapdf/pd/function/PDType0Function.java b/src/main/java/org/verapdf/pd/function/PDType0Function.java index 68bc3d5f..7a6f072e 100644 --- a/src/main/java/org/verapdf/pd/function/PDType0Function.java +++ b/src/main/java/org/verapdf/pd/function/PDType0Function.java @@ -154,7 +154,7 @@ private BitSet getSamples() { functionStream.read(bytes); return BitSet.valueOf(bytes); } catch (IOException e) { - LOGGER.log(Level.WARNING, "Can not parse function", e); + LOGGER.log(Level.WARNING, "Can not parse function"); return new BitSet(); } } @@ -210,7 +210,7 @@ public List getResult(List ops) { } return Collections.unmodifiableList(getValuesInIntervals(result, getRange())); } catch (Exception e) { - LOGGER.log(Level.WARNING, "Error while evaluating type 0 function", e); + LOGGER.log(Level.WARNING, "Error while evaluating type 0 function"); return null; } } diff --git a/src/main/java/org/verapdf/pd/images/PDXImage.java b/src/main/java/org/verapdf/pd/images/PDXImage.java index 3f99ef21..1f00bd75 100644 --- a/src/main/java/org/verapdf/pd/images/PDXImage.java +++ b/src/main/java/org/verapdf/pd/images/PDXImage.java @@ -70,7 +70,7 @@ private void parseJPXAndColorSpace() { this.jpxStream = JPEG2000.fromStream(stream); this.imageCS = this.imageCS == null ? this.jpxStream.getImageColorSpace() : this.imageCS; } catch (IOException e) { - LOGGER.log(Level.FINE, "IO Exception reading JP2K stream", e); + LOGGER.log(Level.FINE, "IO Exception reading JP2K stream"); } } COSObject rawColorSpace = getKey(ASAtom.COLORSPACE); diff --git a/src/main/java/org/verapdf/tools/StaticResources.java b/src/main/java/org/verapdf/tools/StaticResources.java index 27f3b31a..1b2c8ea9 100644 --- a/src/main/java/org/verapdf/tools/StaticResources.java +++ b/src/main/java/org/verapdf/tools/StaticResources.java @@ -144,7 +144,7 @@ public static void clear() { try { fpr.close(); } catch (IOException e) { - LOGGER.log(Level.WARNING, "Exception while closing font program", e); + LOGGER.log(Level.WARNING, "Exception while closing font program"); } } } From 4182d3e807f015c92a15b6c73afa8bb59c05d2a9 Mon Sep 17 00:00:00 2001 From: Kakhnovich Raman Date: Fri, 7 Aug 2026 14:26:07 +0300 Subject: [PATCH 2/4] Remove printStackTrace --- .../java/org/verapdf/cos/visitor/COSCopier.java | 1 - src/main/java/org/verapdf/cos/visitor/Writer.java | 15 +-------------- 2 files changed, 1 insertion(+), 15 deletions(-) diff --git a/src/main/java/org/verapdf/cos/visitor/COSCopier.java b/src/main/java/org/verapdf/cos/visitor/COSCopier.java index da8cdc3a..d5d7f1a5 100644 --- a/src/main/java/org/verapdf/cos/visitor/COSCopier.java +++ b/src/main/java/org/verapdf/cos/visitor/COSCopier.java @@ -103,7 +103,6 @@ public void visitFromIndirect(COSIndirect obj) { this.copy.set(obj); } catch (Exception e) { //TODO : throw - e.printStackTrace(); } } } diff --git a/src/main/java/org/verapdf/cos/visitor/Writer.java b/src/main/java/org/verapdf/cos/visitor/Writer.java index 8ac9bb03..e1f7218b 100644 --- a/src/main/java/org/verapdf/cos/visitor/Writer.java +++ b/src/main/java/org/verapdf/cos/visitor/Writer.java @@ -126,7 +126,6 @@ public void visitFromBoolean(COSBoolean obj) { try { this.write(String.valueOf(obj.get())); } catch (IOException e) { - e.printStackTrace(); } } @@ -135,7 +134,6 @@ public void visitFromInteger(COSInteger obj) { try { this.write(obj.toString()); } catch (IOException e) { - e.printStackTrace(); } } @@ -144,7 +142,6 @@ public void visitFromReal(COSReal obj) { try { this.write(obj.toString()); } catch (IOException e) { - e.printStackTrace(); } } @@ -153,7 +150,6 @@ public void visitFromString(COSString obj) { try { this.write(obj.getPrintableString()); } catch (IOException e) { - e.printStackTrace(); } } @@ -162,7 +158,6 @@ public void visitFromName(COSName obj) { try { this.write(obj.toString()); } catch (IOException e) { - e.printStackTrace(); } } @@ -176,7 +171,6 @@ public void visitFromArray(COSArray obj) { } this.write("]"); } catch (IOException e) { - e.printStackTrace(); } } @@ -192,7 +186,6 @@ public void visitFromDictionary(COSDictionary obj) { } this.write(">>"); } catch (IOException e) { - e.printStackTrace(); } } @@ -265,7 +258,6 @@ public void visitFromNull(COSNull obj) { try { this.write("null"); } catch (IOException e) { - e.printStackTrace(); } } @@ -297,7 +289,7 @@ public void writeHeader(final String header) { this.write(comment); this.write(EOL); } catch (IOException e) { - e.printStackTrace(); + } } @@ -360,7 +352,6 @@ public void writeXRefInfo() { this.write("startxref"); this.write(EOL); this.write(this.info.getStartXRef()); this.write(EOL); this.write("%%EOF"); this.write(EOL); } catch (IOException e) { - e.printStackTrace(); } } @@ -375,7 +366,6 @@ public void clear() { this.toWrite.clear(); this.written.clear(); } catch (Exception e) { - e.printStackTrace(); } } @@ -383,7 +373,6 @@ public void close() { try { this.os.close(); } catch (IOException e) { - e.printStackTrace(); } } @@ -391,7 +380,6 @@ protected long getOffset() { try { return this.os.getOffset(); } catch (IOException e) { - e.printStackTrace(); return 0; } } @@ -417,7 +405,6 @@ protected void generateID() { //TODO : convert to COSArray this.info.getTrailer().setID(idString); } catch (NoSuchAlgorithmException e) { - e.printStackTrace(); } } From 68b352fc87a86d60631f304365575d13b76f3ec1 Mon Sep 17 00:00:00 2001 From: Kakhnovich Raman Date: Fri, 7 Aug 2026 14:35:09 +0300 Subject: [PATCH 3/4] Remove more Exception stack trace --- src/main/java/org/verapdf/pd/function/PDType0Function.java | 4 ++-- src/main/java/org/verapdf/pd/function/PDType4Function.java | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/main/java/org/verapdf/pd/function/PDType0Function.java b/src/main/java/org/verapdf/pd/function/PDType0Function.java index 7a6f072e..38f012c5 100644 --- a/src/main/java/org/verapdf/pd/function/PDType0Function.java +++ b/src/main/java/org/verapdf/pd/function/PDType0Function.java @@ -294,7 +294,7 @@ private List multiLinearInterpolation(List x) { } return result; } catch (Exception e) { - LOGGER.log(Level.WARNING, "Failed to get interpolant coefficients", e); + LOGGER.log(Level.WARNING, "Failed to get interpolant coefficients"); return null; } } @@ -316,7 +316,7 @@ private List multiCubicInterpolation(List x) { } return result; } catch (Exception e) { - LOGGER.log(Level.WARNING, "Failed to get interpolant coefficients", e); + LOGGER.log(Level.WARNING, "Failed to get interpolant coefficients"); return null; } } diff --git a/src/main/java/org/verapdf/pd/function/PDType4Function.java b/src/main/java/org/verapdf/pd/function/PDType4Function.java index 4961c1ae..dc418c69 100644 --- a/src/main/java/org/verapdf/pd/function/PDType4Function.java +++ b/src/main/java/org/verapdf/pd/function/PDType4Function.java @@ -73,7 +73,7 @@ public List parseOperators() { try { parseStream(); } catch (IOException e) { - LOGGER.log(Level.WARNING, "Can not parse function", e); + LOGGER.log(Level.WARNING, "Can not parse function"); return Collections.emptyList(); } } @@ -142,7 +142,7 @@ public List getResult(List operands) { List operandsInRange = new ArrayList<>(operandStack); return Collections.unmodifiableList(getValuesInIntervals(operandsInRange, getRange())); } catch (PostScriptException e) { - LOGGER.log(Level.WARNING, "Invalid operators stream", e); + LOGGER.log(Level.WARNING, "Invalid operators stream"); return null; } } From d15942aabe2d12942bc99acd5535759bc8e2c5e1 Mon Sep 17 00:00:00 2001 From: Kakhnovich Raman Date: Mon, 10 Aug 2026 18:08:27 +0300 Subject: [PATCH 4/4] Add logs instead of printStackTrace --- .../java/org/verapdf/cos/visitor/COSCopier.java | 5 +++++ src/main/java/org/verapdf/cos/visitor/Writer.java | 14 +++++++++++++- 2 files changed, 18 insertions(+), 1 deletion(-) diff --git a/src/main/java/org/verapdf/cos/visitor/COSCopier.java b/src/main/java/org/verapdf/cos/visitor/COSCopier.java index d5d7f1a5..940c5455 100644 --- a/src/main/java/org/verapdf/cos/visitor/COSCopier.java +++ b/src/main/java/org/verapdf/cos/visitor/COSCopier.java @@ -24,12 +24,16 @@ import org.verapdf.cos.*; import java.util.Map; +import java.util.logging.Level; +import java.util.logging.Logger; /** * @author Timur Kamalov */ public class COSCopier implements IVisitor { + private static final Logger LOGGER = Logger.getLogger(COSCopier.class.getCanonicalName()); + private final COSObject copy; public COSCopier(COSObject copy) { @@ -103,6 +107,7 @@ public void visitFromIndirect(COSIndirect obj) { this.copy.set(obj); } catch (Exception e) { //TODO : throw + LOGGER.log(Level.WARNING, "Failed to copy from Indirect object"); } } } diff --git a/src/main/java/org/verapdf/cos/visitor/Writer.java b/src/main/java/org/verapdf/cos/visitor/Writer.java index e1f7218b..5414931e 100644 --- a/src/main/java/org/verapdf/cos/visitor/Writer.java +++ b/src/main/java/org/verapdf/cos/visitor/Writer.java @@ -126,6 +126,7 @@ public void visitFromBoolean(COSBoolean obj) { try { this.write(String.valueOf(obj.get())); } catch (IOException e) { + LOGGER.log(Level.WARNING, "Failed to write boolean"); } } @@ -134,6 +135,7 @@ public void visitFromInteger(COSInteger obj) { try { this.write(obj.toString()); } catch (IOException e) { + LOGGER.log(Level.WARNING, "Failed to write integer"); } } @@ -142,6 +144,7 @@ public void visitFromReal(COSReal obj) { try { this.write(obj.toString()); } catch (IOException e) { + LOGGER.log(Level.WARNING, "Failed to write real"); } } @@ -150,6 +153,7 @@ public void visitFromString(COSString obj) { try { this.write(obj.getPrintableString()); } catch (IOException e) { + LOGGER.log(Level.WARNING, "Failed to write string"); } } @@ -158,6 +162,7 @@ public void visitFromName(COSName obj) { try { this.write(obj.toString()); } catch (IOException e) { + LOGGER.log(Level.WARNING, "Failed to write name"); } } @@ -171,6 +176,7 @@ public void visitFromArray(COSArray obj) { } this.write("]"); } catch (IOException e) { + LOGGER.log(Level.WARNING, "Failed to write array"); } } @@ -186,6 +192,7 @@ public void visitFromDictionary(COSDictionary obj) { } this.write(">>"); } catch (IOException e) { + LOGGER.log(Level.WARNING, "Failed to write dictionary"); } } @@ -258,6 +265,7 @@ public void visitFromNull(COSNull obj) { try { this.write("null"); } catch (IOException e) { + LOGGER.log(Level.WARNING, "Failed to write null"); } } @@ -289,7 +297,7 @@ public void writeHeader(final String header) { this.write(comment); this.write(EOL); } catch (IOException e) { - + LOGGER.log(Level.WARNING, "Failed to write header"); } } @@ -352,6 +360,7 @@ public void writeXRefInfo() { this.write("startxref"); this.write(EOL); this.write(this.info.getStartXRef()); this.write(EOL); this.write("%%EOF"); this.write(EOL); } catch (IOException e) { + LOGGER.log(Level.WARNING, "Failed to write XRefInfo"); } } @@ -366,6 +375,7 @@ public void clear() { this.toWrite.clear(); this.written.clear(); } catch (Exception e) { + LOGGER.log(Level.WARNING, "Failed to clear XRefInfo"); } } @@ -373,6 +383,7 @@ public void close() { try { this.os.close(); } catch (IOException e) { + LOGGER.log(Level.WARNING, "Failed to close stream"); } } @@ -405,6 +416,7 @@ protected void generateID() { //TODO : convert to COSArray this.info.getTrailer().setID(idString); } catch (NoSuchAlgorithmException e) { + LOGGER.log(Level.WARNING, "Failed to generate id"); } }