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

Odroid don't start when adding more than 50+ games in Fav #59

Closed
EduOrtegaDEV opened this issue Feb 28, 2020 · 3 comments
Closed

Odroid don't start when adding more than 50+ games in Fav #59

EduOrtegaDEV opened this issue Feb 28, 2020 · 3 comments
Assignees
Labels
Enhancement New feature or request Feature Request

Comments

@EduOrtegaDEV
Copy link

EduOrtegaDEV commented Feb 28, 2020

Describe the bug
Adding more than 50 games among various systems (GB, GBC, Lynx, etc.) as favorites make the system collapse, and you turn the Odroid on, the screen blinks and won't start.

To Reproduce
Steps to reproduce the behavior:

  1. Add more than 50 games as favorites among various systems.
  2. Turn off, turn on.
  3. See error

Screenshots
Se the problem here:
https://drive.google.com/open?id=1kmSb3esOcrFU1r19Sz5f8Qrqv0MoPxFZ

And see the list of favorites here:
https://drive.google.com/open?id=1O1t0ZEkmNgozieETxDyL-dOvj-BGeL2t

Odroid go (please complete the following information):

  • Version v2.2

Additional context
Once I opened the esp-32-txt file and deleted some of the games from the list, the system booted up correctly.

@32teeth
Copy link
Member

32teeth commented Feb 28, 2020

@EduardoOrtegaME

Thanks for the heads up.
This was by design

    free(FAVORITES);
    FAVORITES = (char**)malloc((50) * sizeof(void*));

We can change this.
What would you consider a tangible number?

or

Should we add a safeguard to limit to 50 favorites?

@32teeth 32teeth self-assigned this Feb 28, 2020
@32teeth 32teeth added Enhancement New feature or request Feature Request labels Feb 28, 2020
@EduOrtegaDEV
Copy link
Author

Oh didn't know that... if this is an intended behavior maybe once we reach this limit the add to favorite feature shouldn't do anything

@obsidianspider
Copy link

If this is by design (performance limitations, etc.) then I think that when the limit is reached the user should be informed or as suggested, that no new favorites can be added. This would be more elegant than the system failing to boot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request Feature Request
Projects
None yet
Development

No branches or pull requests

3 participants