Author: j16sdiz
Date: 2008-09-07 14:28:00 +0000 (Sun, 07 Sep 2008)
New Revision: 22529

Modified:
   trunk/freenet/src/freenet/clients/http/filter/ContentDataFilter.java
   trunk/freenet/src/freenet/clients/http/filter/HTMLFilter.java
Log:
Hashtable -> HashMap + Generic

Modified: trunk/freenet/src/freenet/clients/http/filter/ContentDataFilter.java
===================================================================
--- trunk/freenet/src/freenet/clients/http/filter/ContentDataFilter.java        
2008-09-07 14:27:36 UTC (rev 22528)
+++ trunk/freenet/src/freenet/clients/http/filter/ContentDataFilter.java        
2008-09-07 14:28:00 UTC (rev 22529)
@@ -14,7 +14,9 @@
  */
 public interface ContentDataFilter {

-       public Bucket readFilter(Bucket data, BucketFactory bf, String charset, 
HashMap otherParams, FilterCallback cb) throws DataFilterException, IOException;
+       public Bucket readFilter(Bucket data, BucketFactory bf, String charset, 
HashMap<String, String> otherParams,
+               FilterCallback cb) throws DataFilterException, IOException;

-       public Bucket writeFilter(Bucket data, BucketFactory bf, String 
charset, HashMap otherParams, FilterCallback cb) throws DataFilterException, 
IOException;
+       public Bucket writeFilter(Bucket data, BucketFactory bf, String 
charset, HashMap<String, String> otherParams,
+               FilterCallback cb) throws DataFilterException, IOException;
 }

Modified: trunk/freenet/src/freenet/clients/http/filter/HTMLFilter.java
===================================================================
--- trunk/freenet/src/freenet/clients/http/filter/HTMLFilter.java       
2008-09-07 14:27:36 UTC (rev 22528)
+++ trunk/freenet/src/freenet/clients/http/filter/HTMLFilter.java       
2008-09-07 14:28:00 UTC (rev 22529)
@@ -17,11 +17,10 @@
 import java.io.UnsupportedEncodingException;
 import java.io.Writer;
 import java.nio.charset.MalformedInputException;
-import java.util.Enumeration;
 import java.util.HashMap;
 import java.util.HashSet;
-import java.util.Hashtable;
 import java.util.Iterator;
+import java.util.Map;
 import java.util.StringTokenizer;
 import java.util.Vector;

@@ -43,7 +42,7 @@
        private static boolean deleteWierdStuff = true;
        private static boolean deleteErrors = true;

-       public Bucket readFilter(Bucket bucket, BucketFactory bf, String 
charset, HashMap otherParams,
+       public Bucket readFilter(Bucket bucket, BucketFactory bf, String 
charset, HashMap<String, String> otherParams,
                FilterCallback cb) throws DataFilterException, IOException {
                logMINOR = Logger.shouldLog(Logger.MINOR, this);
                logDEBUG = Logger.shouldLog(Logger.DEBUG, this);
@@ -77,7 +76,7 @@
                return temp;
        }

-       public Bucket writeFilter(Bucket bucket, BucketFactory bf, String 
charset, HashMap otherParams,
+       public Bucket writeFilter(Bucket bucket, BucketFactory bf, String 
charset, HashMap<String, String> otherParams,
                FilterCallback cb) throws DataFilterException, IOException {
                throw new UnsupportedOperationException();
        }
@@ -577,7 +576,7 @@
                }
        }

-       static final Hashtable<String, TagVerifier> allowedTagsVerifiers = new 
Hashtable<String, TagVerifier>();
+       static final Map<String, TagVerifier> allowedTagsVerifiers = new 
HashMap<String, TagVerifier>();
        static final String[] emptyStringArray = new String[0];

        static {
@@ -1160,7 +1159,7 @@
                }

                ParsedTag sanitize(ParsedTag t, HTMLParseContext pc) throws 
DataFilterException {
-                       Hashtable<String, Object> h = new Hashtable<String, 
Object>();
+                       Map<String, Object> h = new HashMap<String, Object>();
                        boolean equals = false;
                        String prevX = "";
                        if (t.unparsedAttrs != null)
@@ -1210,9 +1209,9 @@
                                return new ParsedTag(t, null);
                        String[] outAttrs = new String[h.size()];
                        int i = 0;
-                       for (Enumeration<String> e = h.keys(); 
e.hasMoreElements();) {
-                               String x = e.nextElement();
-                               Object o = h.get(x);
+                       for (Map.Entry<String, Object> entry : h.entrySet()) {
+                               String x = entry.getKey();
+                               Object o = entry.getValue();
                                String y;
                                if (o instanceof String)
                                        y = (String) o;
@@ -1226,13 +1225,13 @@
                        return new ParsedTag(t, outAttrs);
                }

-               Hashtable<String, Object> sanitizeHash(Hashtable<String, 
Object> h,
+               Map<String, Object> sanitizeHash(Map<String, Object> h,
                        ParsedTag p,
                        HTMLParseContext pc) throws DataFilterException {
-                       Hashtable<String, Object> hn = new Hashtable<String, 
Object>();
-                       for (Enumeration<String> e = h.keys(); 
e.hasMoreElements();) {
-                               String x = e.nextElement();
-                               Object o = h.get(x);
+                       Map<String, Object> hn = new HashMap<String, Object>();
+                       for (Map.Entry<String, Object> entry : h.entrySet()) {
+                               String x = entry.getKey();
+                               Object o = entry.getValue();
                                // Straight attribs
                                if (allowedAttrs.contains(x)) {
                                        hn.put(x, o);
@@ -1320,11 +1319,10 @@

                abstract void processStyle(HTMLParseContext pc);

-               @Override
-               Hashtable<String, Object> sanitizeHash(Hashtable<String, 
Object> h,
+               Map<String, Object> sanitizeHash(Map<String, Object> h,
                        ParsedTag p,
                        HTMLParseContext pc) throws DataFilterException {
-                       Hashtable<String, Object> hn = super.sanitizeHash(h, p, 
pc);
+                       Map<String, Object> hn = super.sanitizeHash(h, p, pc);
                        if (p.startSlash) {
                                return finish(h, hn, pc);
                        } else {
@@ -1332,7 +1330,7 @@
                        }
                }

-               Hashtable<String, Object> finish(Hashtable<String, Object> h, 
Hashtable<String, Object> hn,
+               Map<String, Object> finish(Map<String, Object> h, Map<String, 
Object> hn,
                        HTMLParseContext pc) throws DataFilterException {
                        if(logDEBUG) Logger.debug(this, "Finishing 
script/style");
                        // Finishing
@@ -1358,7 +1356,7 @@
                        return hn;
                }

-               Hashtable<String, Object> start(Hashtable<String, Object> h, 
Hashtable<String, Object> hn, HTMLParseContext pc)
+               Map<String, Object> start(Map<String, Object> h, Map<String, 
Object> hn, HTMLParseContext pc)
                        throws DataFilterException {
                        if(logDEBUG) Logger.debug(this, "Starting 
script/style");
                        pc.styleScriptRecurseCount++;
@@ -1429,9 +1427,8 @@
                         */
                }

-               Hashtable<String, Object> sanitizeHash(Hashtable<String, 
Object> hn,
-                       ParsedTag p,
-                       HTMLParseContext pc) throws DataFilterException {
+               Map<String, Object> sanitizeHash(Map<String, Object> hn, 
ParsedTag p, HTMLParseContext pc)
+                       throws DataFilterException {
                        // Call parent so we swallow the scripting
                        super.sanitizeHash(hn, p, pc);
                        return null; // Lose the tags
@@ -1460,10 +1457,10 @@
                        super(tag, allowedAttrs, uriAttrs, inlineURIAttrs);
                }

-               Hashtable<String, Object> sanitizeHash(Hashtable<String, 
Object> h,
+               Map<String, Object> sanitizeHash(Map<String, Object> h,
                        ParsedTag p,
                        HTMLParseContext pc) throws DataFilterException {
-                       Hashtable<String, Object> hn = super.sanitizeHash(h, p, 
pc);
+                       Map<String, Object> hn = super.sanitizeHash(h, p, pc);
                        // %i18n dealt with by TagVerifier
                        // %coreattrs
                        String id = getHashString(h, "id");
@@ -1551,10 +1548,10 @@
                        }
                }

-               Hashtable<String, Object> sanitizeHash(Hashtable<String, 
Object> h,
+               Map<String, Object> sanitizeHash(Map<String, Object> h,
                        ParsedTag p,
                        HTMLParseContext pc) throws DataFilterException {
-                       Hashtable<String, Object> hn = super.sanitizeHash(h, p, 
pc);
+                       Map<String, Object> hn = super.sanitizeHash(h, p, pc);
                        // events (default and added)
                        for (Iterator<String> e = eventAttrs.iterator(); 
e.hasNext();) {
                                String name = e.next();
@@ -1580,10 +1577,10 @@
                        super(tag, allowedAttrs, uriAttrs, inlineURIAttrs, 
eventAttrs);
                }

-               Hashtable<String, Object> sanitizeHash(Hashtable<String, 
Object> h,
+               Map<String, Object> sanitizeHash(Map<String, Object> h,
                        ParsedTag p,
                        HTMLParseContext pc) throws DataFilterException {
-                       Hashtable<String, Object> hn = super.sanitizeHash(h, p, 
pc);
+                       Map<String, Object> hn = super.sanitizeHash(h, p, pc);
                        String hreflang = getHashString(h, "hreflang");
                        String charset = null;
                        String type = getHashString(h, "type");
@@ -1660,10 +1657,10 @@
                        super(tag, allowedAttrs, uriAttrs, null, eventAttrs);
                }

-               Hashtable<String, Object> sanitizeHash(Hashtable<String, 
Object> h,
+               Map<String, Object> sanitizeHash(Map<String, Object> h,
                        ParsedTag p,
                        HTMLParseContext pc) throws DataFilterException {
-                       Hashtable<String, Object> hn = super.sanitizeHash(h, p, 
pc);
+                       Map<String, Object> hn = super.sanitizeHash(h, p, pc);
                        if(p.startSlash) {
                                // Allow, but only with standard elements
                                return hn;
@@ -1716,10 +1713,10 @@
                        }
                }

-               Hashtable<String, Object> sanitizeHash(Hashtable<String, 
Object> h,
+               Map<String, Object> sanitizeHash(Map<String, Object> h,
                        ParsedTag p,
                        HTMLParseContext pc) throws DataFilterException {
-                       Hashtable<String, Object> hn = super.sanitizeHash(h, p, 
pc);
+                       Map<String, Object> hn = super.sanitizeHash(h, p, pc);

                        // We drop the whole <input> if type isn't allowed
                        if(!allowedTypes.contains(hn.get("type"))){
@@ -1735,10 +1732,10 @@
                        super("meta", new String[] { "id" });
                }

-               Hashtable<String, Object> sanitizeHash(Hashtable<String, 
Object> h,
+               Map<String, Object> sanitizeHash(Map<String, Object> h,
                        ParsedTag p,
                        HTMLParseContext pc) throws DataFilterException {
-                       Hashtable<String, Object> hn = super.sanitizeHash(h, p, 
pc);
+                       Map<String, Object> hn = super.sanitizeHash(h, p, pc);
                        /*
                         * Several possibilities: a) meta http-equiv=X 
content=Y b) meta
                         * name=X content=Y
@@ -1818,7 +1815,7 @@
                        super(tag, null);
                }

-               static final Hashtable<String, Object> DTDs = new 
Hashtable<String, Object>();
+               static final Map<String, Object> DTDs = new HashMap<String, 
Object>();

                static {
                        DTDs.put(
@@ -1889,10 +1886,10 @@
                        super("html", new String[] { "id", "version" });
                }

-               Hashtable<String, Object> sanitizeHash(Hashtable<String, 
Object> h,
+               Map<String, Object> sanitizeHash(Map<String, Object> h,
                        ParsedTag p,
                        HTMLParseContext pc) throws DataFilterException {
-                       Hashtable<String, Object> hn = super.sanitizeHash(h, p, 
pc);
+                       Map<String, Object> hn = super.sanitizeHash(h, p, pc);
                        String xmlns = getHashString(h, "xmlns");
                        if ((xmlns != null) && 
xmlns.equals("http://www.w3.org/1999/xhtml";))
                                hn.put("xmlns", xmlns);
@@ -1906,10 +1903,10 @@
                        super(string, strings, strings2, null);
                }

-               Hashtable<String, Object> sanitizeHash(Hashtable<String, 
Object> h,
+               Map<String, Object> sanitizeHash(Map<String, Object> h,
                                ParsedTag p,
                                HTMLParseContext pc) throws DataFilterException 
{
-                       Hashtable<String, Object> hn = super.sanitizeHash(h, p, 
pc);
+                       Map<String, Object> hn = super.sanitizeHash(h, p, pc);
                        // Get the already-sanitized version.
                        String baseHref = getHashString(hn, "href");
                        if(baseHref != null) {
@@ -2065,7 +2062,7 @@
                return cb.processURI(suri, overrideType, false, inline);
        }

-       static String getHashString(Hashtable<String, Object> h, String key) {
+       static String getHashString(Map<String, Object> h, String key) {
                Object o = h.get(key);
                if (o == null)
                        return null;


Reply via email to