Skip to content

Improve FollowXUntilHit#2796

Draft
morbitzer wants to merge 3 commits into
mainfrom
improveFollowX
Draft

Improve FollowXUntilHit#2796
morbitzer wants to merge 3 commits into
mainfrom
improveFollowX

Conversation

@morbitzer

Copy link
Copy Markdown
Collaborator

FollowXUntil hit can take a very long time / infinite when recursion is involved.
Also it should be more efficient, e.g. by caching already searched paths.

PR also contains a fix for the PointsToPass that improves handling of PMVs during recursion

@codecov

codecov Bot commented Jun 19, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 69.56522% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.18%. Comparing base (aeb76d2) to head (b0acad4).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...lin/de/fraunhofer/aisec/cpg/passes/PointsToPass.kt 69.56% 3 Missing and 4 partials ⚠️

❌ Your patch check has failed because the patch coverage (69.56%) is below the target coverage (75.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
Files with missing lines Coverage Δ
...lin/de/fraunhofer/aisec/cpg/passes/PointsToPass.kt 84.93% <69.56%> (+0.13%) ⬆️

... and 4 files with indirect coverage changes

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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