[DynInst_API:] [dyninst/dyninst] 733022: Fix insertion operators in BPatch and Symtab (#1069)

[noreply@xxxxxxxxxx]

  Branch: refs/heads/master
  Home:   https://github.com/dyninst/dyninst
  Commit: 733022f78d7ee173e4503ec05fd6f8286eabcf7e
      https://github.com/dyninst/dyninst/commit/733022f78d7ee173e4503ec05fd6f8286eabcf7e
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2021-06-28 (Mon, 28 Jun 2021)

  Changed paths:
    M dyninstAPI/h/BPatch_basicBlock.h
    M symtabAPI/h/Aggregate.h
    M symtabAPI/h/Symbol.h
    M symtabAPI/h/Variable.h
    M symtabAPI/src/Aggregate.C
    M symtabAPI/src/Variable.C

  Log Message:
  -----------
  Fix insertion operators in BPatch and Symtab (#1069)

* SymtabAPI::Variable

* Provide an exported operator<< for BPatch_basicBlock

* Provide an exported operator<< for SymtabAPI::Symbol

* Fix operator<< in Symtab/Aggregate and Variable

The existing version never worked because they didn't have exported
symbols and, if they had, led to ambiguous overloads because both
operators were in the global namespace and Variable is a child of
Aggregate. Making operator<< a member function for both allows ADL to
disambiguate.