Remove unnecessary and confusing includes from StreamBuffer.h.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 13 Feb 2013 05:17:18 +0000 (05:17 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 13 Feb 2013 05:17:18 +0000 (05:17 +0000)
https://bugs.webkit.org/show_bug.cgi?id=109652

Patch by Takeshi Yoshino <tyoshino@google.com> on 2013-02-12
Reviewed by Benjamin Poulain.

StreamBuffer.h is using OwnPtr for storing Vectors into a Deque.
FixedArray.h and PassOwnPtr.h are included but not used.

VectorTraits defines how to move OwnPtr in Vector. It's done by memcpy.
So, there's no need for PassOwnPtr (Deque<PassOwnPtr<Vector<char> > >
is even slower).

* wtf/StreamBuffer.h:

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

Source/WTF/ChangeLog
Source/WTF/wtf/StreamBuffer.h

index 5cf21a3c0b8c99cb7b2f7c7f6855891e91cdf2c5..97290b153b8c52a3274861683fb36be3f6ceec39 100644 (file)
@@ -1,3 +1,19 @@
+2013-02-12  Takeshi Yoshino  <tyoshino@google.com>
+
+        Remove unnecessary and confusing includes from StreamBuffer.h.
+        https://bugs.webkit.org/show_bug.cgi?id=109652
+
+        Reviewed by Benjamin Poulain.
+
+        StreamBuffer.h is using OwnPtr for storing Vectors into a Deque.
+        FixedArray.h and PassOwnPtr.h are included but not used.
+
+        VectorTraits defines how to move OwnPtr in Vector. It's done by memcpy.
+        So, there's no need for PassOwnPtr (Deque<PassOwnPtr<Vector<char> > >
+        is even slower).
+
+        * wtf/StreamBuffer.h:
+
 2013-02-12  Eric Seidel  <eric@webkit.org>
 
         Teach more WTF string classes about vectors with inline capacity
index e7d958f69c32fc9251fce1a03a3d9f3cebde63ef..3015fc2797863476a30a88a30b561c5273e8def2 100644 (file)
@@ -32,9 +32,7 @@
 #define WTF_StreamBuffer_h
 
 #include <wtf/Deque.h>
-#include <wtf/FixedArray.h>
 #include <wtf/OwnPtr.h>
-#include <wtf/PassOwnPtr.h>
 
 namespace WTF {