Changeset 11763 in webkit for trunk/JavaScriptCore/kjs/identifier.cpp

Timestamp:

Dec 25, 2005, 1:22:35 AM (19 years ago)

Author:

mjs

Message:

Reviewed by Eric and Dave Hyatt.

• make even const PassRefPtrs give transfer of ownership semantics ​http://bugzilla.opendarwin.org/show_bug.cgi?id=6238

This is a somewhat cheesy change. Having to use PassRefPtr_Ref creates ambiguities
in assignment and copy construction. And this makes life way easier and removes
the need for pass(). It is not really correct, but we pretty much never need a real
const PassRefPtr, and this takes care of things for PassRefPtr temporaries.

• kjs/identifier.cpp: (KJS::Identifier::add): No more need for pass()
• kjs/property_map.cpp: (KJS::PropertyMap::addSparseArrayPropertiesToReferenceList): No more need for pass()
• kjs/ustring.cpp: (KJS::UString::Rep::create): Use adoptRef (KJS::UString::UString): No more need for pass (KJS::UString::append): No more need for pass (KJS::UString::substr): No more need for pass
• kxmlcore/PassRefPtr.h: made m_ptr mutable (ugh) (KXMLCore::PassRefPtr::PassRefPtr): Take a const PassRefPtr reference (KXMLCore::PassRefPtr::release): Made this a const method (ugh) (KXMLCore::PassRefPtr::operator=): clean up appropriately (KXMLCore::adoptRef): Added this to use instead of PassRefPtr<T>::adopt, I think it makes the behavior more clear and it is less verbose. (KXMLCore::static_pointer_cast): use adoptRef (KXMLCore::const_pointer_cast): use adoptRef
• kxmlcore/RefPtr.h: (KXMLCore::RefPtr::RefPtr): take const PassRefPtr& (KXMLCore::PassRefPtr::operator=): take const PassRefPtr&

File:

• trunk/JavaScriptCore/kjs/identifier.cpp (modified) (3 diffs)

trunk/JavaScriptCore/kjs/identifier.cpp

@@ -130 +130 @@
 {
     if (!c)
-        return pass(&UString::Rep::null);
+        return &UString::Rep::null;
     int length = strlen(c);
     if (length == 0)
-        return pass(&UString::Rep::empty);
+        return &UString::Rep::empty;
     
-    return pass(*identifierTable().insert<const char *, CStringTranslator>(c).first);
+    return *identifierTable().insert<const char *, CStringTranslator>(c).first;
 }
 
@@ -173 +173 @@
 {
     if (length == 0)
-        return pass(&UString::Rep::empty);
+        return &UString::Rep::empty;
     
     UCharBuffer buf = {s, length}; 
-    return pass(*identifierTable().insert<UCharBuffer, UCharBufferTranslator>(buf).first);
+    return *identifierTable().insert<UCharBuffer, UCharBufferTranslator>(buf).first;
 }
 
@@ -182 +182 @@
 {
     if (r->isIdentifier)
-        return pass(r);
+        return r;
 
     if (r->len == 0)
-        return pass(&UString::Rep::empty);
+        return &UString::Rep::empty;
 
     UString::Rep *result = *identifierTable().insert(r).first;
     if (result == r)
         r->isIdentifier = true;
-    return pass(result);
+    return result;
 }