Community
Participate
Working Groups
CDI-GDB allows users edit the content of the register groups. This is not necessarily very practical but for feature equivalence, DSF-GDB should implement it.
My users like the idea of being able to create manual register groups to better control what registers are being looked at. As an improvement to the register-grouping dialog used by CDI, it was suggested to use in-view multi-selection. That way, the user would select all registers that should be in a group, and then right-click -> "Add to new group" To make the UI simple, such a new group would be given a default name. It would be nice if the user could modify that name in-line in the view, but if not, the CDI "Edit register group" UI could be enough. Just like CDI, we feel that group persistence should be per launch (unlike the Expressions view, which has its persistence per workspace, although I'm not sure why). If I don't get any comments against it, I'll work on a multi-select solution and post the result for review.
The first draft of this functionality is ready for review: see https://git.eclipse.org/r/#/c/13980/ Manual test has been done along the implementation, Junit testing is still pending for a later patch set.
I've pushed a patch based on Alvaro's feature that moves the register group actions to the command framework: https://git.eclipse.org/r/30984
(In reply to Marc Khouzam from comment #3) > I've pushed a patch based on Alvaro's feature that moves the register group > actions to the command framework: > https://git.eclipse.org/r/30984 We decided to merge this second review into Alvaro's original one, so that comments can be addressed on top of the change to the command framework but yet, in the same review as the comments were input. So all changes are now part of: https://git.eclipse.org/r/#/c/13980/
This feature is now merged to master ! Thanks a lot Marc K. for all your valuable comments, dedication and contribution to this feature. I will update the N&N before marking this bug as fixed.
New and Noteworthy for CDT 8.6 have been updated see: https://wiki.eclipse.org/CDT/User/NewIn86