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

expose simple step_into() for SBThread() #32

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

roccoblues
Copy link
Contributor

This exposes the simple SBThreadStepInto with only one parameter as step_into() and the existing one as step_into3 (better name suggestion welcome, maybe step_into_until()?).

src/thread.rs Outdated Show resolved Hide resolved
src/thread.rs Outdated
}

#[allow(missing_docs)]
pub fn step_into3(
Copy link
Contributor

Choose a reason for hiding this comment

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

Your other suggestion for a name is better. What is the target name parameter actually for?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Quoting from https://lldb.llvm.org/python_api/lldb.SBThread.html#lldb.SBThread.StepInto:

Step the current thread from the current source line to the line given by end_line, stopping if the thread steps into the function given by target_name. If target_name is None, then stepping will stop in any of the places we would normally stop.

I had trouble getting this to work with default parameters. That's why I added the simple version. Maybe until and target_name should be Option<> and the method then calls the correct low-level method based on the provided params? But what do you call if only until is provided?

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.

2 participants