Changeset 155262 in webkit for trunk/Source/JavaScriptCore/runtime

Timestamp:

Sep 7, 2013, 3:15:57 PM (12 years ago)

Author:

[email protected]

Message:

Get rid of PassOwnArrayPtr
​https://bugs.webkit.org/show_bug.cgi?id=120964

Reviewed by Andreas Kling.

Source/JavaScriptCore:

Use OwnArrayPtr instead of PassOwnArrayPtr.

• bytecompiler/BytecodeGenerator.cpp:

(JSC::BytecodeGenerator::BytecodeGenerator):

• runtime/SymbolTable.h:

(JSC::SharedSymbolTable::setSlowArguments):

Source/WebCore:

Use OwnArrayPtr instead of PassOwnArrayPtr.

• html/canvas/WebGLRenderingContext.cpp:
• platform/Length.cpp:

(WebCore::newCoordsArray):
(WebCore::newLengthArray):

• platform/Length.h:
• platform/audio/FFTFrame.h:
• platform/graphics/GraphicsContext3D.cpp:
• platform/graphics/GraphicsContext3D.h:
• platform/graphics/texmap/TextureMapperGL.cpp:
• platform/graphics/wince/SharedBitmap.cpp:

Source/WebKit2:

Remove unneeded includes.

• Shared/ImmutableArray.h:
• WebProcess/InjectedBundle/InjectedBundle.cpp:

Source/WTF:

Make OwnArrayPtr a proper move-only type and get rid of PassOwnArrayPtr.

Ultimately I'd like to get rid of OwnArrayPtr as well and just have it be a specialization
of OwnPtr<T[]>, but this is a step in the right direction.

• GNUmakefile.list.am:
• WTF.pro:
• WTF.vcxproj/WTF.vcxproj:
• WTF.vcxproj/WTF.vcxproj.filters:
• WTF.xcodeproj/project.pbxproj:
• wtf/Assertions.cpp:
• wtf/CMakeLists.txt:
• wtf/DateMath.h:
• wtf/FastBitVector.h:
• wtf/Forward.h:
• wtf/OwnArrayPtr.h:
• wtf/PassOwnArrayPtr.h: Removed.

File:

• trunk/Source/JavaScriptCore/runtime/SymbolTable.h (modified) (1 diff)

trunk/Source/JavaScriptCore/runtime/SymbolTable.h

@@ -473 +473 @@
     // 0 if we don't capture any arguments; parameterCount() in length if we do.
     const SlowArgument* slowArguments() { return m_slowArguments.get(); }
-    void setSlowArguments(PassOwnArrayPtr<SlowArgument> slowArguments) { m_slowArguments = slowArguments; }
+    void setSlowArguments(OwnArrayPtr<SlowArgument> slowArguments) { m_slowArguments = std::move(slowArguments); }
 
     DECLARE_EXPORT_INFO;