Opened 2 years ago

Closed 2 years ago

#13743 closed defect (fixed)

TOON: Toonstruck music not playing 2.6.0

Reported by: Scapetti Owned by: antoniou79
Priority: normal Component: Engine: Toon
Version: Keywords:
Cc: Game: Toonstruck

Description

It worked fine in the previous stable release

Change History (10)

comment:1 by antoniou79, 2 years ago

Please provide more details on this issue.
Which platform are you playing on? (Windows, Linux etc)
Which version of the game do you have? (GOG, Steam, original boxed release etc)

I've just re-tested with the final 2.6.0 stable release on Windows 10 x64 with the GOG version of the game and I do get music in-game.

Please make sure you have not accidentally muted music or lowered its volume way down from the in-game menu (F1) or the ScummVM in-game menu (typically brought up with Ctrl-F5)

comment:2 by dwatteau, 2 years ago

I think I can reproduce this with 2.7.0git1616-g3a9f80451bc on macOS 12.5 x64.

I had both the English and French versions (from GOG), and it seems unlikely that I would have muted the music by accident in both (I also don't mute anything in my global settings).

Here's what I had by default when starting the game:

[toon]
engineid=toon
gui_saveload_last_pos=0
platform=pc
guioptions=sndNoMIDI lang_English
description=Toonstruck (DOS/English)
path=/path/to/TOONSTRUCK_EN
gameid=toon
language=en

and to get any music, I had to press F1 and check the SFX and music buttons, which changed my config to this:

[toon]
engineid=toon
gui_saveload_last_pos=0
platform=pc
guioptions=sndNoMIDI lang_English
description=Toonstruck (DOS/English)
path=/path/to/TOONSTRUCK_EN
gameid=toon
language=en
music_mute=false
sfx_mute=false

i.e. music_mute=true and sfx_mute=true act like they're the defaults? Although toon.cpp seems to say otherwise.

Last edited 2 years ago by dwatteau (previous) (diff)

comment:3 by antoniou79, 2 years ago

Ah, you're right. I've added the game anew, and music and sfx are muted by default.

I'll debug to see what's going on. At least I can reproduce it so it should be easy to fix (and a possible bugfix release ie. 2.6.1 will get it).

comment:4 by Scapetti, 2 years ago

Thanks guys, sorry I didn't provide more details, I was in a rush but decided to post anyway in case I forgot. Glad you figured it out!

By the way, something completely aside. I was surprised that Escape from Monkey Island still isn't playable in this stable release despite the pre-warning. Was it intentionally still disabled in this release? Just to check, thanks!

comment:5 by antoniou79, 2 years ago

I think I got it. By mistake I didn't use a variable when setting the mute state of the sound types, and in this particular case, when no additional custom volume setting was done, they were getting muted.

The fix was committed here:
https://github.com/scummvm/scummvm/commit/e55490a8ba3976bb5f4c97566846d9421098d2a0

and it's cherrypicked for the 2.6 branch too.

in reply to:  4 comment:6 by antoniou79, 2 years ago

Replying to Scapetti:

By the way, something completely aside. I was surprised that Escape from Monkey Island still isn't playable in this stable release despite the pre-warning. Was it intentionally still disabled in this release? Just to check, thanks!

Yes, I believe it's intentional, but I'm not exactly certain what's keeping it. Probably some known issues (?) I know there are a few people who mentioned path finding issues with Guybrush at some locations. Anyway, this is not something to discuss in this ticket.

comment:7 by Scapetti, 2 years ago

No problem, thanks for the information!

comment:8 by antoniou79, 2 years ago

Component: AudioEngine: Toon
Owner: set to antoniou79
Resolution: fixed
Status: newpending

comment:9 by aquadran, 2 years ago

Summary: Toonstruck music not playing 2.6.0TOON: Toonstruck music not playing 2.6.0

comment:10 by antoniou79, 2 years ago

Status: pendingclosed
Note: See TracTickets for help on using tickets.