libretro / scummvm

ScummVM main repository
https://www.scummvm.org
GNU General Public License v3.0
7 stars 9 forks source link

Riven cursor problem #22

Closed KamiLordus closed 1 year ago

KamiLordus commented 1 year ago

Riven (DVD Windows) - the cursor has a large black square

Bez tytułu

RA 1.15 Win 11 Pro

spleen1981 commented 1 year ago

Were those working before?

KamiLordus commented 1 year ago

Blade Runner was still working a day/two ago (100% sure) Unfortunately, I can't say about the rest. Riven and Myst - Masterpiece were last launched a month ago.

Edit: I can also add: Myst and Riven definitely work without bugs on 2.6.1.4-7 (the only older version I have and checked).

spleen1981 commented 1 year ago

ok, it's a regression due to the latest Dreamweb fix, 5f93be7d97bcb768338e7faca9bb4768c6fb5f87 reverts that, Myst and Blade Runner should be working now. 7d6f2ced5037e43e0f245cdedb07d4e1bc9edc2d is the new fix for Dreamweb. Riven cursor looks like a separate issue.

KamiLordus commented 1 year ago

I tested the latest version 3a132f72 Dremweb works very well Blade Runner image is back Myst Masterpice - works very well.

I'll edit the post and leave only Riven Thank you very much for the bug fix.

i30817 commented 1 year ago

Cursor in riven and myst and 11th hour and starship titanic was a old bug that also existed the diablodiab core.

https://github.com/diablodiab/libretro-scummvm-backend/issues?q=is%3Aissue+myst+is%3Aclosed

not sure if the same thing. Anyway, the cause is explained in the issue linked with a patch to the problem (then).

i30817 commented 1 year ago

Opps wrong link. It's: https://github.com/diablodiab/libretro-scummvm-backend/issues/3

i30817 commented 1 year ago

Mmm. No i don't think it's the same anymore. The patch is already applied to libretro-os.cpp and of the games i mentioned, only riven has the problem.

KamiLordus commented 1 year ago

@i30817 In version 2.6.1.4-7 of 02/26/2023, Riven has no cursor bug. I'm pretty sure it wasn't in later 2.7.0 git either. The bug appeared in some version 2.8.0.

@spleen1981 we go back to version 75051b55 without the Dremweb fix?

spleen1981 commented 1 year ago

we go back to version https://github.com/libretro/scummvm/commit/75051b551db89b5a9ede3f35b75fcf49cd3483fe without the Dremweb fix?

Check the previous commit https://github.com/libretro/scummvm/commit/7d6f2ced5037e43e0f245cdedb07d4e1bc9edc2d, that's the fix. Then there is also a separate PR pending for that engine, regarding how titles are displayed. https://github.com/scummvm/scummvm/pull/4848

asciibeats commented 1 year ago

Blade Runner works again for me when i strip down the function splashScreen() in engines/engine.cpp to:

void splashScreen() {
  g_system->showOverlay();
  g_system->updateScreen();
  g_system->hideOverlay();

  splash = true;
}
asciibeats commented 1 year ago

Apologies. Did not read the comment :+1:

i30817 commented 1 year ago

I'm not even sure if i should mention this, but there is a random crash on the longest journy, although it takes a long time to trigger and i'm not sure if it's not upstream too. Reloading the last save prevents it.

Apologies because i closed the command console where it happened. It mentioned a 'linear allocator' failure. I didn't get a stacktrace.

I also noticed small graphics bugs on the game although those are minor.

KamiLordus commented 1 year ago

@i30817 Maybe try opening a separate ticket. Some time ago I tried Longest Journey and so far the graphical errors are quite disturbing (for example, a character surrounded by a rainbow of colors).

i30817 commented 1 year ago

There are 4 graphical errors i noticed (of course, this is all in software renderer mode, because i'm playing the scummvm libretro core, which might affect things).

The 'linear allocator' thing i didn't manage to trigger in debug, but it usually takes a long time playing so i'm not sure if it wouldn't be possible. It's a 'tinygl' thing, a library that scummvm uses to abstract opengl 'platform specific' work.

KamiLordus commented 1 year ago

Today I turned on Riven and noticed that the problem with the cursor disappeared. So I'm closing the ticket. I tested both CD and DVD versions. Bez tytułu