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

Remove or scale-down Forest class #392

Open
aionick opened this issue Apr 17, 2019 · 0 comments
Open

Remove or scale-down Forest class #392

aionick opened this issue Apr 17, 2019 · 0 comments

Comments

@aionick
Copy link
Contributor

aionick commented Apr 17, 2019

With the new ClassHierarchy the previous one (ClassHierarchyForest) likely can be heavily reduced in terms of functionality. The previous forest is currently only be used to compute the sizes of the classes and so it should just become the most mininal data structure we need in order to accomplish this task - the forest aspect is likely overkill now. If the forest aspect is required to compute these sizes then it probably makes more sense to just move this capacity into the new hierarchy and entirely remove the old one.

@aionick aionick self-assigned this Apr 17, 2019
@aionick aionick removed their assignment May 1, 2019
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

No branches or pull requests

1 participant