[DynInst_API:] [dyninst/dyninst] be2521: Remove unused member variables


Date: Thu, 10 Aug 2023 14:42:02 -0700
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] be2521: Remove unused member variables
  Branch: refs/heads/thaines/symtab_lineinformation_cleanup
  Home:   https://github.com/dyninst/dyninst
  Commit: be25211ad3ada89ad39761535c97b3ae113e1a50
      https://github.com/dyninst/dyninst/commit/be25211ad3ada89ad39761535c97b3ae113e1a50
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2023-08-10 (Thu, 10 Aug 2023)

  Changed paths:
    M symtabAPI/h/LineInformation.h
    M symtabAPI/src/LineInformation.C

  Log Message:
  -----------
  Remove unused member variables

These were added by 0dee1c8f8 in 2016, but never used.


  Commit: 09becfee0005f027277c19d13800facfdb7b3ba8
      https://github.com/dyninst/dyninst/commit/09becfee0005f027277c19d13800facfdb7b3ba8
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2023-08-10 (Thu, 10 Aug 2023)

  Changed paths:
    M symtabAPI/h/LineInformation.h

  Log Message:
  -----------
  Remove redundant visibility specifiers


  Commit: 1df36da7ec185d07c2bac2b18cb172fb7fe80239
      https://github.com/dyninst/dyninst/commit/1df36da7ec185d07c2bac2b18cb172fb7fe80239
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2023-08-10 (Thu, 10 Aug 2023)

  Changed paths:
    M symtabAPI/h/LineInformation.h
    M symtabAPI/src/LineInformation.C

  Log Message:
  -----------
  Remove useless comments

These are either incorrect or just noise.


  Commit: 5cf7eb7e472f9f89f28602ca5237560979a4a6b2
      https://github.com/dyninst/dyninst/commit/5cf7eb7e472f9f89f28602ca5237560979a4a6b2
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2023-08-10 (Thu, 10 Aug 2023)

  Changed paths:
    M symtabAPI/h/LineInformation.h

  Log Message:
  -----------
  Make the class uninheritable

This appears to be some remnants of someone's scratch/dev work.


  Commit: d4cf21bcea6bbd0d6a144fbe58800f23edb5bf45
      https://github.com/dyninst/dyninst/commit/d4cf21bcea6bbd0d6a144fbe58800f23edb5bf45
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2023-08-10 (Thu, 10 Aug 2023)

  Changed paths:
    M symtabAPI/h/LineInformation.h
    M symtabAPI/src/LineInformation.C

  Log Message:
  -----------
  Use a default dtor

No reason to have it.


  Commit: 81ff3a35b24c74e6feced91003b83ddb3b3fbe79
      https://github.com/dyninst/dyninst/commit/81ff3a35b24c74e6feced91003b83ddb3b3fbe79
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2023-08-10 (Thu, 10 Aug 2023)

  Changed paths:
    M symtabAPI/h/LineInformation.h
    M symtabAPI/src/LineInformation.C

  Log Message:
  -----------
  Reduce string copies

These parameters are only ever read from.


Compare: https://github.com/dyninst/dyninst/compare/be25211ad3ad%5E...81ff3a35b24c
[← Prev in Thread] Current Thread [Next in Thread→]
  • [DynInst_API:] [dyninst/dyninst] be2521: Remove unused member variables, Tim Haines <=