Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Move inference helper files out of the symbol resolver #1769

Closed
oxisto opened this issue Oct 2, 2024 · 0 comments · Fixed by #1777
Closed

Move inference helper files out of the symbol resolver #1769

oxisto opened this issue Oct 2, 2024 · 0 comments · Fixed by #1777
Assignees
@oxisto

Comments

@oxisto
Copy link
Member

oxisto commented Oct 2, 2024

Let's move them to a dedicated file, such as InferenceHelper.kt and clean up some of their naming, e.g. avoid names like createMethodDummies.
@oxisto oxisto self-assigned this Oct 2, 2024
oxisto added a commit that referenced this issue Oct 3, 2024
@oxisto
Cleanup of SymbolResolver
4e7c47b 
Unfortunatly, this cannot really be split into smaller pieces. It basically does the following:
- Move all inference related functions out of the symbol resolve
- Try to restrict / remove calls to the legacy `resolveReference` function

Fixes #1769
Fixes #1771
@oxisto oxisto mentioned this issue Oct 3, 2024
Merged
oxisto added a commit that referenced this issue Oct 3, 2024
@oxisto
Cleanup of SymbolResolver
5523490 
Unfortunatly, this cannot really be split into smaller pieces. It basically does the following:
- Move all inference related functions out of the symbol resolve
- Try to restrict / remove calls to the legacy `resolveReference` function

Fixes #1769
Fixes #1771
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@oxisto oxisto

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Cleanup of SymbolResolver Fraunhofer-AISEC/cpg
1 participant
@oxisto