#13198 closed defect (fixed)
SHERLOCK: Intro too slow in the German Version since version 2.5.1
Reported by: | Commodusaurel | Owned by: | eriktorbjorn |
---|---|---|---|
Priority: | high | Component: | Engine: Sherlock |
Version: | Keywords: | ||
Cc: | Game: | Sherlock Holmes: Case of the Serrated Scalpel |
Description (last modified by )
Hello. Since version 2.5.1 the intro of "The Lost Files of Sherlock Holmes: The Case of the Serrated Scalpel" (German version) is to slow / OS Win 8.1
Change History (13)
comment:1 by , 3 years ago
Description: | modified (diff) |
---|
comment:2 by , 3 years ago
Summary: | Intro since 2.5.1 in German Version to slow → SHERLOCK: Intro too slow in the German Version since version 2.5.1 |
---|
comment:3 by , 3 years ago
comment:4 by , 3 years ago
Actually, it seems I can reproduce it with the English version, if I selected a small installation. (I have two English versions, and this option was only available for one of them. It reduces the size of the data files from ~28 MB to ~13 MB.) I'll see if I can figure out what caused that, and hopefully it will apply to the German version as well.
comment:5 by , 3 years ago
The slowdown was introduced when we started decoding frames on demand, rather than in advance, to fix a crash at the lab table. See https://github.com/scummvm/scummvm/commit/046e39428bae509f087bc0ffc25b1f925f565435
If I understand correctly, the intro animation does lots of reads from the same resource file, and for each of them it has to decompress the entire resource file. So while each image is decoded in a few milliseconds, that adds up to a lot.
comment:6 by , 3 years ago
Owner: | set to |
---|---|
Resolution: | → fixed |
Status: | new → closed |
I've committed a fix which I hope will take care of it. It should be in the next nightly build, and in 2.5.2 if that's ever released. Let me know if it works for you. If not, I'll reopen the bug report.
comment:7 by , 3 years ago
The problem is only partially fixed. The intro of Sherlock runs flawlessly with Soundblaster audio, but with Roland MT-32 audio there are severe sound crackles.
comment:8 by , 3 years ago
Resolution: | fixed |
---|---|
Status: | closed → new |
Ok, I'll reopen it for now. But this time I'm afraid that I can't reproduce the problem with my English version. I don't hear any crackling with the MT-32 emulator, and CPU usage stays at reasonable levels throughout even with a 4x graphics scaler. (Even more so if I compile it in release mode instead of debug mode.)
And this is on a computer that I bought around... 2015, I think? So it's hardly a powerhouse by today's standards.
comment:9 by , 3 years ago
Since build of January 9th. I have the MT-32 problem in almost every game. In 2.5.1 or Daily Build Installer from December 27th was everything OK
comment:10 by , 3 years ago
Sounds like the MT-32 problem should be its own separate bug report then, since the fix I made for the intro slowdown was very much limited to the Sherlock engine.
comment:11 by , 3 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
I'm closing this again, since the intro slowdown is fixed. If the MT-32 problem (which can't be related to this since you say it affects multiple game engines, not just Sherlock) persists, could you please file a new bug report about that?
comment:12 by , 3 years ago
I'm waiting for an updated Daily Build Windows Installer. The current one is from December 27th, although January 8th is behind the download. I think the MT-32 sound problem comes from copying the ScummVM daily build manually. Thanks for the fix.
comment:13 by , 3 years ago
With the latest Daily Build Installer, Sherlock works great with MT-32 Music and Intro Thanks again
I don't have the German version, but just to be clear... it ran at the correct speed in 2.5.0, but not 2.5.1? If so, that seems strange to me because I'm not aware of any changes in 2.5.1 that would affect that.