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

Use the raw_lines field in stackprof for more precise lines #483

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

Use the raw_lines field in stackprof for more precise lines #483

wants to merge 2 commits into from

Commits on Oct 11, 2024

  1. Use the raw_lines field in stackprof for more precise lines 

    Since tmm1/stackprof#213 the actual source line within
the method, not just the method start line, should be available.
    @dalehamel
    dalehamel committed Oct 11, 2024
    Configuration menu
    Copy the full SHA
    5175aa1 View commit details
    Browse the repository at this point in the history

  2. Include lineno in stackprof key to avoid frame collisions 

    The stackprof frame key is updated to include the frame id in the upper
48 bits, and line number in the lower 16 bits.

To prevent truncation when casting to a number, this resulting BigInt
is converted to a string.

This ensures that frames with the same start line but different call lines
do not collide when deduping frames.
    @dalehamel
    dalehamel committed Oct 11, 2024
    Configuration menu
    Copy the full SHA
    136dd99 View commit details
    Browse the repository at this point in the history