Revert of [Courgette] Clean up Disassembler; fix ELF Memory leaks.

courgette/disassembler_elf_32_arm.h

Index: courgette/disassembler_elf_32_arm.h
diff --git a/courgette/disassembler_elf_32_arm.h b/courgette/disassembler_elf_32_arm.h
index 5dc6897f47c03bc89eeca252a3527bd50b70cbf6..17ebb2557530749218510bacf18e8ded45553dbb 100644
--- a/courgette/disassembler_elf_32_arm.h
+++ b/courgette/disassembler_elf_32_arm.h
@@ -8,10 +8,9 @@
 #include <stddef.h>
 #include <stdint.h>
 
-#include <map>
-
 #include "base/macros.h"
 #include "courgette/disassembler_elf_32.h"
+#include "courgette/memory_allocator.h"
 #include "courgette/types_elf.h"
 
 namespace courgette {
@@ -31,60 +30,51 @@
   class TypedRVAARM : public TypedRVA {
    public:
     TypedRVAARM(ARM_RVA type, RVA rva) : TypedRVA(rva), type_(type) { }
-    ~TypedRVAARM() override { }
-
-    // TypedRVA interfaces.
-    CheckBool ComputeRelativeTarget(const uint8_t* op_pointer) override;
-    CheckBool EmitInstruction(AssemblyProgram* program,
-                              RVA target_rva) override;
-    uint16_t op_size() const override;
 
     uint16_t c_op() const { return c_op_; }
 
+    virtual CheckBool ComputeRelativeTarget(const uint8_t* op_pointer);
+
+    virtual CheckBool EmitInstruction(AssemblyProgram* program,
+                                      RVA target_rva);
+
+    virtual uint16_t op_size() const;
+
    private:
     ARM_RVA type_;
-    uint16_t c_op_;  // Set by ComputeRelativeTarget().
+
+    uint16_t c_op_;  // set by ComputeRelativeTarget()
     const uint8_t* arm_op_;
   };
 
-  DisassemblerElf32ARM(const void* start, size_t length);
+  explicit DisassemblerElf32ARM(const void* start, size_t length);
 
-  ~DisassemblerElf32ARM() override { }
+  virtual ExecutableType kind() { return EXE_ELF_32_ARM; }
 
-  // DisassemblerElf32 interfaces.
-  ExecutableType kind() const override { return EXE_ELF_32_ARM; }
-  e_machine_values ElfEM() const override { return EM_ARM; }
+  virtual e_machine_values ElfEM() { return EM_ARM; }
 
-  // Takes an ARM or thumb opcode |arm_op| of specified |type| and located at
-  // |rva|, extracts the instruction-relative target RVA into |*addr| and
-  // encodes the corresponding Courgette opcode as |*c_op|.
-  //
-  // Details on ARM opcodes, and target RVA extraction are taken from
-  // "ARM Architecture Reference Manual", section A4.1.5 and
-  // "Thumb-2 supplement", section 4.6.12.
-  // ARM_OFF24 is for the ARM opcode. The rest are for thumb opcodes.
   static CheckBool Compress(ARM_RVA type,
                             uint32_t arm_op,
                             RVA rva,
                             uint16_t* c_op /* out */,
                             uint32_t* addr /* out */);
 
-  // Inverse for Compress(). Takes Courgette op |c_op| and relative address
-  // |addr| to reconstruct the original ARM or thumb op |*arm_op|.
   static CheckBool Decompress(ARM_RVA type,
                               uint16_t c_op,
                               uint32_t addr,
                               uint32_t* arm_op /* out */);
 
  protected:
-  // DisassemblerElf32 interfaces.
-  CheckBool RelToRVA(Elf32_Rel rel,
-                     RVA* result) const override WARN_UNUSED_RESULT;
-  CheckBool ParseRelocationSection(const Elf32_Shdr* section_header,
-                                   AssemblyProgram* program)
-      override WARN_UNUSED_RESULT;
-  CheckBool ParseRel32RelocsFromSection(const Elf32_Shdr* section)
-      override WARN_UNUSED_RESULT;
+
+  virtual CheckBool RelToRVA(Elf32_Rel rel, RVA* result)
+    const WARN_UNUSED_RESULT;
+
+  virtual CheckBool ParseRelocationSection(
+      const Elf32_Shdr *section_header,
+        AssemblyProgram* program) WARN_UNUSED_RESULT;
+
+  virtual CheckBool ParseRel32RelocsFromSection(
+      const Elf32_Shdr* section) WARN_UNUSED_RESULT;
 
 #if COURGETTE_HISTOGRAM_TARGETS
   std::map<RVA, int> rel32_target_rvas_;