[DynInst_API:] [dyninst/dyninst] 94252b: Clean up memoryTracker usage in binaryEdit (#877)

[noreply@xxxxxxxxxx]

  Branch: refs/heads/master
  Home:   https://github.com/dyninst/dyninst
  Commit: 94252b8ce87293da4b10d0cee550e7ad0507a3ae
      https://github.com/dyninst/dyninst/commit/94252b8ce87293da4b10d0cee550e7ad0507a3ae
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2021-11-28 (Sun, 28 Nov 2021)

  Changed paths:
    M dyninstAPI/src/binaryEdit.C
    M dyninstAPI/src/binaryEdit.h
    M dyninstAPI/src/codeRange.h

  Log Message:
  -----------
  Clean up memoryTracker usage in binaryEdit (#877)

* Store memoryTracker_ as a value instead of pointer

This provides cleaner semantics for its usage and ensures that the it's
default-constructed correctly.

* Code cleanup

* Give codeRange a virtual destructor

This will ensure that memoryTracker objects removed from a codeTree will
correctly get destroyed.

* Simplify inferiorFree

* Free any remaining memoryTrackers on destruction