Skip to content

Make frames with different files distinct for Firefox profiler categories#144

Draft
joshuay03 wants to merge 1 commit intomainfrom
fix-133
Draft

Make frames with different files distinct for Firefox profiler categories#144
joshuay03 wants to merge 1 commit intomainfrom
fix-133

Conversation

@joshuay03
Copy link
Collaborator

An attempt at fixing #133.

Need to debug the test failures...


bool operator==(const Frame& lhs, const Frame& rhs) noexcept {
return lhs.frame == rhs.frame && lhs.line == rhs.line;
return lhs.frame == rhs.frame && lhs.file == rhs.file && lhs.line == rhs.line;
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This is the core of the fix. See a2660a7 for some context.

@joshuay03 joshuay03 moved this to In Progress / Pending Review in Open Source Mar 31, 2025
@joshuay03 joshuay03 force-pushed the fix-133 branch 4 times, most recently from 335162b to b4ccd62 Compare June 3, 2025 11:59
@joshuay03 joshuay03 force-pushed the fix-133 branch 2 times, most recently from cdf432d to 3216295 Compare October 2, 2025 10:53
@joshuay03 joshuay03 changed the title Only reuse frames if file (and therefore category) is similar Make frames with different files distinct for Firefox profiler categories Oct 2, 2025
@joshuay03 joshuay03 force-pushed the fix-133 branch 2 times, most recently from 8a4297e to dc422e9 Compare October 2, 2025 11:11
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.

1 participant