From f265937d791a003b771be0b36212781bb816b1bf Mon Sep 17 00:00:00 2001 From: Sun Yuhan Date: Thu, 19 Jun 2025 11:31:04 +0800 Subject: [PATCH] refactor: Replace `System.err.println` with log output Signed-off-by: Sun Yuhan --- .../ai/reader/pdf/layout/ForkPDFLayoutTextStripper.java | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/document-readers/pdf-reader/src/main/java/org/springframework/ai/reader/pdf/layout/ForkPDFLayoutTextStripper.java b/document-readers/pdf-reader/src/main/java/org/springframework/ai/reader/pdf/layout/ForkPDFLayoutTextStripper.java index 25903d880e1..dcc10f09c84 100644 --- a/document-readers/pdf-reader/src/main/java/org/springframework/ai/reader/pdf/layout/ForkPDFLayoutTextStripper.java +++ b/document-readers/pdf-reader/src/main/java/org/springframework/ai/reader/pdf/layout/ForkPDFLayoutTextStripper.java @@ -27,6 +27,8 @@ import org.apache.pdfbox.text.PDFTextStripper; import org.apache.pdfbox.text.TextPosition; import org.apache.pdfbox.text.TextPositionComparator; +import org.slf4j.Logger; +import org.slf4j.LoggerFactory; /** * This class extends PDFTextStripper to provide custom text extraction and formatting @@ -38,6 +40,8 @@ */ public class ForkPDFLayoutTextStripper extends PDFTextStripper { + private final static Logger logger = LoggerFactory.getLogger(ForkPDFLayoutTextStripper.class); + public static final boolean DEBUG = false; public static final int OUTPUT_SPACE_CHARACTER_WIDTH_IN_PT = 4; @@ -80,7 +84,7 @@ protected void writePage() throws IOException { this.sortTextPositionList(textList); } catch (java.lang.IllegalArgumentException e) { - System.err.println(e); + logger.error("Error sorting text positions", e); } this.iterateThroughTextList(textList.iterator()); }