]> source.dussan.org Git - poi.git/commitdiff
bug 59748: replace Hashtable with HashMap; contributed by Axel Howind
authorJaven O'Neal <onealj@apache.org>
Fri, 24 Jun 2016 21:39:44 +0000 (21:39 +0000)
committerJaven O'Neal <onealj@apache.org>
Fri, 24 Jun 2016 21:39:44 +0000 (21:39 +0000)
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1750168 13f79535-47bb-0310-9956-ffa450edef68

17 files changed:
src/java/org/apache/poi/hssf/eventusermodel/FormatTrackingHSSFListener.java
src/java/org/apache/poi/hssf/usermodel/HSSFWorkbook.java
src/java/org/apache/poi/hssf/util/HSSFColor.java
src/ooxml/java/org/apache/poi/openxml4j/opc/OPCPackage.java
src/ooxml/java/org/apache/poi/openxml4j/opc/internal/ContentType.java
src/ooxml/testcases/org/apache/poi/openxml4j/opc/TestContentType.java
src/scratchpad/src/org/apache/poi/hdgf/chunks/ChunkFactory.java
src/scratchpad/src/org/apache/poi/hslf/record/DocumentEncryptionAtom.java
src/scratchpad/src/org/apache/poi/hslf/record/ExOleObjStg.java
src/scratchpad/src/org/apache/poi/hslf/record/PersistPtrHolder.java
src/scratchpad/src/org/apache/poi/hslf/record/PositionDependentRecord.java
src/scratchpad/src/org/apache/poi/hslf/record/PositionDependentRecordAtom.java
src/scratchpad/src/org/apache/poi/hslf/record/PositionDependentRecordContainer.java
src/scratchpad/src/org/apache/poi/hslf/record/UserEditAtom.java
src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFSlideShowImpl.java
src/scratchpad/testcases/org/apache/poi/hslf/TestReWriteSanity.java
src/testcases/org/apache/poi/hssf/util/TestHSSFColor.java

index 074c294c542faf27cd8b90926b885bdf6457f585..3547a5815d9be626c6f4058f421021ae7bd5042b 100644 (file)
@@ -18,7 +18,7 @@ package org.apache.poi.hssf.eventusermodel;
 
 import java.text.NumberFormat;
 import java.util.ArrayList;
-import java.util.Hashtable;
+import java.util.HashMap;
 import java.util.List;
 import java.util.Locale;
 import java.util.Map;
@@ -45,7 +45,7 @@ public class FormatTrackingHSSFListener implements HSSFListener {
        private final HSSFListener _childListener;
        private final HSSFDataFormatter _formatter;
        private final NumberFormat _defaultFormat;
-       private final Map<Integer, FormatRecord> _customFormatRecords = new Hashtable<Integer, FormatRecord>();
+       private final Map<Integer, FormatRecord> _customFormatRecords = new HashMap<Integer, FormatRecord>();
        private final List<ExtendedFormatRecord> _xfRecords = new ArrayList<ExtendedFormatRecord>();
 
        /**
index d35aa4b571416706793410a9cd4d0ecc3575a9fc..32cf2229d69e0acb0916d5b4fe8c2b03bbb09893 100644 (file)
@@ -33,9 +33,8 @@ import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Collection;
 import java.util.Collections;
-import java.util.HashMap;
 import java.util.HashSet;
-import java.util.Hashtable;
+import java.util.HashMap;
 import java.util.Iterator;
 import java.util.List;
 import java.util.Map;
@@ -155,7 +154,7 @@ public final class HSSFWorkbook extends POIDocument implements org.apache.poi.ss
      * this holds the HSSFFont objects attached to this workbook.
      * We only create these from the low level records as required.
      */
-    private Hashtable<Short,HSSFFont> fonts;
+    private Map<Short,HSSFFont> fonts;
 
     /**
      * holds whether or not to preserve other nodes in the POIFS.  Used
@@ -1204,7 +1203,7 @@ public final class HSSFWorkbook extends POIDocument implements org.apache.poi.ss
      */
     @Override
     public HSSFFont getFontAt(short idx) {
-        if(fonts == null) fonts = new Hashtable<Short, HSSFFont>();
+        if(fonts == null) fonts = new HashMap<Short, HSSFFont>();
 
         // So we don't confuse users, give them back
         //  the same object every time, but create
@@ -1228,7 +1227,7 @@ public final class HSSFWorkbook extends POIDocument implements org.apache.poi.ss
      *  and that's not something you should normally do
      */
     protected void resetFontCache() {
-        fonts = new Hashtable<Short, HSSFFont>();
+        fonts = new HashMap<Short, HSSFFont>();
     }
 
     /**
index ea4b3e2783c923fd7f540ac5b935945442e5f1ed..55ff46bf1e3d027cd1e5025a6b5ac5df769838ce 100644 (file)
@@ -19,7 +19,7 @@ package org.apache.poi.hssf.util;
 
 import java.lang.reflect.Field;
 import java.util.Collections;
-import java.util.Hashtable;
+import java.util.HashMap;
 import java.util.Map;
 
 import org.apache.poi.ss.usermodel.Color;
@@ -57,18 +57,18 @@ public class HSSFColor implements Color {
         return indexHash;
     }
     /**
-     * This function returns all the Colours, stored in a Hashtable that
+     * This function returns all the Colours, stored in a Map that
      *  can be edited. No caching is performed. If you don't need to edit
      *  the table, then call {@link #getIndexHash()} which returns a
      *  statically cached imuatable map of colours.
      */
-    public final static Hashtable<Integer,HSSFColor> getMutableIndexHash() {
+    public final static Map<Integer,HSSFColor> getMutableIndexHash() {
        return createColorsByIndexMap();
     }
 
-    private static Hashtable<Integer,HSSFColor> createColorsByIndexMap() {
+    private static Map<Integer,HSSFColor> createColorsByIndexMap() {
         HSSFColor[] colors = getAllColors();
-        Hashtable<Integer,HSSFColor> result = new Hashtable<Integer,HSSFColor>(colors.length * 3 / 2);
+        Map<Integer,HSSFColor> result = new HashMap<Integer,HSSFColor>(colors.length * 3 / 2);
 
         for (int i = 0; i < colors.length; i++) {
             HSSFColor color = colors[i];
@@ -148,16 +148,16 @@ public class HSSFColor implements Color {
      * it takes to create it once per request but you will not hold onto it
      * if you have none of those requests.
      *
-     * @return a hashtable containing all colors keyed by String gnumeric-like triplets
+     * @return a Map containing all colors keyed by String gnumeric-like triplets
      */
-    public final static Hashtable<String,HSSFColor> getTripletHash()
+    public final static Map<String,HSSFColor> getTripletHash()
     {
         return createColorsByHexStringMap();
     }
 
-    private static Hashtable<String,HSSFColor> createColorsByHexStringMap() {
+    private static Map<String,HSSFColor> createColorsByHexStringMap() {
         HSSFColor[] colors = getAllColors();
-        Hashtable<String,HSSFColor> result = new Hashtable<String,HSSFColor>(colors.length * 3 / 2);
+        Map<String,HSSFColor> result = new HashMap<String,HSSFColor>(colors.length * 3 / 2);
 
         for (int i = 0; i < colors.length; i++) {
             HSSFColor color = colors[i];
@@ -1683,7 +1683,7 @@ public class HSSFColor implements Color {
 
     /**
      * Special Default/Normal/Automatic color.
-     * <p><i>Note:</i> This class is NOT in the default HashTables returned by HSSFColor.
+     * <p><i>Note:</i> This class is NOT in the default Map returned by HSSFColor.
      * The index is a special case which is interpreted in the various setXXXColor calls.
      *
      * @author Jason
index 1b15c407d1858a0a328d8a8fd9f0667196aa91b5..c70e496593bb9fc756a7e87edf0a8ab1d415a552 100644 (file)
@@ -30,7 +30,7 @@ import java.net.URISyntaxException;
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.Date;
-import java.util.Hashtable;
+import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 import java.util.concurrent.locks.ReentrantReadWriteLock;
@@ -145,8 +145,8 @@ public abstract class OPCPackage implements RelationshipSource, Closeable {
         * Initialize the package instance.
         */
        private void init() {
-               this.partMarshallers = new Hashtable<ContentType, PartMarshaller>(5);
-               this.partUnmarshallers = new Hashtable<ContentType, PartUnmarshaller>(2);
+               this.partMarshallers = new HashMap<ContentType, PartMarshaller>(5);
+               this.partUnmarshallers = new HashMap<ContentType, PartUnmarshaller>(2);
 
                try {
                        // Add 'default' unmarshaller
index 24afccb212f8e89145acc81065ee10fe76dd54e5..a56e0cdf35bfe2846f8daac3f27adbd542d0b541 100644 (file)
 
 package org.apache.poi.openxml4j.opc.internal;
 
-import java.util.Hashtable;
+import java.util.Collections;
+import java.util.HashMap;
 import java.util.Map;
+import java.util.Map.Entry;
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
 
@@ -53,17 +55,17 @@ public final class ContentType {
        /**
         * Type in Type/Subtype.
         */
-       private String type;
+       private final String type;
 
        /**
         * Subtype
         */
-       private String subType;
+       private final String subType;
 
        /**
         * Parameters
         */
-       private Hashtable<String, String> parameters;
+       private final Map<String, String> parameters;
 
        /**
         * Media type compiled pattern, without parameters
@@ -144,15 +146,15 @@ public final class ContentType {
        public ContentType(String contentType) throws InvalidFormatException {
                Matcher mMediaType = patternTypeSubType.matcher(contentType);
                if (!mMediaType.matches()) {
-                   // How about with parameters?
-                   mMediaType = patternTypeSubTypeParams.matcher(contentType);
+                       // How about with parameters?
+                       mMediaType = patternTypeSubTypeParams.matcher(contentType);
                }
-        if (!mMediaType.matches()) {
+               if (!mMediaType.matches()) {
                        throw new InvalidFormatException(
                                        "The specified content type '"
                                        + contentType
                                        + "' is not compliant with RFC 2616: malformed content type.");
-        }
+               }
 
                // Type/subtype
                if (mMediaType.groupCount() >= 2) {
@@ -160,15 +162,20 @@ public final class ContentType {
                        this.subType = mMediaType.group(2);
                        
                        // Parameters
-                       this.parameters = new Hashtable<String, String>(1);
+                       this.parameters = new HashMap<String, String>();
                        // Java RegExps are unhelpful, and won't do multiple group captures
                        // See http://docs.oracle.com/javase/6/docs/api/java/util/regex/Pattern.html#cg
                        if (mMediaType.groupCount() >= 5) {
-                           Matcher mParams = patternParams.matcher(contentType.substring(mMediaType.end(2)));
-                           while (mParams.find()) {
-                               this.parameters.put(mParams.group(1), mParams.group(2));
-                           }
+                               Matcher mParams = patternParams.matcher(contentType.substring(mMediaType.end(2)));
+                               while (mParams.find()) {
+                                       this.parameters.put(mParams.group(1), mParams.group(2));
+                               }
                        }
+               } else {
+                       // missing media type and subtype
+                       this.type = "";
+                       this.subType = "";
+                       this.parameters = Collections.emptyMap(); 
                }
        }
 
@@ -187,7 +194,7 @@ public final class ContentType {
            retVal.append(this.getSubType());
 
            if (withParameters) {
-               for (Map.Entry<String, String> me : parameters.entrySet()) {
+               for (Entry<String, String> me : parameters.entrySet()) {
                    retVal.append(";");
                    retVal.append(me.getKey());
                    retVal.append("=");
index e77e4f84827640a7c8357356eb33fb8d6d7a8304..21eb24ad83789783ce6dbdf97edcfad20e5b617d 100644 (file)
@@ -181,53 +181,65 @@ public final class TestContentType extends TestCase {
         
         // Check the types on everything
         for (PackagePart part : p.getParts()) {
+            final String contentType = part.getContentType();
+            final ContentType details = part.getContentTypeDetails();
+            final int length = details.getParameterKeys().length;
+            final boolean hasParameters = details.hasParameters();
+
             // _rels type doesn't have any params
             if (part.isRelationshipPart()) {
-                assertEquals(ContentTypes.RELATIONSHIPS_PART, part.getContentType());
-                assertEquals(ContentTypes.RELATIONSHIPS_PART, part.getContentTypeDetails().toString());
-                assertEquals(false, part.getContentTypeDetails().hasParameters());
-                assertEquals(0, part.getContentTypeDetails().getParameterKeys().length);
+                assertEquals(ContentTypes.RELATIONSHIPS_PART, contentType);
+                assertEquals(ContentTypes.RELATIONSHIPS_PART, details.toString());
+                assertEquals(false, hasParameters);
+                assertEquals(0, length);
             }
             // Core type doesn't have any params
             else if (part.getPartName().toString().equals("/docProps/core.xml")) {
-                assertEquals(ContentTypes.CORE_PROPERTIES_PART, part.getContentType());
-                assertEquals(ContentTypes.CORE_PROPERTIES_PART, part.getContentTypeDetails().toString());
-                assertEquals(false, part.getContentTypeDetails().hasParameters());
-                assertEquals(0, part.getContentTypeDetails().getParameterKeys().length);
+                assertEquals(ContentTypes.CORE_PROPERTIES_PART, contentType);
+                assertEquals(ContentTypes.CORE_PROPERTIES_PART, details.toString());
+                assertEquals(false, hasParameters);
+                assertEquals(0, length);
             }
             // Global Crs types do have params
             else if (part.getPartName().toString().equals("/global1dCrs.xml")) {
-                assertEquals(typeResqml, part.getContentType().substring(0, typeResqml.length()));
-                assertEquals(typeResqml, part.getContentTypeDetails().toString(false));
-                assertEquals(true, part.getContentTypeDetails().hasParameters());
-                assertEquals(typeResqml+";version=2.0;type=obj_global1dCrs", part.getContentTypeDetails().toString());
-                assertEquals(2, part.getContentTypeDetails().getParameterKeys().length);
-                assertEquals("2.0", part.getContentTypeDetails().getParameter("version"));
-                assertEquals("obj_global1dCrs", part.getContentTypeDetails().getParameter("type"));
+                assertTrue(part.getContentType().startsWith(typeResqml));
+                assertEquals(typeResqml, details.toString(false));
+                assertEquals(true, hasParameters);
+                assertContains("version=2.0", details.toString());
+                assertContains("type=obj_global1dCrs", details.toString());
+                assertEquals(2, length);
+                assertEquals("2.0", details.getParameter("version"));
+                assertEquals("obj_global1dCrs", details.getParameter("type"));
             }
             else if (part.getPartName().toString().equals("/global2dCrs.xml")) {
-                assertEquals(typeResqml, part.getContentType().substring(0, typeResqml.length()));
-                assertEquals(typeResqml, part.getContentTypeDetails().toString(false));
-                assertEquals(true, part.getContentTypeDetails().hasParameters());
-                assertEquals(typeResqml+";version=2.0;type=obj_global2dCrs", part.getContentTypeDetails().toString());
-                assertEquals(2, part.getContentTypeDetails().getParameterKeys().length);
-                assertEquals("2.0", part.getContentTypeDetails().getParameter("version"));
-                assertEquals("obj_global2dCrs", part.getContentTypeDetails().getParameter("type"));
+                assertTrue(part.getContentType().startsWith(typeResqml));
+                assertEquals(typeResqml, details.toString(false));
+                assertEquals(true, hasParameters);
+                assertContains("version=2.0", details.toString());
+                assertContains("type=obj_global2dCrs", details.toString());
+                assertEquals(2, length);
+                assertEquals("2.0", details.getParameter("version"));
+                assertEquals("obj_global2dCrs", details.getParameter("type"));
             }
             // Other thingy
             else if (part.getPartName().toString().equals("/myTestingGuid.xml")) {
-                assertEquals(typeResqml, part.getContentType().substring(0, typeResqml.length()));
-                assertEquals(typeResqml, part.getContentTypeDetails().toString(false));
-                assertEquals(true, part.getContentTypeDetails().hasParameters());
-                assertEquals(typeResqml+";version=2.0;type=obj_tectonicBoundaryFeature", part.getContentTypeDetails().toString());
-                assertEquals(2, part.getContentTypeDetails().getParameterKeys().length);
-                assertEquals("2.0", part.getContentTypeDetails().getParameter("version"));
-                assertEquals("obj_tectonicBoundaryFeature", part.getContentTypeDetails().getParameter("type"));
+                assertTrue(part.getContentType().startsWith(typeResqml));
+                assertEquals(typeResqml, details.toString(false));
+                assertEquals(true, hasParameters);
+                assertContains("version=2.0", details.toString());
+                assertContains("type=obj_tectonicBoundaryFeature", details.toString());
+                assertEquals(2, length);
+                assertEquals("2.0", details.getParameter("version"));
+                assertEquals("obj_tectonicBoundaryFeature", details.getParameter("type"));
             }
             // That should be it!
             else {
                 fail("Unexpected part " + part);
             }
         }
-       }
+    }
+
+    private static void assertContains(String needle, String haystack) {
+        assertTrue(haystack.contains(needle));
+    }
 }
index 819051ff13305156e794e4c357de3731a366863c..4b69576ee3f8f5474e0c3572a01292575f6e1f3a 100644 (file)
@@ -22,7 +22,7 @@ import java.io.IOException;
 import java.io.InputStream;
 import java.io.InputStreamReader;
 import java.util.ArrayList;
-import java.util.Hashtable;
+import java.util.HashMap;
 import java.util.Map;
 import java.util.StringTokenizer;
 
@@ -44,7 +44,7 @@ public final class ChunkFactory {
         * Key is a Chunk's type, value is an array of its CommandDefinitions
         */
        private final Map<Integer, CommandDefinition[]> chunkCommandDefinitions = 
-             new Hashtable<Integer, CommandDefinition[]>();
+             new HashMap<Integer, CommandDefinition[]>();
        /**
         * What the name is of the chunk table definitions file?
         * This file comes from the scratchpad resources directory.
@@ -105,7 +105,7 @@ public final class ChunkFactory {
     
                        CommandDefinition[] defs = defsL.toArray(new CommandDefinition[defsL.size()]);
     
-                       // Add to the hashtable
+                       // Add to the map
                        chunkCommandDefinitions.put(Integer.valueOf(chunkType), defs);
                }
                } finally {
index fa2f6d560c13662fd9a8e671adcafbf8cfe2745d..99c6c68835b6eb281021c03cd69566893fe9a280 100644 (file)
@@ -20,7 +20,7 @@ package org.apache.poi.hslf.record;
 import java.io.ByteArrayInputStream;
 import java.io.IOException;
 import java.io.OutputStream;
-import java.util.Hashtable;
+import java.util.Map;
 
 import org.apache.poi.poifs.crypt.CipherAlgorithm;
 import org.apache.poi.poifs.crypt.EncryptionInfo;
@@ -123,7 +123,7 @@ public final class DocumentEncryptionAtom extends PositionDependentRecordAtom {
                out.write(data, 0, bos.getWriteIndex());
        }
 
-    public void updateOtherRecordReferences(Hashtable<Integer,Integer> oldToNewReferencesLookup) {
+    public void updateOtherRecordReferences(Map<Integer,Integer> oldToNewReferencesLookup) {
         
     }
 }
index b78ccdc8232d2f85e668112b038e14e448e51f68..5ef523159f13756a5a94b23c19c97b00b87b9df5 100644 (file)
@@ -22,7 +22,7 @@ import java.io.ByteArrayOutputStream;
 import java.io.IOException;
 import java.io.InputStream;
 import java.io.OutputStream;
-import java.util.Hashtable;
+import java.util.Map;
 import java.util.zip.DeflaterOutputStream;
 import java.util.zip.InflaterInputStream;
 
@@ -184,7 +184,7 @@ public class ExOleObjStg extends RecordAtom implements PositionDependentRecord,
         myLastOnDiskOffset = offset;
     }
 
-    public void updateOtherRecordReferences(Hashtable<Integer,Integer> oldToNewReferencesLookup) {
+    public void updateOtherRecordReferences(Map<Integer,Integer> oldToNewReferencesLookup) {
         return;
     }
 
index 32b36a21b92e8821b4f3fa39a870900d5c3ec14c..b108241486be6981324258435120a4d8ebfc9ca2 100644 (file)
@@ -20,8 +20,10 @@ package org.apache.poi.hslf.record;
 import java.io.ByteArrayOutputStream;
 import java.io.IOException;
 import java.io.OutputStream;
-import java.util.Hashtable;
+import java.util.Collections;
+import java.util.HashMap;
 import java.util.Map;
+import java.util.Map.Entry;
 import java.util.TreeMap;
 
 import org.apache.poi.hslf.exceptions.CorruptPowerPointFileException;
@@ -52,7 +54,7 @@ public final class PersistPtrHolder extends PositionDependentRecordAtom
         * You always need to check the most recent PersistPtrHolder
         *  that knows about a given slide to find the right location
         */
-       private Hashtable<Integer,Integer> _slideLocations;
+       private Map<Integer,Integer> _slideLocations;
 
        private static final BitField persistIdFld = new BitField(0X000FFFFF);
        private static final BitField cntPersistFld  = new BitField(0XFFF00000);
@@ -64,7 +66,7 @@ public final class PersistPtrHolder extends PositionDependentRecordAtom
 
        /**
         * Get the list of slides that this PersistPtrHolder knows about.
-        * (They will be the keys in the hashtable for looking up the positions
+        * (They will be the keys in the map for looking up the positions
         *  of these slides)
         */
        public int[] getKnownSlideIDs() {
@@ -80,8 +82,8 @@ public final class PersistPtrHolder extends PositionDependentRecordAtom
         * Get the lookup from slide numbers to byte offsets, for the slides
         *  known about by this PersistPtrHolder.
         */
-       public Hashtable<Integer,Integer> getSlideLocationsLookup() {
-               return _slideLocations;
+       public Map<Integer,Integer> getSlideLocationsLookup() {
+               return Collections.unmodifiableMap(_slideLocations);
        }
        
        /**
@@ -104,7 +106,7 @@ public final class PersistPtrHolder extends PositionDependentRecordAtom
                //      base number for these entries
                //   count * 32 bit offsets
                // Repeat as many times as you have data
-               _slideLocations = new Hashtable<Integer,Integer>();
+               _slideLocations = new HashMap<Integer,Integer>();
                _ptrData = new byte[len-8];
                System.arraycopy(source,start+8,_ptrData,0,_ptrData.length);
 
@@ -157,10 +159,10 @@ public final class PersistPtrHolder extends PositionDependentRecordAtom
         * At write-out time, update the references to the sheets to their
         *  new positions
         */
-       public void updateOtherRecordReferences(Hashtable<Integer,Integer> oldToNewReferencesLookup) {
+       public void updateOtherRecordReferences(Map<Integer,Integer> oldToNewReferencesLookup) {
                // Loop over all the slides we know about
                // Find where they used to live, and where they now live
-           for (Map.Entry<Integer,Integer> me : _slideLocations.entrySet()) {
+           for (Entry<Integer,Integer> me : _slideLocations.entrySet()) {
                Integer oldPos = me.getValue();
                Integer newPos = oldToNewReferencesLookup.get(oldPos);
 
@@ -182,7 +184,7 @@ public final class PersistPtrHolder extends PositionDependentRecordAtom
         byte intbuf[] = new byte[4];
         int lastPersistEntry = -1;
         int lastSlideId = -1;
-        for (Map.Entry<Integer,Integer> me : orderedSlideLocations.entrySet()) {
+        for (Entry<Integer,Integer> me : orderedSlideLocations.entrySet()) {
             int nextSlideId = me.getKey();
             int offset = me.getValue();
             try {
index e73323c02dd6c719ca6bb0c859ceafb08f495765..844c14d45dae0ed50c8659f698db51c4f713b565 100644 (file)
@@ -16,7 +16,7 @@
 ==================================================================== */
 
 package org.apache.poi.hslf.record;
-import java.util.Hashtable;
+import java.util.Map;
 
 /**
  * Records which either care about where they are on disk, or have other
@@ -47,5 +47,5 @@ public interface PositionDependentRecord
         * Offer the record the list of records that have changed their
         *  location as part of the writeout.
         */
-       public void updateOtherRecordReferences(Hashtable<Integer,Integer> oldToNewReferencesLookup);
+       public void updateOtherRecordReferences(Map<Integer,Integer> oldToNewReferencesLookup);
 }
index 28228eab362488ec2f25825d8c7fbc3200df5596..76d8f49eb7aead2459f69c3366793240c9796c74 100644 (file)
@@ -16,7 +16,7 @@
 ==================================================================== */
 
 package org.apache.poi.hslf.record;
-import java.util.Hashtable;
+import java.util.Map;
 
 /**
  * A special (and dangerous) kind of Record Atom that cares about where
@@ -48,5 +48,5 @@ public abstract class PositionDependentRecordAtom extends RecordAtom implements
         * Allows records to update their internal pointers to other records
         *  locations
         */
-       public abstract void updateOtherRecordReferences(Hashtable<Integer,Integer> oldToNewReferencesLookup);
+       public abstract void updateOtherRecordReferences(Map<Integer,Integer> oldToNewReferencesLookup);
 }
index a297f302b69c9116aa69d203b07032c3b08db691..14b1932fbff12d6314d3f837d8c7c73fdb95d9e8 100644 (file)
@@ -16,7 +16,7 @@
 ==================================================================== */
 
 package org.apache.poi.hslf.record;
-import java.util.Hashtable;
+import java.util.Map;
 
 /**
  * A special (and dangerous) kind of Record Container, for which other
@@ -60,7 +60,7 @@ public abstract class PositionDependentRecordContainer extends RecordContainer i
         * Since we're a container, we don't mind if other records move about.
         * If we're told they have, just return straight off.
         */
-       public void updateOtherRecordReferences(Hashtable<Integer,Integer> oldToNewReferencesLookup) {
+       public void updateOtherRecordReferences(Map<Integer,Integer> oldToNewReferencesLookup) {
                return;
        }
 }
index d0c8b563d79e06b897769656f6e29cf8c10afcb5..b1bd6bba3e4ee59e67093feec37355e61a6bf7df 100644 (file)
@@ -22,7 +22,7 @@ import org.apache.poi.util.LittleEndianConsts;
 
 import java.io.IOException;
 import java.io.OutputStream;
-import java.util.Hashtable;
+import java.util.Map;
 
 /**
  * A UserEdit Atom (type 4085). Holds information which bits of the file
@@ -146,7 +146,7 @@ public final class UserEditAtom extends PositionDependentRecordAtom
         * At write-out time, update the references to PersistPtrs and
         *  other UserEditAtoms to point to their new positions
         */
-       public void updateOtherRecordReferences(Hashtable<Integer,Integer> oldToNewReferencesLookup) {
+       public void updateOtherRecordReferences(Map<Integer,Integer> oldToNewReferencesLookup) {
                // Look up the new positions of our preceding UserEditAtomOffset
                if(lastUserEditAtomOffset != 0) {
                        Integer newLocation = oldToNewReferencesLookup.get(Integer.valueOf(lastUserEditAtomOffset));
index d878182b3715d03832407a8ba1ce3129dfe0b021..b316a0e9e875e1ce1cd694c56944c5476681afe7 100644 (file)
@@ -28,7 +28,7 @@ import java.security.GeneralSecurityException;
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.HashMap;
-import java.util.Hashtable;
+import java.util.Map;
 import java.util.List;
 import java.util.Map;
 import java.util.NavigableMap;
@@ -464,7 +464,7 @@ public final class HSLFSlideShowImpl extends POIDocument implements Closeable {
         // For position dependent records, hold where they were and now are
         // As we go along, update, and hand over, to any Position Dependent
         //  records we happen across
-        Hashtable<Integer,Integer> oldToNewPositions = new Hashtable<Integer,Integer>();
+        Map<Integer,Integer> oldToNewPositions = new HashMap<Integer,Integer>();
 
         // First pass - figure out where all the position dependent
         //   records are going to end up, in the new scheme
index fefa202009aed8090d321fd9efb73eee1994f0d2..8de67e5da3dedb0c9e61b43fb3f4ccad435fde3b 100644 (file)
@@ -60,8 +60,8 @@ public final class TestReWriteSanity extends TestCase {
                // Find the location of the PersistPtrIncrementalBlocks and
                // UserEditAtoms
                Record[] r = wss.getRecords();
-               Map<Integer,Record> pp = new Hashtable<Integer,Record>();
-               Map<Integer,Object> ue = new Hashtable<Integer,Object>();
+               Map<Integer,Record> pp = new HashMap<Integer,Record>();
+               Map<Integer,Object> ue = new HashMap<Integer,Object>();
                ue.put(Integer.valueOf(0),Integer.valueOf(0)); // Will show 0 if first
                int pos = 0;
                int lastUEPos = -1;
index d93f5d7b08f925557c876dd0cb92ef9fdeed7eec..319a7449ecfed3f497cac3e5471d8754ed760c95 100644 (file)
@@ -17,7 +17,7 @@
 
 package org.apache.poi.hssf.util;
 
-import java.util.Hashtable;
+import java.util.Map;
 
 import junit.framework.TestCase;
 /**
@@ -40,7 +40,7 @@ public final class TestHSSFColor extends TestCase {
        }
        
        public void testTrippletHash() {
-               Hashtable tripplets = HSSFColor.getTripletHash();
+               Map<String, HSSFColor> tripplets = HSSFColor.getTripletHash();
                
                assertEquals(
                                HSSFColor.MAROON.class,