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.

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

Refactoring context (WIP) #3782

Merged
merged 21 commits into from
Dec 12, 2022
Merged

Conversation

jlerbsc
Copy link
Collaborator

@jlerbsc jlerbsc commented Dec 12, 2022

This PR is the first part of a larger change to better integrate the JP core module and the SymbolSolver module. The first objective of this PR is to support the creation of contexts no longer by the JavaParserFactory class but by the expressions themselves. It therefore introduces changes that will cause a break in the APIs used by project users.

…TypeExtractor class so as not to have to systematically declare a ReflectionTypeSolver, adapt unit tests accordingly and classes that do not support LazyType
…on. The goal of removing dependencies with the javaparser-symbol-solver-core module is to allow the TypeExtractor class to be migrated to the JP core module with the ultimate goal of letting nodes create their own context.
…flexionClassDeclaration (symbol solver module)
… SymbolSolver class because not only is it more logical for this functionality to be implemented in SymbolSolver, but also this modification makes it possible to remove class dependencies with the JavaParserFacade class which is specific to the SymbolSolver module.
@jlerbsc jlerbsc merged commit 9a2a75a into javaparser:master Dec 12, 2022
@jlerbsc jlerbsc added this to the next release milestone Dec 12, 2022
@jlerbsc jlerbsc added the PR: Added A PR that introduces new behaviour (e.g. functionality, tests) label Dec 12, 2022
@jlerbsc jlerbsc modified the milestones: next release, 4.0.0 Dec 12, 2022
@jlerbsc jlerbsc changed the title Refactoring context Refactoring context (WIP) Dec 12, 2022
@jlerbsc jlerbsc modified the milestones: 4.0.0, next release Jan 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: Added A PR that introduces new behaviour (e.g. functionality, tests)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant