search

MiSTer-devel / PCXT_MiSTer

PCXT port for MiSTer by spark2k06.
GNU General Public License v3.0
13 stars 7 forks source link

TGA Lemmings crashes when music starts #49

Closed famiclone6502 closed 1 year ago

famiclone6502 commented 1 year ago

When starting a level on the TGA version of Lemmings (TGALEMMI.EXE), it does make sounds as the door opens, but as the first lemming drops and starts to play music, it hangs. I've tested with the latest version from 10/27. You must set it to Adlib invisible, otherwise it will detect and use it instead of Tandy music.

I've tested a number of configurations and they all hang at that same point. I'm using the same files as I do on my real Tandy, which does play properly.

spark2k06 commented 1 year ago

Sorry, I am unable to reproduce the issue...

https://youtu.be/HWUXOYfq_ck

famiclone6502 commented 1 year ago

Can you share your other menu settings? I will match your setup and try again.

spark2k06 commented 1 year ago

I downloaded two lemmings versions of the TDL file and I can't reproduce it... Is it possible that you downloaded it corrupted? Without further proof, I must close this issue...

image

spark2k06 commented 1 year ago

Apparently, the bug only occurs in the version "Lemmings (1991)(Psygnosis Limited)[Strategy, Action].zip".

To reproduce it, you need to hide the Adlib soundcard in the OSD menu and use this version of the game. So, what happens is that it crashes when using Tandy sound, but only with this version.

I'm marking it as a bug, pending to review.

spark2k06 commented 1 year ago

From the tests I have been able to do, everything seems to indicate that the problem is in the Tandy ROM that uses the core, 1.01.00... ROM that in any case we can't replace, as the other versions are not compatible with the core.

The fact that there is a later version of Lemmings for Tandy that works fine, suggests that it was made to correct this problem, among others possibly.

I therefore close the subject.

spark2k06 commented 1 year ago

I have finally found the cause:

https://github.com/MiSTer-devel/PCXT_MiSTer/commit/ed109209f755c121a5afa68bb31bd9247aba9de8

It will be fixed in the next release.