Changeset 239427 in webkit for trunk/Source/JavaScriptCore/runtime/JSString.cpp

Timestamp:

Dec 19, 2018, 8:41:11 PM (7 years ago)

Author:

Chris Dumez

Message:

wtf/Optional.h: move-constructor and move-assignment operator should disengage the value being moved from
​https://bugs.webkit.org/show_bug.cgi?id=192728
<rdar://problem/46746779>

Reviewed by Geoff Garen.

Source/JavaScriptCore:

• API/*:
• Scripts/*:
• assembler/*:
• b3/*:
• bytecode/*:
• bytecompiler/*:
• debugger/*:
• dfg/*:
• ftl/*:
• heap/*:
• inspector/*:
• jit/*:
• llint/*:
• parser/*:
• runtime/*:
• tools/*:
• wasm/*:
• yarr/*:

Source/WebCore:

• Modules/*:
• animation/*:
• bindings/*:
• crypto/*:
• css/*:
• dom/*:
• editing/*:
• fileapi/*:
• html/*:
• inspector/*:
• layout/*:
• loader/*:
• mathml/*:
• page/*:
• platform/*:
• plugins/*:
• rendering/*:
• testing/*:
• workers/*:
• xml/*:

Source/WebCore/PAL:

• pal/*:

Source/WebDriver:

• :

Source/WebKit:

• NetworkProcess/*:
• Platform/*:
• Scripts/*:
• Shared/*:
• UIProcess/*:
• WebProcess/*:

Source/WebKitLegacy/mac:

• DOM/*:
• Plugins/*:
• WebCoreSupport/*:
• WebView/*:

Source/WebKitLegacy/win:

• Plugins/*:
• WebCoreSupport/*:

Source/WTF:

Update optional's move-constructor and move-assignment operator to disengage the value being moved from.
Rename to optional to Optional, make_optional() to makeOptional(), and move class from std to WTF namespace.

Based on patch by David Kilzer.

• wtf/*:

Tools:

• DumpRenderTree/*:
• MiniBrowser/*:
• TestRunnerShared/*:
• TestWebKitAPI/*:
• WebGPUAPIStructure/*:
• WebKitTestRunner/*:

File:

• trunk/Source/JavaScriptCore/runtime/JSString.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/runtime/JSString.cpp

@@ -445 +445 @@
     }
     
-    std::optional<uint32_t> index = parseIndex(propertyName);
+    Optional<uint32_t> index = parseIndex(propertyName);
     if (index && index.value() < length()) {
         descriptor.setDescriptor(getIndex(exec, index.value()), PropertyAttribute::DontDelete | PropertyAttribute::ReadOnly);