[Courgette] Simple AssemblyProgram and Disassembler cleanups.

courgette/disassembler_elf_32_x86_unittest.cc

 // Copyright (c) 2011 The Chromium 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 "courgette/disassembler_elf_32_x86.h"
 
 #include <ctype.h>
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 71 matching lines @@
   EXPECT_EQ(file1.length(), disassembler->length());
 
   const uint8_t* offset_p = disassembler->FileOffsetToPointer(0);
   EXPECT_EQ(reinterpret_cast<const void*>(file1.c_str()),
             reinterpret_cast<const void*>(offset_p));
   EXPECT_EQ(0x7F, offset_p[0]);
   EXPECT_EQ('E', offset_p[1]);
   EXPECT_EQ('L', offset_p[2]);
   EXPECT_EQ('F', offset_p[3]);
 
-  std::unique_ptr<AssemblyProgram> program(new AssemblyProgram(EXE_ELF_32_X86));
+  std::unique_ptr<AssemblyProgram> program(

[grt (UTC plus 2), 2017/01/12 11:56:41]

  auto program = base::MakeUnique<...>(...);
(may as well do line 72, too)

[huangs, 2017/01/12 19:54:42]

Done.

+      new AssemblyProgram(EXE_ELF_32_X86, 0));
 
   EXPECT_TRUE(disassembler->Disassemble(program.get()));
 
   const std::vector<RVA>& abs32_list = disassembler->Abs32Locations();
 
   // Flatten the list typed rel32 to a list of rel32 RVAs.
   std::vector<RVA> rel32_list;
   rel32_list.reserve(disassembler->Rel32Locations().size());
   for (auto& typed_rel32 : disassembler->Rel32Locations())
     rel32_list.push_back(typed_rel32->rva());
@@ skipping 26 matching lines @@
 }
 
 }  // namespace
 
 TEST_F(DisassemblerElf32X86Test, All) {
   TestExe("elf-32-1", 200, 3337);
   TestExe("elf-32-high-bss", 0, 4);
 }
 
 }  // namespace courgette