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

dyninstAPI documentation typo #41

Closed
pefoley2 opened this issue May 11, 2016 · 3 comments
Closed

dyninstAPI documentation typo #41

pefoley2 opened this issue May 11, 2016 · 3 comments
Assignees
@jdetter
Labels
bug
Milestone
9.2 Release

Comments

@pefoley2
Copy link
Contributor

In the description of the BPatch class, the documentation contains the api call
void setInstrStackFrame(bool) instead of void setInstrStackFrames(bool)
@wrwilliams wrwilliams added this to the 9.2 Release milestone May 12, 2016
@jdetter jdetter self-assigned this Jun 14, 2016
jdetter pushed a commit that referenced this issue Jun 14, 2016
Fixed documentation typo #41
e64ff00
@jdetter
Copy link
Contributor

jdetter commented Jun 14, 2016

Fixed by 73da5f1

@jdetter
Copy link
Contributor

jdetter commented Jun 14, 2016

VEX will hopefully be merged into master before the release of 9.2.X. If this is not the case, I will make a pull request to cherry pick this documentation fix into master for the release.

@jdetter
Copy link
Contributor

jdetter commented Jun 21, 2016

fixed by #102

@jdetter jdetter closed this as completed Jun 21, 2016
@wrwilliams wrwilliams added the bug label Apr 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jdetter jdetter

Labels
bug
Projects
None yet
Milestone
9.2 Release
Development

No branches or pull requests
3 participants
@pefoley2 @wrwilliams @jdetter