Skip to content

Conversation

@oltolm
Copy link
Contributor

@oltolm oltolm commented Nov 22, 2025

I made the code in mgwhelp.cpp use the dwarfstack library. I had to fork it and change its API. All tests are green.

Dwarfstack fixes this problem #97 and also some other small problems. The code has good quality and I fixed a performance problem that I found.

@codecov
Copy link

codecov bot commented Nov 22, 2025

Codecov Report

❌ Patch coverage is 85.00000% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 50.93%. Comparing base (5dc71ed) to head (0e66e8d).

Files with missing lines Patch % Lines
src/mgwhelp/dwarf_find.cpp 79.31% 4 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #98      +/-   ##
==========================================
- Coverage   51.11%   50.93%   -0.18%     
==========================================
  Files          15       15              
  Lines        2160     1820     -340     
  Branches      824      671     -153     
==========================================
- Hits         1104      927     -177     
+ Misses        809      691     -118     
+ Partials      247      202      -45     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jrfonseca
Copy link
Owner

Sorry for the silence. I haven't had much time to sit down at my computer these past weekends.

Thanks. I think this is a good direction.

Ideally we'd use a git submodule for dwarfstack. If we need modifications, then use our own fork, though ideally we'd upstream those. If we really need a fork long term, then I might need to promote drmingw to a GitHub organization to better organize those forks. In the meanwhile, you can just point to your own dwarfstack fork.

Could you please provide the diff of your dwarfstack changes? I'd like to understand what are these changes.

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