RESOLVED FIXED 121154
OwnPtr: Use copy/move-and-swap for assignment operators 
https://bugs.webkit.org/show_bug.cgi?id=121154
Summary OwnPtr: Use copy/move-and-swap for assignment operators
Mikhail Pozdnyakov
Reported 2013-09-11 05:17:57 PDT
Rationals: - decrease of repeated code - consistency with RefPtr
Attachments
patch (2.36 KB, patch)
2013-09-11 06:33 PDT, Mikhail Pozdnyakov 		andersca: review+
andersca: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Mikhail Pozdnyakov
Comment 1 2013-09-11 06:33:59 PDT
Created attachment 211306 [details] patch
Anders Carlsson
Comment 2 2013-09-11 07:17:16 PDT
Comment on attachment 211306 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=211306&action=review > Source/WTF/wtf/OwnPtr.h:140 > - PtrType ptr = m_ptr; > - m_ptr = o.leakPtr(); > - ASSERT(!ptr || m_ptr != ptr); > - deleteOwnedPtr(ptr); > + ASSERT(!o || o != m_ptr); > + OwnPtr ptr = o; > + swap(ptr); I wouldn't worry about modifying PassOwnPtr since then plan is to get rid of it. > Source/WTF/wtf/OwnPtr.h:148 > - PtrType ptr = m_ptr; > - m_ptr = o.leakPtr(); > - ASSERT(!ptr || m_ptr != ptr); > - deleteOwnedPtr(ptr); > + ASSERT(!o || o != m_ptr); > + OwnPtr ptr = o; > + swap(ptr); Ditto. > Source/WTF/wtf/OwnPtr.h:166 > + OwnPtr ptr = std::move(o); Can just use auto here. > Source/WTF/wtf/OwnPtr.h:174 > + OwnPtr ptr = std::move(o); Ditto.
Mikhail Pozdnyakov
Comment 3 2013-09-11 07:32:22 PDT
Committed r155526: <http://trac.webkit.org/changeset/155526>
Note You need to log in before you can comment on or make changes to this bug.