Skip to content

Conversation

jreineckearm
Copy link
Contributor

@jreineckearm jreineckearm commented Oct 15, 2025

Another attempt to fix #400 .

  • Array names shouldn't be put into square brackets, only their children.
  • Identify array entry indexes by being numbers if varobj parent not present.
  • Additionally check parent varobj to be an array for array children. May be redundant but reduces diff to previous code for this commit.

Thanks to @thorstendb-ARM for the initial investigation of this in #390 and #391 !

Review of the functionality uncovered a few other oddities to address. See #460 .
However, those are not critical to unblock our use of the extension we aim to use in our solution. Hence, they can be addressed separately and step by step.

Array names shouldn't be put into square brackets, only their children.
Identify array entry indexes by being numbers if varobj parent not present.
Additionally check parent varobj to be an array for array children. May be redundant but reduces diff to previous code for this commit.

Signed-off-by: Jens Reinecke <[email protected]>
Expanding test program for future tests.
Tweaked test names.

Signed-off-by: Jens Reinecke <[email protected]>
Signed-off-by: Jens Reinecke <[email protected]>
@jreineckearm jreineckearm marked this pull request as ready for review October 17, 2025 11:53
@jreineckearm
Copy link
Contributor Author

Appreciate it only makes a start on testing and leaves room for breaking something. But without it we'll never get to a more robust and complete solution.

Copy link
Contributor

@thorstendb-ARM thorstendb-ARM left a comment

Choose a reason for hiding this comment

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

Tested with RTOS Viewer / Arm:RTX5, works

@jreineckearm jreineckearm merged commit 2628bb3 into eclipse-cdt-cloud:main Oct 20, 2025
4 checks passed
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.

Evaluation of variables to support RTOS Views extension

2 participants