Remove Ref(const T&) constructor.
authorakling@apple.com <akling@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 15 Sep 2013 12:51:26 +0000 (12:51 +0000)
committerakling@apple.com <akling@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 15 Sep 2013 12:51:26 +0000 (12:51 +0000)
<https://webkit.org/b/121384>

Reviewed by Anders Carlsson.

This constructor was added as a stopgap measure until we got better support
for move semantics in Vector.

* wtf/Ref.h:

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

Source/WTF/ChangeLog
Source/WTF/wtf/Ref.h

index edb437f..4227081 100644 (file)
@@ -1,5 +1,17 @@
 2013-09-15  Andreas Kling  <akling@apple.com>
 
+        Remove Ref(const T&) constructor.
+        <https://webkit.org/b/121384>
+
+        Reviewed by Anders Carlsson.
+
+        This constructor was added as a stopgap measure until we got better support
+        for move semantics in Vector.
+
+        * wtf/Ref.h:
+
+2013-09-15  Andreas Kling  <akling@apple.com>
+
         Unreviewed, fix ridiculous bug in WeakPtr.
 
         I made operator!() null check the pointer to the WeakReference. :|
index 6cecd9b..6946ec2 100644 (file)
@@ -32,8 +32,6 @@ namespace WTF {
 
 template<typename T> class Ref {
 public:
-    // Ref(const T&) looks awkward but makes Vector<Ref<T>> possible.
-    Ref(const T& object) : m_ptr(&const_cast<T&>(object)) { m_ptr->ref(); }
     Ref(T& object) : m_ptr(&object) { m_ptr->ref(); }
 
     ~Ref() { m_ptr->deref(); }