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

Hiscores not loading in some games #702

Closed
eadmaster opened this issue Mar 31, 2021 · 10 comments
Closed

Hiscores not loading in some games #702

eadmaster opened this issue Mar 31, 2021 · 10 comments

Comments

@eadmaster
Copy link

eadmaster commented Mar 31, 2021

One of these games is mysticri.

I've found this is due to this check that rely on the game driver to specify if the hiscore is supported or not.

Since new games are added constantly to the MAME datfile maybe it is better to disable the check?

Related issue: #220

@eadmaster
Copy link
Author

Another issue is this hard limit for the ranges.

There a few games in the current datfile with more than 20 ranges associated: boblbobl, bublboblp, cosmicg.

eadmaster pushed a commit to eadmaster/FBNeo-chvolmod that referenced this issue Mar 31, 2021
@barbudreadmon
Copy link
Collaborator

barbudreadmon commented Mar 31, 2021

Duplicate of finalburnneo#378 , and no we can't remove that check : you can break hiscores when resetting if HiscoreReset isn't called appropriately for each game with enabled hiscore.
I didn't know about the range part, thanks for the heads-up, will look into this.

barbudreadmon added a commit to finalburnneo/FBNeo that referenced this issue Mar 31, 2021
as pointed in libretro#702, some games require 50+ ranges
@eadmaster
Copy link
Author

eadmaster commented Mar 31, 2021

ok, so if i disable the check and call HiscoreReset() every time a game is reset it should be fine then?

@barbudreadmon
Copy link
Collaborator

yes

@eadmaster
Copy link
Author

Fixed in my libretro-only branch by calling HiscoreReset() in retro_reset().
I'll monitor the other issue for a better solution.

eadmaster pushed a commit to eadmaster/FBNeo-chvolmod that referenced this issue Apr 1, 2021
@barbudreadmon
Copy link
Collaborator

well, no, you didn't fix anything, you should read the other issue...

@eadmaster
Copy link
Author

eadmaster commented Apr 1, 2021

I've just tried resetting the libretro core with an hiscore loaded multiple times, and my .hi file stays intact, so i guess this fixes the problem with the libretro port, at least...

@barbudreadmon
Copy link
Collaborator

barbudreadmon commented Apr 2, 2021

As mentioned in the other issue, games can decide to reset themselves internally for various reasons, what you fixed is only manual reset. So that's not a fix, that's only a dirty hack that can only take care of the easy case.

@eadmaster
Copy link
Author

eadmaster commented Apr 2, 2021

Ok, i see your point now. I'm thinking another driver-agnostic solution now to detect a reset, i'll post in finalburnneo#378 after studying the code to see if it is doable.

@dinkc64
Copy link
Collaborator

dinkc64 commented Apr 2, 2021

dinkfucious say:
"It's best not to try to short-cut things, and do them the right way."

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

3 participants