Skip to content

Generics Cleanup#34

Merged
schweikart merged 9 commits intodevelopfrom
feat/generics-cleanup
Jul 29, 2023
Merged

Generics Cleanup#34
schweikart merged 9 commits intodevelopfrom
feat/generics-cleanup

Conversation

@Elscrux
Copy link
Copy Markdown
Member

@Elscrux Elscrux commented Jun 30, 2023

Attempt to fix most of the issues with raw generics.
SubRoutinePool still has issues with casting to the right ProblemControllerProvider. If you have an idea how to fix that, let me know @schweikart.

@Elscrux Elscrux requested a review from schweikart June 30, 2023 11:01
Comment thread src/main/java/edu/kit/provideq/toolbox/meta/MetaSolver.java
@Component
public class MetaSolverFeatureModelAnomaly extends MetaSolver<FeatureModelAnomalySolver> {
public class MetaSolverFeatureModelAnomaly extends MetaSolver<
edu.kit.provideq.toolbox.featuremodel.anomaly.FeatureModelAnomalyProblem,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can you use an import for this??

Copy link
Copy Markdown
Member

@schweikart schweikart Jul 28, 2023

Choose a reason for hiding this comment

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

The same thing applies to line 29, can you change this there as well?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yeah, sorry I'm blind

@schweikart
Copy link
Copy Markdown
Member

Can you rebase onto develop?

@Elscrux Elscrux force-pushed the feat/generics-cleanup branch from e00206d to c392cc5 Compare July 28, 2023 12:38
@Elscrux
Copy link
Copy Markdown
Member Author

Elscrux commented Jul 28, 2023

@schweikart I rebased the branch

@Elscrux Elscrux force-pushed the feat/generics-cleanup branch from 6c85d6e to 8a5926c Compare July 28, 2023 13:27
@schweikart schweikart merged commit b3beeb9 into develop Jul 29, 2023
@schweikart schweikart mentioned this pull request Sep 28, 2023
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