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

Lists need some conceptual changes #7

Open
Jatoxo opened this issue May 16, 2021 · 0 comments
Open

Lists need some conceptual changes #7

Jatoxo opened this issue May 16, 2021 · 0 comments
Labels
enhancement New feature or request rework Something is getting a rework

Comments

@Jatoxo
Copy link
Collaborator

Jatoxo commented May 16, 2021

For whatever reason, I thought it would be a genius idea if all objects that wanted to be displayed in a list inherited from a base ListItem class, that defines everything needed for something to be a list item. I didn't realize that this basically meant every object that I want to have represented in a list needs to be initialized.
So, making a list of programs will actually initialize every single program. Given that we're working in a performance-critical environment with very limited resources, this idea was quite brilliant.

I only noticed this flaw until a program was actually leaking memory when it was being initialized, resulting in the program list eventually crashing.

So yea, change that

@Jatoxo Jatoxo added enhancement New feature or request rework Something is getting a rework labels May 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request rework Something is getting a rework
Projects
None yet
Development

No branches or pull requests

1 participant