lr-mame_2003 Tab menu problem
-
@jonnykesh heh - me too... hours!
-
@andrewh I had a look on github and you seem to have identified the issue. As to how to fix it...
-
Should be an easy fix, although I'm no programmer so I'm not going to touch it...
At the start of ui_text.h we have one entry above the copyright section
but in ui_text.c we have two - presumably this then pushes every subsequent string out by one
-
@andrewh said in lr-mame_2003 Tab menu problem:
Should be an easy fix, although I'm no programmer so I'm not going to touch it...
At the start of ui_text.h we have one entry above the copyright section
but in ui_text.c we have two - presumably this then pushes every subsequent string out by one
Eek. This does look like the issue, and as the one whose been messing around in that file, I'd say I'm the one responsible. I'll try to put together a PR tomorrow or the next day but please if anyone else would like to go ahead sooner that would be fine with me.
-
@markwkidd Nice one Mark. I like the way there are warnings in both files that they have to match... Hahahaha! If I had a github account I would submit a PR but I'll leave it to you.
-
There are more problems after some of the recent changes - eg the text when launching galaga -
before:
after:
I am going to rollback the RetroPie libretro mame2003 for now. What is the purpose of the recent changes to catver etc? - I checked a few PR, but there is a lack of detail in there. Looks like there was limited testing of some of these changes :/
[edit] I think this problem is related - due to the wrong string offsets.
To help us help you - please make sure you read the sticky topics before posting - https://retropie.org.uk/forum/topic/3/read-this-first
-
@buzz I think the issue is the same. A line was omitted and it has displaced everything in the mame dialogues / menu.
-
@jonnykesh Yep - submitted a PR here - https://github.com/libretro/mame2003-libretro/pull/118
-
I originally put the ifdef back in, but then just removed the MESS string as I saw that all the other MESS strings have been removed. Why were the mess strings removed? I don't think MESS is hooked up in this core, but I don't see any benefit of removing just a few MESS related strings from this file.
To help us help you - please make sure you read the sticky topics before posting - https://retropie.org.uk/forum/topic/3/read-this-first
-
@buzz Hi Buzz, just updated from source and it's all fixed. Thank you very much. And twinaphex for merging it as well, of course.
Contributions to the project are always appreciated, so if you would like to support us with a donation you can do so here.
Hosting provided by Mythic-Beasts. See the Hosting Information page for more information.