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