[DynInst_API:] [dyninst/dyninst] 5e5bee: Remove Symtab::addRegion(Region*)

[Tim Haines <noreply@xxxxxxxxxx>]

  Branch: refs/heads/thaines/remove_symtab_addRegion
  Home:   https://github.com/dyninst/dyninst
  Commit: 5e5beee2093ee4386920b0940e9de53d061fb7a7
      https://github.com/dyninst/dyninst/commit/5e5beee2093ee4386920b0940e9de53d061fb7a7
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-03-28 (Sat, 28 Mar 2026)

  Changed paths:
    M symtabAPI/doc/API/Symtab/Symtab.tex
    M symtabAPI/h/Symtab.h
    M symtabAPI/src/Symtab.C

  Log Message:
  -----------
  Remove Symtab::addRegion(Region*)

The following are not equivalent:

  Symtab::addRegion(Region::createRegion(<args>));

  Symtab::addRegion(<args>);

where '<args>' is the big list of variables for the Region constructor.

This is confusing and could easily cause incorrect behavior. The second
form isn't used anywhere in Dyninst. It is documented, but I can't
imagine that users are using it.



To unsubscribe from these emails, change your notification settings at https://github.com/dyninst/dyninst/settings/notifications