[DynInst_API:] [dyninst/dyninst] 272db7: Fix responsibility inversion in Module::finalizeRa...


Date: Thu, 31 Aug 2023 08:52:54 -0700
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] 272db7: Fix responsibility inversion in Module::finalizeRa...
  Branch: refs/heads/thaines/module_findRanges_inversion
  Home:   https://github.com/dyninst/dyninst
  Commit: 272db77ff2e1441447dace2c8745932a19698b67
      https://github.com/dyninst/dyninst/commit/272db77ff2e1441447dace2c8745932a19698b67
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
    M symtabAPI/h/Module.h
    M symtabAPI/src/Module.C
    M symtabAPI/src/Symtab.C

  Log Message:
  -----------
  Fix responsibility inversion in Module::finalizeRanges

A Module shouldn't modify the Symtab object to which it belongs. It
knows what ranges belong to it (see
Object::fix_global_symbol_modules_static_dwarf), so it can munge them
into a collection of `ModRange`s.


[← Prev in Thread] Current Thread [Next in Thread→]
  • [DynInst_API:] [dyninst/dyninst] 272db7: Fix responsibility inversion in Module::finalizeRa..., Tim Haines <=