Replace SmartArrayPointer<T> with unique_ptr<T[]>

test/cctest/compiler/test-operator.cc

 // Copyright 2013 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+#include <memory>
 #include <sstream>
 
 #include "src/compiler/operator.h"
 #include "test/cctest/cctest.h"
 
 namespace v8 {
 namespace internal {
 namespace compiler {
 
 #define NONE Operator::kNoProperties
@@ skipping 45 matching lines @@
   CHECK(!op1a.Equals(&op2b));
   CHECK(!op1b.Equals(&op2a));
   CHECK(!op1b.Equals(&op2b));
 
   CHECK(!op2a.Equals(&op1a));
   CHECK(!op2a.Equals(&op1b));
   CHECK(!op2b.Equals(&op1a));
   CHECK(!op2b.Equals(&op1b));
 }
 
-
-static v8::base::SmartArrayPointer<const char> OperatorToString(Operator* op) {
+static std::unique_ptr<const char[]> OperatorToString(Operator* op) {
   std::ostringstream os;
   os << *op;
-  return v8::base::SmartArrayPointer<const char>(StrDup(os.str().c_str()));
+  return std::unique_ptr<const char[]>(
+      const_cast<const char*>(StrDup(os.str().c_str())));

[Igor Sheludko, 2016/07/25 08:56:44]

Unnecessary const_cast and probably constness of a returned array.

[jochen (gone - plz use gerrit), 2016/07/25 09:14:48]

done

 }
 
 
 TEST(TestOperator_Print) {
   Operator op1a(19, NONE, "Another1", 0, 0, 0, 0, 0, 0);
   Operator op1b(19, FOLD, "Another2", 2, 0, 0, 2, 0, 0);
 
   CHECK_EQ(0, strcmp("Another1", OperatorToString(&op1a).get()));
   CHECK_EQ(0, strcmp("Another2", OperatorToString(&op1b).get()));
 
@@ skipping 192 matching lines @@
   CHECK_EQ(33, op.ControlInputCount());
 
   CHECK_EQ(44, op.ValueOutputCount());
   CHECK_EQ(55, op.EffectOutputCount());
   CHECK_EQ(66, op.ControlOutputCount());
 }
 
 }  // namespace compiler
 }  // namespace internal
 }  // namespace v8