JavaScriptCore:
authordarin@apple.com <darin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 14 Dec 2007 20:50:30 +0000 (20:50 +0000)
committerdarin@apple.com <darin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 14 Dec 2007 20:50:30 +0000 (20:50 +0000)
        Reviewed by Sam.

        - removed unnecessary includes of "Vector.h"

        * wtf/HashMap.h:
        (WTF::copyKeysToVector): Make the type of the vector be a template parameter.
        This allows copying keys into a vector of a base class or one with an inline capacity.
        (WTF::copyValuesToVector): Ditto.
        * wtf/HashSet.h:
        (WTF::copyToVector): Ditto.

WebCore:

        Reviewed by Sam.

        - fix http://bugs.webkit.org/show_bug.cgi?id=16351
          FontFallbackList.h doesn't include wtf/PassRefPtr.h

        * platform/graphics/FontFallbackList.h: Added include of <wtf/Forward.h>
        * platform/graphics/GlyphPageTreeNode.h: Removed an unneeded include.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@28721 268f45cc-cd09-0410-ab3c-d52691b4dbfc

JavaScriptCore/ChangeLog
JavaScriptCore/wtf/HashMap.h
JavaScriptCore/wtf/HashSet.h
WebCore/ChangeLog
WebCore/platform/graphics/FontFallbackList.h
WebCore/platform/graphics/GlyphPageTreeNode.h

index f3a836790d29e7590f02848dfd5477ff7709db9e..9702ce54653b777f3dfcf4e4c914d961a816efd6 100644 (file)
@@ -1,3 +1,16 @@
+2007-12-14  Darin Adler  <darin@apple.com>
+
+        Reviewed by Sam.
+
+        - removed unnecessary includes of "Vector.h"
+
+        * wtf/HashMap.h:
+        (WTF::copyKeysToVector): Make the type of the vector be a template parameter.
+        This allows copying keys into a vector of a base class or one with an inline capacity.
+        (WTF::copyValuesToVector): Ditto.
+        * wtf/HashSet.h:
+        (WTF::copyToVector): Ditto.
+
 2007-12-14  Anders Carlsson  <andersca@apple.com>
 
         Reviewed by Darin and Geoff.
index a598b3d138893d95114a88d25b92ae5374f7400a..da174582e42d3629a8eb22c8a9e4473dc103337b 100644 (file)
@@ -23,7 +23,6 @@
 #define WTF_HashMap_h
 
 #include "HashTable.h"
-#include "Vector.h"
 
 namespace WTF {
 
@@ -393,8 +392,8 @@ namespace WTF {
         deleteAllPairFirsts<typename HashMap<T, U, V, W, X>::KeyType>(collection);
     }
     
-    template<typename T, typename U, typename V, typename W, typename X>
-    inline void copyKeysToVector(const HashMap<T, U, V, W, X>& collection, Vector<T>& vector)
+    template<typename T, typename U, typename V, typename W, typename X, typename Y>
+    inline void copyKeysToVector(const HashMap<T, U, V, W, X>& collection, Y& vector)
     {
         typedef typename HashMap<T, U, V, W, X>::const_iterator::Keys iterator;
         
@@ -406,8 +405,8 @@ namespace WTF {
             vector[i] = *it;
     }  
 
-    template<typename T, typename U, typename V, typename W, typename X>
-    inline void copyValuesToVector(const HashMap<T, U, V, W, X>& collection, Vector<U>& vector)
+    template<typename T, typename U, typename V, typename W, typename X, typename Y>
+    inline void copyValuesToVector(const HashMap<T, U, V, W, X>& collection, Y& vector)
     {
         typedef typename HashMap<T, U, V, W, X>::const_iterator::Values iterator;
         
index 7da41f67300fcc2474efb42982a027dc52da6dbc..f6909560016cb1c78725b361a63bbbee697c735d 100644 (file)
@@ -1,8 +1,6 @@
 // -*- mode: c++; c-basic-offset: 4 -*-
 /*
- * This file is part of the KDE libraries
- *
- * Copyright (C) 2005, 2006 Apple Computer, Inc.
+ * Copyright (C) 2005, 2006, 2007 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ -25,7 +23,6 @@
 #define WTF_HashSet_h
 
 #include "HashTable.h"
-#include "Vector.h"
 
 namespace WTF {
 
@@ -317,8 +314,8 @@ namespace WTF {
         deleteAllValues<typename HashSet<T, U, V>::ValueType>(collection.m_impl);
     }
     
-    template<typename T, typename U, typename V>
-    inline void copyToVector(const HashSet<T, U, V>& collection, Vector<T>& vector)
+    template<typename T, typename U, typename V, typename W>
+    inline void copyToVector(const HashSet<T, U, V>& collection, W& vector)
     {
         typedef typename HashSet<T, U, V>::const_iterator iterator;
         
@@ -329,6 +326,7 @@ namespace WTF {
         for (unsigned i = 0; it != end; ++it, ++i)
             vector[i] = *it;
     }  
+
 } // namespace WTF
 
 using WTF::HashSet;
index 97c170244093302de21ae1cf9bd0b9aeb8ae14a4..6500f098c1441b74977a685e53be34f6c2dbe836 100644 (file)
@@ -1,3 +1,13 @@
+2007-12-14  Darin Adler  <darin@apple.com>
+
+        Reviewed by Sam.
+
+        - fix http://bugs.webkit.org/show_bug.cgi?id=16351
+          FontFallbackList.h doesn't include wtf/PassRefPtr.h
+          
+        * platform/graphics/FontFallbackList.h: Added include of <wtf/Forward.h>
+        * platform/graphics/GlyphPageTreeNode.h: Removed an unneeded include.
+
 2007-12-14  Darin Adler  <darin@apple.com>
 
         Reviewed by Alexey.
index b7873f9c84f120a994bfb1cb0f387c1894487c30..faf04608049e4f63ad45f1b0467c37224fd96be0 100644 (file)
  */
 
 // This file has no guards on purpose in order to detect redundant includes. This is a private header
-// and so this should catch anyone trying to include this file in public cpp files.
+// and so this may catch someone trying to include this file in public cpp files.
 
 #include "FontData.h"
-#include <wtf/RefCounted.h>
 #include "FontSelector.h"
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace WebCore {
 
index c9037d4ee2a7adfc676b2a8a2e2257a31ef32649..53074ef3029dfb2761ac31cb822f61907c017dc4 100644 (file)
@@ -31,7 +31,6 @@
 
 #include <wtf/RefCounted.h>
 #include <wtf/unicode/Unicode.h>
-#include <wtf/Noncopyable.h>
 #include <wtf/HashMap.h>
 
 namespace WebCore {