[DynInst_API:] [dyninst/dyninst] a9801c: Remove Symtab::addRegion(Region*) (#2184)

[Tim Haines <noreply@xxxxxxxxxx>]

  Branch: refs/heads/master
  Home:   https://github.com/dyninst/dyninst
  Commit: a9801cf729a701aebd0031a547d358734065b1f5
      https://github.com/dyninst/dyninst/commit/a9801cf729a701aebd0031a547d358734065b1f5
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2026-03-30 (Mon, 30 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*) (#2184)

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