Changeset 193623 in webkit

Timestamp:

Dec 7, 2015 5:49:41 AM (8 years ago)

Author:

ddkilzer@apple.com

Message:

TestNetscapePlugIn: Address review comment on r193607
<​http://webkit.org/b/151881>

• DumpRenderTree/TestNetscapePlugIn/PluginObject.cpp:

(testSetStatus): Initializing std::unique_ptr<char[]> with
nullptr is redundant.

Location:

trunk/Tools

Files:

• ChangeLog (modified) (1 diff)
• DumpRenderTree/TestNetscapePlugIn/PluginObject.cpp (modified) (1 diff)

trunk/Tools/ChangeLog

@@ +1 @@
+2015-12-07  David Kilzer  <ddkilzer@apple.com>
+
+        TestNetscapePlugIn: Address review comment on 193607
+        <http://webkit.org/b/151881>
+
+        * DumpRenderTree/TestNetscapePlugIn/PluginObject.cpp:
+        (testSetStatus): Initializing std::unique_ptr<char[]> with
+        nullptr is redundant.
+
 2015-12-07  Carlos Garcia Campos  <cgarcia@igalia.com>
 

trunk/Tools/DumpRenderTree/TestNetscapePlugIn/PluginObject.cpp

@@ -975 +975 @@
 static bool testSetStatus(PluginObject* obj, const NPVariant* args, uint32_t argCount, NPVariant* result)
 {
-    std::unique_ptr<char[]> message(nullptr);
+    std::unique_ptr<char[]> message;
     if (argCount && NPVARIANT_IS_STRING(args[0])) {
         NPString statusString = NPVARIANT_TO_STRING(args[0]);