Skip to content

adding commands to change output radix to Live Watch#990

Merged
JonatanAntoni merged 3 commits into
mainfrom
addGlobalRadixChangeToLiveWatch
May 13, 2026
Merged

adding commands to change output radix to Live Watch#990
JonatanAntoni merged 3 commits into
mainfrom
addGlobalRadixChangeToLiveWatch

Conversation

@omarArm
Copy link
Copy Markdown
Collaborator

@omarArm omarArm commented May 11, 2026

Fixes

Changes

Screenshots

Screenshot 2026-05-12 at 12 49 33 Screenshot 2026-05-12 at 12 49 15

Checklist

  • 🤖 This change is covered by unit tests (if applicable).
  • 🤹 Manual testing has been performed (if necessary).
  • 🛡️ Security impacts have been considered (if relevant).
  • 📖 Documentation updates are complete (if required).
  • 🧠 Third-party dependencies and TPIP updated (if required).

@qltysh
Copy link
Copy Markdown

qltysh Bot commented May 11, 2026

Qlty


Coverage Impact

This PR will not change total coverage.

🚦 See full report on Qlty Cloud »

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

@omarArm omarArm changed the title adding commands to change output radix to liveWatch adding commands to change output radix to Live Watch May 11, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds Live Watch context-menu entries to switch the global output radix during a debug session, aligning Live Watch behavior with existing radix controls (per issue #975).

Changes:

  • Contributes Live Watch context menu actions to set output radix to Decimal or Hex.
  • Adds when-clause gating so the actions only appear in debug mode and depend on the current radix value.

Comment thread package.json Outdated
Comment thread package.json Outdated
@JonatanAntoni
Copy link
Copy Markdown
Member

@omarArm, please check the Copilot remarks. Could you provide a screenshot as a reference where this feature appears in the ui?

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@omarArm
Copy link
Copy Markdown
Collaborator Author

omarArm commented May 12, 2026

@JonatanAntoni done. The feature is added to the UI of the LiveWatch. It has already been added to the watch window, variables window and source window in a separate CDT Cloud PR. Available here eclipse-cdt-cloud/cdt-gdb-vscode#217

@JonatanAntoni JonatanAntoni merged commit 47c8280 into main May 13, 2026
21 checks passed
@JonatanAntoni JonatanAntoni deleted the addGlobalRadixChangeToLiveWatch branch May 13, 2026 09:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants