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

Jump to the correct source line as shown in list views #58

Merged
merged 1 commit into from
Jul 11, 2018

Conversation

schellingb
Copy link
Contributor

  • Applies to 'Called From' and 'Call Stacks' view where the line numbers are not the procedure entry points
  • Makes multiple entries of the same function in the 'Called From' jump to the correct source line
  • Keep the correct line number reference in the Back/Forward history
  • This also fixes viewing 64bit process profile .sleepy files in the 32bit version (avoid 64bit ptr to long cast)

This is mostly for convenience because I find it unintuitive to (single) click through the entries in the 'Called From' list and have it jump not to the line of the actual call. Fixing that also lead to fixing crashes when loading a 64bit process profile .sleepy file in the 32bit version.

- Applies to 'Called From' and 'Call Stacks' view where the line numbers are not the procedure entry points
- Makes multiple entries of the same function in the 'Called From' jump to the correct source line
- Keep the correct line number reference in the Back/Forward history
- This also fixes viewing 64bit process profile .sleepy files in the 32bit version (avoid 64bit ptr to long cast)
Copy link
Member

@CyberShadow CyberShadow left a comment

Choose a reason for hiding this comment

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

Thank you for these improvements. Unfortunately my Windows VM is currently inaccessible, but the source changes look good, and the CI is passing (for some reason it's not showing up here though), so I'll go ahead and merge these.

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