Branch: refs/heads/thaines/cleanup_MachRegister
  Home:   https://github.com/dyninst/dyninst
  Commit: ca7f5ced6156d0c7f037630cba44054b002141d4
      
https://github.com/dyninst/dyninst/commit/ca7f5ced6156d0c7f037630cba44054b002141d4
  Author: Tim Haines <thaines.as...@gmail.com>
  Date:   2023-10-28 (Sat, 28 Oct 2023)

  Changed paths:
    M common/src/registers/MachRegister.C

  Log Message:
  -----------
  Add missing switch breaks in size()


  Commit: df01524d099c6f1c812de6469014b62adfe88716
      
https://github.com/dyninst/dyninst/commit/df01524d099c6f1c812de6469014b62adfe88716
  Author: Tim Haines <thaines.as...@gmail.com>
  Date:   2023-10-28 (Sat, 28 Oct 2023)

  Changed paths:
    M common/src/registers/MachRegister.C

  Log Message:
  -----------
  Add break after assert in case statement


  Commit: ff92ca0d8a60560e8454163b5ffbc362e1f72696
      
https://github.com/dyninst/dyninst/commit/ff92ca0d8a60560e8454163b5ffbc362e1f72696
  Author: Tim Haines <thaines.as...@gmail.com>
  Date:   2023-10-28 (Sat, 28 Oct 2023)

  Changed paths:
    M common/src/registers/MachRegister.C

  Log Message:
  -----------
  Move assert into default case in 'size'

This is semantically equivalent, but fixes the 'incomplete without
default' linter warning.


  Commit: 6be212f12eec4ba76a66d08a556491d5b59b94d1
      
https://github.com/dyninst/dyninst/commit/6be212f12eec4ba76a66d08a556491d5b59b94d1
  Author: Tim Haines <thaines.as...@gmail.com>
  Date:   2023-10-28 (Sat, 28 Oct 2023)

  Changed paths:
    M common/src/registers/MachRegister.C

  Log Message:
  -----------
  Add error messages in getROSERegister

I don't know if it's correct to assert in these conditions. This is
semantically equivalent, fixes the 'incomplete without default' linter
warning, and avoids asserting.


  Commit: 7e46e45d6cfcc89eba4ed944f2c995c70fe4da28
      
https://github.com/dyninst/dyninst/commit/7e46e45d6cfcc89eba4ed944f2c995c70fe4da28
  Author: Tim Haines <thaines.as...@gmail.com>
  Date:   2023-10-28 (Sat, 28 Oct 2023)

  Changed paths:
    M common/src/registers/MachRegister.C

  Log Message:
  -----------
  Return InvalidRegister in DwarfEncToReg

This makes them all consisent and fixes the 'incomplete without default'
linter warning.


  Commit: 57da13aee730d994e6f6e218289d1dac4dade57c
      
https://github.com/dyninst/dyninst/commit/57da13aee730d994e6f6e218289d1dac4dade57c
  Author: Tim Haines <thaines.as...@gmail.com>
  Date:   2023-10-28 (Sat, 28 Oct 2023)

  Changed paths:
    M common/src/registers/MachRegister.C

  Log Message:
  -----------
  Remove dead code

I left the commented-out switch cases. I'm not sure why they aren't
used.


  Commit: ea474bb0acb0cfa03aac8b62879292caaabdcec9
      
https://github.com/dyninst/dyninst/commit/ea474bb0acb0cfa03aac8b62879292caaabdcec9
  Author: Tim Haines <thaines.as...@gmail.com>
  Date:   2023-10-28 (Sat, 28 Oct 2023)

  Changed paths:
    M common/src/registers/MachRegister.C

  Log Message:
  -----------
  Remove useless asserts


  Commit: 2d6d57fdc53db9dd8a35fb43cc40540850f4e24f
      
https://github.com/dyninst/dyninst/commit/2d6d57fdc53db9dd8a35fb43cc40540850f4e24f
  Author: Tim Haines <thaines.as...@gmail.com>
  Date:   2023-10-28 (Sat, 28 Oct 2023)

  Changed paths:
    M common/h/registers/MachRegister.h

  Log Message:
  -----------
  Remove 'init_names' declaration

It was never implemented.


Compare: 
https://github.com/dyninst/dyninst/compare/ca7f5ced6156%5E...2d6d57fdc53d
_______________________________________________
Dyninst-api mailing list
Dyninst-api@cs.wisc.edu
https://lists.cs.wisc.edu/mailman/listinfo/dyninst-api

Reply via email to