make OPCPackage#close() unsynchronized and add javadoc note saying method is not...

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1842171 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/src/ooxml/java/org/apache/poi/openxml4j/opc/OPCPackage.java b/src/ooxml/java/org/apache/poi/openxml4j/opc/OPCPackage.java
index 949ec6952024553ef546a4d3ec478253152df2dd..9625a86ff91b3422909e9053b01a16d4fe5af3a8 100644 (file)
--- a/src/ooxml/java/org/apache/poi/openxml4j/opc/OPCPackage.java
+++ b/src/ooxml/java/org/apache/poi/openxml4j/opc/OPCPackage.java
@@ -415,11 +415,13 @@ public abstract class OPCPackage implements RelationshipSource, Closeable {
         * If your package is open read only, then you should call {@link #revert()}
         *  when finished with the package.
         *
+        * This method is not thread-safe.
+        *
         * @throws IOException
         *             If an IO exception occur during the saving process.
         */
        @Override
-    public synchronized void close() throws IOException {
+    public void close() throws IOException {
                if (this.packageAccess == PackageAccess.READ) {
                        logger.log(POILogger.WARN, 
                                "The close() method is intended to SAVE a package. This package is open in READ ONLY mode, use the revert() method instead !");