Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add to watch list fills in wrong variable name (in a different way) [fix included] #159

Closed
gigaj0ule opened this issue Apr 18, 2019 · 2 comments

Comments

@gigaj0ule
Copy link

gigaj0ule commented Apr 18, 2019

In the watch list, when inspecting a variable, "public" is always a member of the tree.

So you would see...

Grandfather.public.Parent.public.Child

Adding the variable to the watch list would cause an invalid expression error unless the .public's were removed manually. Changing one line in my gdb.js fixes this, from:

const varObjName = watch_${exp.replace(/\./g, '__').replace(/\[/g, '_').replace(/\]/g, '_')};
to:

const varObjName = watch_${exp.replace(/\./g, '__').replace(/\[/g, '_').replace(/\]/g, '_').replace(/\.(public)/g, '')};

@gigaj0ule gigaj0ule changed the title Add to watch list fills in wrong variable name (in a different way) Add to watch list fills in wrong variable name (in a different way) [fix included] Apr 18, 2019
Marus added a commit that referenced this issue May 15, 2019
…xes issues with it not being able to use certain expressions with reserved characters/string. Should fix issues #157 and #159
@Marus
Copy link
Owner

Marus commented May 15, 2019

I think this should be fixed by release v0.2.6 - which should be available in the marketplace shortly. Please confirm this fixes your issue.

@Marus
Copy link
Owner

Marus commented May 16, 2019

I'm going to mark this as closed - if you still have an issue please reopen the ticket.

@Marus Marus closed this as completed May 16, 2019
mayjs pushed a commit to mayjs/cortex-debug that referenced this issue Apr 6, 2022
…xes issues with it not being able to use certain expressions with reserved characters/string. Should fix issues Marus#157 and Marus#159
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

No branches or pull requests

2 participants