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.

Sign up for GitHub

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

Jump to bottom

conditional breakpoints support #12

Closed
nulld opened this issue Oct 8, 2018 · 3 comments
Closed

conditional breakpoints support #12

nulld opened this issue Oct 8, 2018 · 3 comments
Assignees
@nulld

Comments

@nulld
Copy link
Member

nulld commented Oct 8, 2018 •
edited
Loading

example
@nulld nulld self-assigned this Oct 8, 2018
@nulld nulld closed this as completed Oct 8, 2018
@Gama11
Copy link
Member

Gama11 commented Oct 8, 2018

Is it necessary to prefix expressions with this for it to work (like in the watch panel)? If so, it seems like that shouldn't be required.

@nulld
Copy link
Member Author

nulld commented Oct 9, 2018

Yes, for now, it required. Good point, will think about it.

@nulld
Copy link
Member Author

nulld commented Oct 9, 2018

@Gama11 it works now without requiring this., but it's more performance draining solution. I'm not sure, what's the lesser of two evils... Can you test it for your project with two options? You can easily switch it here:

hxcpp-debugger/hxcpp-debug-server/hxcpp/debug/jsonrpc/Server.hx

Line 501 in 80ab728

interp = VariablesPrinter.initInterp(threadNumber, getTopFrame(), true);
with last bool parameter. true means include members to interpreter context.

@nulld nulld reopened this Oct 9, 2018
@nulld nulld closed this as completed Nov 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nulld nulld

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Gama11 @nulld