Remove debugMode option from WriterProperties

It was related to pdfDebug only.

DEVSIX-5706

Author: yulian-gaponenko

commons/pom.xml

@@ -7,7 +7,7 @@
     <version>7.2.0-SNAPSHOT</version>
   </parent>
   <artifactId>commons</artifactId>
-  <name>iText 7 - events</name>
+  <name>iText 7 - commons</name>
   <url>https://itextpdf.com/</url>
 
   <dependencies>

kernel/src/main/java/com/itextpdf/kernel/pdf/PdfWriter.java

@@ -64,9 +64,6 @@ public class PdfWriter extends PdfOutputStream {
     private static final byte[] obj = ByteUtils.getIsoBytes(" obj\n");
     private static final byte[] endobj = ByteUtils.getIsoBytes("\nendobj\n");
 
-    // For internal usage only
-    private PdfOutputStream duplicateStream = null;
-
     protected WriterProperties properties;
 
     /**
@@ -114,9 +111,6 @@ public PdfWriter(java.io.OutputStream os) {
     public PdfWriter(java.io.OutputStream os, WriterProperties properties) {
         super(new CountOutputStream(FileUtil.wrapWithBufferedOutputStream(os)));
         this.properties = properties;
-        if (properties.debugMode) {
-            duplicateStream = new PdfOutputStream(new ByteArrayOutputStream());
-        }
     }
 
     /**
@@ -192,80 +186,6 @@ public PdfWriter setSmartMode(boolean smartMode) {
         return this;
     }
 
-    /**
-     * Write an integer to the underlying stream
-     *
-     * @param b integer to write
-     * @throws java.io.IOException if an I/O error occurs. In particular,
-     *                             an <code>IOException</code> may be thrown if the output stream
-     *                             has been closed.
-     */
-    @Override
-    public void write(int b) throws java.io.IOException {
-        super.write(b);
-        if (duplicateStream != null) {
-            duplicateStream.write(b);
-        }
-    }
-
-    /**
-     * Write a byte array to the underlying stream
-     *
-     * @param b byte array to write
-     * @throws java.io.IOException if an I/O error occurs. In particular,
-     *                             an <code>IOException</code> may be thrown if the output stream
-     *                             has been closed.
-     */
-    @Override
-    public void write(byte[] b) throws java.io.IOException {
-        super.write(b);
-        if (duplicateStream != null) {
-            duplicateStream.write(b);
-        }
-    }
-
-    /**
-     * Write a slice of the passed byte array to the underlying stream
-     *
-     * @param b   byte array to slice and write.
-     * @param off starting index of the slice.
-     * @param len length of the slice.
-     * @throws java.io.IOException if an I/O error occurs. In particular,
-     *                             an <code>IOException</code> may be thrown if the output stream
-     *                             has been closed.
-     */
-    @Override
-    public void write(byte[] b, int off, int len) throws java.io.IOException {
-        super.write(b, off, len);
-        if (duplicateStream != null) {
-            duplicateStream.write(b, off, len);
-        }
-    }
-
-
-    /**
-     * Close the writer and underlying streams.
-     *
-     * @throws java.io.IOException if an I/O error occurs. In particular,
-     *                             an <code>IOException</code> may be thrown if the output stream
-     *                             has been closed previously.
-     */
-    @Override
-    public void close() throws IOException {
-        try {
-            super.close();
-        } finally {
-            try {
-                if (duplicateStream != null) {
-                    duplicateStream.close();
-                }
-            } catch (Exception ex) {
-                Logger logger = LoggerFactory.getLogger(PdfWriter.class);
-                logger.error("Closing of the duplicatedStream failed.", ex);
-            }
-        }
-    }
-
     /**
      * Gets the current object stream.
      *

kernel/src/main/java/com/itextpdf/kernel/pdf/WriterProperties.java

@@ -60,7 +60,6 @@ public class WriterProperties {
      * and reused if there's an object with the same content later.
      */
     protected boolean smartMode;
-    protected boolean debugMode;
     protected boolean addXmpMetadata;
     protected boolean addUAXmpMetadata;
     protected PdfVersion pdfVersion;
@@ -77,7 +76,6 @@ public class WriterProperties {
 
     public WriterProperties() {
         smartMode = false;
-        debugMode = false;
         addUAXmpMetadata = false;
         compressionLevel = CompressionConstants.DEFAULT_COMPRESSION;
         isFullCompression = null;
@@ -250,17 +248,6 @@ public WriterProperties setModifiedDocumentId(PdfString modifiedDocumentId) {
         this.modifiedDocumentId = modifiedDocumentId;
         return this;
     }
-    /**
-     * This activates debug mode with pdfDebug tool.
-     * It causes additional overhead of duplicating document bytes into memory, so use it careful.
-     * NEVER use it in production or in any other cases except pdfDebug.
-     *
-     * @return this {@link WriterProperties} instance
-     */
-    public WriterProperties useDebugMode() {
-        this.debugMode = true;
-        return this;
-    }
 
     /**
      * This method marks the document as PDF/UA and sets related flags is XMPMetaData.