]> source.dussan.org Git - poi.git/commitdiff
remove more deprecated code
authorPJ Fanning <fanningpj@apache.org>
Wed, 9 Dec 2020 17:35:29 +0000 (17:35 +0000)
committerPJ Fanning <fanningpj@apache.org>
Wed, 9 Dec 2020 17:35:29 +0000 (17:35 +0000)
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1884262 13f79535-47bb-0310-9956-ffa450edef68

20 files changed:
src/java/org/apache/poi/ddf/AbstractEscherOptRecord.java
src/java/org/apache/poi/ddf/EscherArrayProperty.java
src/java/org/apache/poi/ddf/EscherComplexProperty.java
src/java/org/apache/poi/hssf/record/CalcCountRecord.java
src/java/org/apache/poi/hssf/record/CalcModeRecord.java
src/java/org/apache/poi/hssf/record/GridsetRecord.java
src/java/org/apache/poi/hssf/record/GroupMarkerSubRecord.java
src/java/org/apache/poi/hssf/record/HorizontalPageBreakRecord.java
src/java/org/apache/poi/hssf/record/LabelRecord.java
src/java/org/apache/poi/hssf/record/LbsDataSubRecord.java
src/java/org/apache/poi/hssf/record/NoteStructureSubRecord.java
src/java/org/apache/poi/hssf/record/ObjRecord.java
src/java/org/apache/poi/hssf/record/PaneRecord.java
src/java/org/apache/poi/hssf/record/PrintHeadersRecord.java
src/java/org/apache/poi/hssf/record/PrintSetupRecord.java
src/java/org/apache/poi/hssf/record/RightMarginRecord.java
src/java/org/apache/poi/hssf/record/StringRecord.java
src/java/org/apache/poi/hssf/record/TextObjectRecord.java
src/java/org/apache/poi/hssf/record/VerticalPageBreakRecord.java
src/java/org/apache/poi/util/IOUtils.java

index dd98cc11e073c67a914d8b3cbfffab8b8e481169..1fdf63380e82f0c526446013bd68d99ba5ed476c 100644 (file)
@@ -24,7 +24,6 @@ import java.util.function.Supplier;
 
 import org.apache.poi.util.GenericRecordUtil;
 import org.apache.poi.util.LittleEndian;
-import org.apache.poi.util.Removal;
 
 /**
  * Common abstract class for {@link EscherOptRecord} and
@@ -153,15 +152,6 @@ public abstract class AbstractEscherOptRecord extends EscherRecord {
         sortProperties();
     }
 
-    /**
-     * @deprecated use {@link #removeEscherProperty(EscherPropertyTypes)} instead
-     */
-    @Deprecated
-    @Removal(version = "5.0.0")
-    public void removeEscherProperty(int num){
-        properties.removeIf(prop -> prop.getPropertyNumber() == num);
-    }
-
     public void removeEscherProperty(EscherPropertyTypes type){
         properties.removeIf(prop -> prop.getPropertyNumber() == type.propNumber);
     }
index c1ac06b7bc6cf4c5761fdd509e0f50e6c1ac39ae..f2c555496ec1f74631e5404443bfd013569236d6 100644 (file)
@@ -54,24 +54,6 @@ public final class EscherArrayProperty extends EscherComplexProperty implements
      */
     private final boolean emptyComplexPart;
 
-    /**
-     * Create an instance of an escher array property.
-     * This constructor defaults to a 6 bytes header if the complexData is null or byte[0].
-     *
-     * @param id          The id consists of the property number, a flag indicating whether this is a blip id and a flag
-     *                    indicating that this is a complex property.
-     * @param complexData The value of this property.
-     *
-     * @deprecated use {@link #EscherArrayProperty(EscherPropertyTypes, boolean, int)} and {@link #setComplexData(byte[])}
-     */
-    @Deprecated
-    @Removal(version = "5.0.0")
-    @Internal
-    public EscherArrayProperty(short id, byte[] complexData) {
-        this(id, safeSize(complexData == null ? 0 : complexData.length));
-        setComplexData(complexData);
-    }
-
     /**
      * Create an instance of an escher array property.
      * This constructor can be used to create emptyComplexParts with a complexSize = 0.
index fc8e75b978ff42b2d214adf4471cf642226c97f7..1f33cd735ada64558515beab352c4c4b4c08c42e 100644 (file)
@@ -24,7 +24,6 @@ import java.util.function.Supplier;
 import org.apache.poi.util.GenericRecordUtil;
 import org.apache.poi.util.IOUtils;
 import org.apache.poi.util.LittleEndian;
-import org.apache.poi.util.Removal;
 
 /**
  * A complex property differs from a simple property in that the data can not fit inside a 32 bit
@@ -37,40 +36,6 @@ public class EscherComplexProperty extends EscherProperty {
 
     private byte[] complexData;
 
-    /**
-     * Create a complex property using the property id and a byte array containing the complex
-     * data value.
-     *
-     * @param id          The id consists of the property number, a flag indicating whether this is a blip id and a flag
-     *                    indicating that this is a complex property.
-     * @param complexData The value of this property.
-     *
-     * @deprecated use {@link #EscherComplexProperty(short, int)} and {@link #setComplexData(byte[])} instead
-     */
-    @Deprecated
-    @Removal(version = "5.0.0")
-    public EscherComplexProperty(short id, byte[] complexData) {
-        this(id, complexData == null ? 0 : complexData.length);
-        setComplexData(complexData);
-    }
-
-    /**
-     * Create a complex property using the property number, a flag to indicate whether this is a
-     * blip reference and the complex property data.
-     *
-     * @param propertyNumber The property number
-     * @param isBlipId       Whether this is a blip id.  Should be false.
-     * @param complexData    The value of this complex property.
-     *
-     * @deprecated use {@link #EscherComplexProperty(EscherPropertyTypes, boolean, int)} and {@link #setComplexData(byte[])} instead
-     */
-    @Deprecated
-    @Removal(version = "5.0.0")
-    public EscherComplexProperty(short propertyNumber, boolean isBlipId, byte[] complexData) {
-        this(propertyNumber, isBlipId, complexData == null ? 0 : complexData.length);
-        setComplexData(complexData);
-    }
-
     /**
      * Create a complex property using the property id and a byte array containing the complex
      * data value size.
index 4f865ecf4241e7db5ddda6af84d845f6f9c84576..3fec4a9b431187ae42807eaa49f810f08735804c 100644 (file)
@@ -24,7 +24,6 @@ import java.util.function.Supplier;
 
 import org.apache.poi.util.GenericRecordUtil;
 import org.apache.poi.util.LittleEndianOutput;
-import org.apache.poi.util.Removal;
 
 /**
  * Specifies the maximum times the gui should perform a formula recalculation.
@@ -86,16 +85,6 @@ public final class CalcCountRecord extends StandardRecord {
         return sid;
     }
 
-    /**
-     * @deprecated use {@link #copy()} instead
-     */
-    @SuppressWarnings({"squid:S2975", "MethodDoesntCallSuperMethod"})
-    @Deprecated
-    @Removal(version = "5.0.0")
-    public CalcCountRecord clone() {
-        return copy();
-    }
-
     @Override
     public CalcCountRecord copy() {
         return new CalcCountRecord(this);
index 1dd6f93cd06ac0ed5266552cd625d2b42d424f86..1cac3134ac3980ee0dfc75039414d58ac2d6b271 100644 (file)
@@ -24,7 +24,6 @@ import java.util.function.Supplier;
 
 import org.apache.poi.util.GenericRecordUtil;
 import org.apache.poi.util.LittleEndianOutput;
-import org.apache.poi.util.Removal;
 
 /**
  * Tells the gui whether to calculate formulas automatically, manually or automatically except for tables.
@@ -101,16 +100,6 @@ public final class CalcModeRecord extends StandardRecord {
         return sid;
     }
 
-    /**
-     * @deprecated use {@link #copy()} instead
-     */
-    @SuppressWarnings({"squid:S2975", "MethodDoesntCallSuperMethod"})
-    @Deprecated
-    @Removal(version = "5.0.0")
-    public CalcModeRecord clone() {
-        return copy();
-    }
-
     @Override
     public CalcModeRecord copy() {
         return new CalcModeRecord(this);
index a7f050b2aa51fcb45c8220cd2e45e661209d3481..d5f705071f60c7cf586b225b1daed071a1c184ed 100644 (file)
@@ -22,7 +22,6 @@ import java.util.function.Supplier;
 
 import org.apache.poi.util.GenericRecordUtil;
 import org.apache.poi.util.LittleEndianOutput;
-import org.apache.poi.util.Removal;
 
 /**
  * Flag denoting whether the user specified that gridlines are used when printing.
@@ -80,16 +79,6 @@ public final class GridsetRecord extends StandardRecord {
         return sid;
     }
 
-    /**
-     * @deprecated use {@link #copy()} instead
-     */
-    @SuppressWarnings({"squid:S2975", "MethodDoesntCallSuperMethod"})
-    @Deprecated
-    @Removal(version = "5.0.0")
-    public GridsetRecord clone() {
-        return copy();
-    }
-
     @Override
     public GridsetRecord copy() {
         return new GridsetRecord(this);
index d7fe77df8688550f9337743cceb5b01f3b239242..805bca34421bf69ad534d6efb72c474b167050d4 100644 (file)
@@ -24,7 +24,6 @@ import org.apache.poi.util.GenericRecordUtil;
 import org.apache.poi.util.IOUtils;
 import org.apache.poi.util.LittleEndianInput;
 import org.apache.poi.util.LittleEndianOutput;
-import org.apache.poi.util.Removal;
 
 /**
  * ftGmo (0x0006)<p>
@@ -75,16 +74,6 @@ public final class GroupMarkerSubRecord extends SubRecord {
         return sid;
     }
 
-    /**
-     * @deprecated use {@link #copy()} instead
-     */
-    @SuppressWarnings("squid:S2975")
-    @Deprecated
-    @Removal(version = "5.0.0")
-    public GroupMarkerSubRecord clone() {
-        return copy();
-    }
-
     @Override
     public GroupMarkerSubRecord copy() {
         return new GroupMarkerSubRecord(this);
index 328239d6cae3d603a9924bc52d5c8aea04115c6a..67b16b3d0b1d24035c99f3d2a4609c95248f270d 100644 (file)
@@ -17,8 +17,6 @@
 
 package org.apache.poi.hssf.record;
 
-import org.apache.poi.util.Removal;
-
 /**
  * HorizontalPageBreak (0x001B) record that stores page breaks at rows
  *
@@ -48,17 +46,6 @@ public final class HorizontalPageBreakRecord extends PageBreakRecord {
                return sid;
        }
 
-       /**
-        * @deprecated use {@link #copy()} instead
-        */
-       @Override
-       @SuppressWarnings({"squid:S2975", "MethodDoesntCallSuperMethod"})
-       @Deprecated
-       @Removal(version = "5.0.0")
-       public PageBreakRecord clone() {
-               return copy();
-       }
-
        @Override
        public HorizontalPageBreakRecord copy() {
                return new HorizontalPageBreakRecord(this);
index d6b1d031aee7acff014b056d9472b63d91b06fcf..9683c86e4cf6f156a5e3ee0969e96f8951369787 100644 (file)
@@ -25,7 +25,6 @@ import org.apache.poi.util.HexDump;
 import org.apache.poi.util.POILogFactory;
 import org.apache.poi.util.POILogger;
 import org.apache.poi.util.RecordFormatException;
-import org.apache.poi.util.Removal;
 
 /**
  * Label Record (0x0204) - read only support for strings stored directly in the cell...
@@ -173,19 +172,7 @@ public final class LabelRecord extends Record implements CellValueRecordInterfac
      * no op!
      */
     @Override
-    public void setXFIndex(short xf)
-    {
-    }
-
-    /**
-     * @deprecated use {@link #copy()} instead
-     */
-    @SuppressWarnings({"squid:S2975", "MethodDoesntCallSuperMethod"})
-    @Deprecated
-    @Removal(version = "5.0.0")
-    public LabelRecord clone() {
-        return copy();
-    }
+    public void setXFIndex(short xf) {}
 
     @Override
     public LabelRecord copy() {
index 3838f76bd2bd151e1cb6bed7f212dd317e7f7256..cf06408b3f19ec59a4ab9a12670494a8ffd7db51 100644 (file)
@@ -29,7 +29,6 @@ import org.apache.poi.util.GenericRecordUtil;
 import org.apache.poi.util.LittleEndianInput;
 import org.apache.poi.util.LittleEndianOutput;
 import org.apache.poi.util.RecordFormatException;
-import org.apache.poi.util.Removal;
 import org.apache.poi.util.StringUtil;
 
 /**
@@ -281,16 +280,6 @@ public class LbsDataSubRecord extends SubRecord {
         }
     }
 
-    /**
-     * @deprecated use {@link #copy()} instead
-     */
-    @SuppressWarnings("squid:S2975")
-    @Deprecated
-    @Removal(version = "5.0.0")
-    public LbsDataSubRecord clone() {
-        return copy();
-    }
-
     @Override
     public LbsDataSubRecord copy() {
         return new LbsDataSubRecord(this);
index 85c553792e7dc1450ead25dc1d49567e8b2be683..3adff3856d7478bfe76b49f7b9ff26aa9cf72839 100644 (file)
@@ -25,7 +25,6 @@ import org.apache.poi.util.IOUtils;
 import org.apache.poi.util.LittleEndianInput;
 import org.apache.poi.util.LittleEndianOutput;
 import org.apache.poi.util.RecordFormatException;
-import org.apache.poi.util.Removal;
 
 /**
  * ftNts (0x000D)<p>
@@ -99,16 +98,6 @@ public final class NoteStructureSubRecord extends SubRecord {
         return sid;
     }
 
-    /**
-     * @deprecated use {@link #copy()} instead
-     */
-    @SuppressWarnings("squid:S2975")
-    @Deprecated
-    @Removal(version = "5.0.0")
-    public NoteStructureSubRecord clone() {
-        return copy();
-    }
-
     @Override
     public NoteStructureSubRecord copy() {
         return new NoteStructureSubRecord(this);
index 2efd1bfa225a89a2181a2ef7ab418e034b00a88f..beea8ac7e872db449a41363d4b0164d4836efead 100644 (file)
@@ -29,7 +29,6 @@ import org.apache.poi.util.LittleEndian;
 import org.apache.poi.util.LittleEndianByteArrayInputStream;
 import org.apache.poi.util.LittleEndianByteArrayOutputStream;
 import org.apache.poi.util.RecordFormatException;
-import org.apache.poi.util.Removal;
 
 /**
  * OBJRECORD (0x005D)<p>
@@ -215,16 +214,6 @@ public final class ObjRecord extends Record {
         return subrecords.add(o);
     }
 
-    /**
-     * @deprecated use {@link #copy()} instead
-     */
-    @SuppressWarnings({"squid:S2975", "MethodDoesntCallSuperMethod"})
-    @Deprecated
-    @Removal(version = "5.0.0")
-    public ObjRecord clone() {
-        return copy();
-    }
-
     @Override
     public ObjRecord copy() {
         return new ObjRecord(this);
index 6db2d7a15c1f109f7f3dbefce7736ade55bcd35c..fa3834696155a2a29ebe0afa98c762a26f3f9b86 100644 (file)
@@ -25,7 +25,6 @@ import java.util.function.Supplier;
 
 import org.apache.poi.util.GenericRecordUtil;
 import org.apache.poi.util.LittleEndianOutput;
-import org.apache.poi.util.Removal;
 
 /**
  * Describes the frozen and unfrozen panes.
@@ -82,16 +81,6 @@ public final class PaneRecord extends StandardRecord {
         return sid;
     }
 
-    /**
-     * @deprecated use {@link #copy()} instead
-     */
-    @SuppressWarnings({"squid:S2975", "MethodDoesntCallSuperMethod"})
-    @Deprecated
-    @Removal(version = "5.0.0")
-    public PaneRecord clone() {
-        return copy();
-    }
-
     @Override
     public PaneRecord copy() {
         return new PaneRecord(this);
index ec3906f7d3a04a85dc2ed05a8fec51f488983062..07752d70d4501af51ef90d31c90492267bfbeee0 100644 (file)
@@ -22,7 +22,6 @@ import java.util.function.Supplier;
 
 import org.apache.poi.util.GenericRecordUtil;
 import org.apache.poi.util.LittleEndianOutput;
-import org.apache.poi.util.Removal;
 
 /**
  * Whether or not to print the row/column headers when you enjoy your spreadsheet in the physical form.
@@ -79,16 +78,6 @@ public final class PrintHeadersRecord extends StandardRecord {
         return sid;
     }
 
-    /**
-     * @deprecated use {@link #copy()} instead
-     */
-    @SuppressWarnings({"squid:S2975", "MethodDoesntCallSuperMethod"})
-    @Deprecated
-    @Removal(version = "5.0.0")
-    public PrintHeadersRecord clone() {
-        return copy();
-    }
-
     public PrintHeadersRecord copy() {
       return new PrintHeadersRecord(this);
     }
index 00ebf2dbb630bcb529f8d5feab2cc42d205f189d..d6404e6245b4db60d55b63d94c730cde0dd1e810 100644 (file)
@@ -28,7 +28,6 @@ import org.apache.poi.ss.usermodel.PrintSetup;
 import org.apache.poi.util.BitField;
 import org.apache.poi.util.BitFieldFactory;
 import org.apache.poi.util.LittleEndianOutput;
-import org.apache.poi.util.Removal;
 
 /**
  * Stores print setup options -- bogus for HSSF (and marked as such)
@@ -316,16 +315,6 @@ public final class PrintSetupRecord extends StandardRecord {
         return sid;
     }
 
-    /**
-     * @deprecated use {@link #copy()} instead
-     */
-    @SuppressWarnings({"squid:S2975", "MethodDoesntCallSuperMethod"})
-    @Deprecated
-    @Removal(version = "5.0.0")
-    public PrintSetupRecord clone() {
-        return copy();
-    }
-
     @Override
     public PrintSetupRecord copy() {
       return new PrintSetupRecord(this);
index a19ac0c01c29165d4eaeb904ba1d7d83e1d4f1e2..94fc0645b2e174f142bc95a12616dd972ba8f95f 100644 (file)
@@ -22,7 +22,6 @@ import java.util.function.Supplier;
 
 import org.apache.poi.util.GenericRecordUtil;
 import org.apache.poi.util.LittleEndianOutput;
-import org.apache.poi.util.Removal;
 
 /**
  * Record for the right margin.
@@ -63,16 +62,6 @@ public final class RightMarginRecord extends StandardRecord implements Margin {
     public void setMargin( double field_1_margin )
     {        this.field_1_margin = field_1_margin;    }
 
-    /**
-     * @deprecated use {@link #copy()} instead
-     */
-    @SuppressWarnings({"squid:S2975", "MethodDoesntCallSuperMethod"})
-    @Deprecated
-    @Removal(version = "5.0.0")
-    public RightMarginRecord clone() {
-        return copy();
-    }
-
     public RightMarginRecord copy() {
         return new RightMarginRecord(this);
     }
index 96a22f8bd301dd3cb1120c1f18d891370cd31ce4..efd86f1f790320b1e037f93a078b64fc44b048a5 100644 (file)
@@ -23,7 +23,6 @@ import java.util.function.Supplier;
 import org.apache.poi.hssf.record.cont.ContinuableRecord;
 import org.apache.poi.hssf.record.cont.ContinuableRecordOutput;
 import org.apache.poi.util.GenericRecordUtil;
-import org.apache.poi.util.Removal;
 import org.apache.poi.util.StringUtil;
 
 /**
@@ -86,16 +85,6 @@ public final class StringRecord extends ContinuableRecord {
         _is16bitUnicode = StringUtil.hasMultibyte(string);
     }
 
-    /**
-     * @deprecated use {@link #copy()} instead
-     */
-    @SuppressWarnings({"squid:S2975", "MethodDoesntCallSuperMethod"})
-    @Deprecated
-    @Removal(version = "5.0.0")
-    public StringRecord clone() {
-        return copy();
-    }
-
     public StringRecord copy() {
         return new StringRecord(this);
     }
index c3377e2c896535de9264571c5e7ae1582f9479de..8beeb3a621100e0423773f17fa679c4d03b7a9b1 100644 (file)
@@ -30,7 +30,6 @@ import org.apache.poi.ss.formula.ptg.Ptg;
 import org.apache.poi.util.BitField;
 import org.apache.poi.util.BitFieldFactory;
 import org.apache.poi.util.RecordFormatException;
-import org.apache.poi.util.Removal;
 
 /**
  * The TXO record (0x01B6) is used to define the properties of a text box. It is
@@ -317,17 +316,6 @@ public final class TextObjectRecord extends ContinuableRecord {
                return _linkRefPtg;
        }
 
-       /**
-        * @deprecated use {@link #copy()} instead
-        */
-       @Override
-       @SuppressWarnings({"squid:S2975", "MethodDoesntCallSuperMethod"})
-       @Deprecated
-       @Removal(version = "5.0.0")
-       public TextObjectRecord clone() {
-               return copy();
-       }
-
        @Override
        public TextObjectRecord copy() {
                return new TextObjectRecord(this);
index 7644a54417e47bf96ae7c9d383d7cd451a2a64a1..bd28b190c27ff1a2f11ab7ac63618a8a9647dc70 100644 (file)
@@ -17,8 +17,6 @@
 
 package org.apache.poi.hssf.record;
 
-import org.apache.poi.util.Removal;
-
 /**
  * VerticalPageBreak (0x001A) record that stores page breaks at columns
  *
@@ -48,17 +46,6 @@ public final class VerticalPageBreakRecord extends PageBreakRecord {
                return sid;
        }
 
-       /**
-        * @deprecated use {@link #copy()} instead
-        */
-       @Override
-       @SuppressWarnings({"squid:S2975", "MethodDoesntCallSuperMethod"})
-       @Deprecated
-       @Removal(version = "5.0.0")
-       public VerticalPageBreakRecord clone() {
-               return copy();
-       }
-
        @Override
        public VerticalPageBreakRecord copy() {
                return new VerticalPageBreakRecord(this);
index 68f5bbddc5b22403277e58edf228a5d2cce6523f..fc75f4f6520fa0579ef064cd38903d26a49c33ac 100644 (file)
@@ -33,9 +33,6 @@ import java.util.zip.CRC32;
 import java.util.zip.Checksum;
 
 import org.apache.poi.EmptyFileException;
-import org.apache.poi.POIDocument;
-import org.apache.poi.ss.usermodel.Workbook;
-
 
 @Internal
 public final class IOUtils {
@@ -290,137 +287,6 @@ public final class IOUtils {
         }
     }
 
-    /**
-     * Write a POI Document ({@link org.apache.poi.ss.usermodel.Workbook}, {@link org.apache.poi.sl.usermodel.SlideShow}, etc) to an output stream and close the output stream.
-     * This will attempt to close the output stream at the end even if there was a problem writing the document to the stream.
-     *
-     * If you are using Java 7 or higher, you may prefer to use a try-with-resources statement instead.
-     * This function exists for Java 6 code.
-     *
-     * @param doc  a writeable document to write to the output stream
-     * @param out  the output stream that the document is written to
-     * @throws IOException thrown on errors writing to the stream
-     *
-     * @deprecated since 4.0, use try-with-resources, will be removed in 4.2
-     */
-    @Deprecated
-    @Removal(version="4.2")
-    public static void write(POIDocument doc, OutputStream out) throws IOException {
-        try {
-            doc.write(out);
-        } finally {
-            closeQuietly(out);
-        }
-    }
-
-    /**
-     * Write a ({@link org.apache.poi.ss.usermodel.Workbook}) to an output stream and close the output stream.
-     * This will attempt to close the output stream at the end even if there was a problem writing the document to the stream.
-     *
-     * If you are using Java 7 or higher, you may prefer to use a try-with-resources statement instead.
-     * This function exists for Java 6 code.
-     *
-     * @param doc  a writeable document to write to the output stream
-     * @param out  the output stream that the document is written to
-     * @throws IOException thrown on errors writing to the stream
-     *
-     * @deprecated since 4.0, use try-with-resources, will be removed in 4.2
-     */
-    @Deprecated
-    @Removal(version="4.2")
-    public static void write(Workbook doc, OutputStream out) throws IOException {
-        try {
-            doc.write(out);
-        } finally {
-            closeQuietly(out);
-        }
-    }
-
-    /**
-     * Write a POI Document ({@link org.apache.poi.ss.usermodel.Workbook}, {@link org.apache.poi.sl.usermodel.SlideShow}, etc) to an output stream and close the output stream.
-     * This will attempt to close the output stream at the end even if there was a problem writing the document to the stream.
-     * This will also attempt to close the document, even if an error occurred while writing the document or closing the output stream.
-     *
-     * If you are using Java 7 or higher, you may prefer to use a try-with-resources statement instead.
-     * This function exists for Java 6 code.
-     *
-     * @param doc  a writeable and closeable document to write to the output stream, then close
-     * @param out  the output stream that the document is written to
-     * @throws IOException thrown on errors writing to the stream
-     *
-     * @deprecated since 4.0, use try-with-resources, will be removed in 4.2
-     */
-    @Deprecated
-    @Removal(version="4.2")
-    public static void writeAndClose(POIDocument doc, OutputStream out) throws IOException {
-        try {
-            write(doc, out);
-        } finally {
-            closeQuietly(doc);
-        }
-    }
-
-    /**
-     * Like {@link #writeAndClose(POIDocument, OutputStream)}, but for writing to a File instead of an OutputStream.
-     * This will attempt to close the document, even if an error occurred while writing the document.
-     *
-     * If you are using Java 7 or higher, you may prefer to use a try-with-resources statement instead.
-     * This function exists for Java 6 code.
-     *
-     * @param doc  a writeable and closeable document to write to the output file, then close
-     * @param out  the output file that the document is written to
-     * @throws IOException thrown on errors writing to the stream
-     *
-     * @deprecated since 4.0, use try-with-resources, will be removed in 4.2
-     */
-    @Deprecated
-    @Removal(version="4.2")
-    public static void writeAndClose(POIDocument doc, File out) throws IOException {
-        try {
-            doc.write(out);
-        } finally {
-            closeQuietly(doc);
-        }
-    }
-
-    /**
-     * Like {@link #writeAndClose(POIDocument, File)}, but for writing a POI Document in place (to the same file that it was opened from).
-     * This will attempt to close the document, even if an error occurred while writing the document.
-     *
-     * If you are using Java 7 or higher, you may prefer to use a try-with-resources statement instead.
-     * This function exists for Java 6 code.
-     *
-     * @param doc  a writeable document to write in-place
-     * @throws IOException thrown on errors writing to the file
-     *
-     * @deprecated since 4.0, use try-with-resources, will be removed in 4.2
-     */
-    @Deprecated
-    @Removal(version="4.2")
-    public static void writeAndClose(POIDocument doc) throws IOException {
-        try {
-            doc.write();
-        } finally {
-            closeQuietly(doc);
-        }
-    }
-
-    // Since the Workbook interface doesn't derive from POIDocument
-    // We'll likely need one of these for each document container interface
-    /**
-     *
-     * @deprecated since 4.0, use try-with-resources, will be removed in 4.2
-     */
-    @Deprecated
-    @Removal(version="4.2")
-    public static void writeAndClose(Workbook doc, OutputStream out) throws IOException {
-        try {
-            doc.write(out);
-        } finally {
-            closeQuietly(doc);
-        }
-    }
-
     /**
      * Copies all the data from the given InputStream to the OutputStream. It
      * leaves both streams open, so you will still need to close them once done.