TheAssemblyArmada / Vanilla-Conquer

Vanilla Conquer provides clean, cross-platform builds of the C&C Remastered Collection and the standalone legacy games.
Other
330 stars 51 forks source link

[TD] Music fails to auto-start in Multiplayer and Skirmish modes #729

Open Gerwin2k opened 2 years ago

Gerwin2k commented 2 years ago

Moved into a separate issue, originally mentioned in issue #719

For reference: In original C&C Gold music auto-starts when entering a multiplayer game. (Skirmish was not an available option in the original game.)

In Current VanillaTD:

1) Any of the multiplayer modes, in tiberiandawn\init.cpp after line 1110 it sets "Options.ScoreVolume = 0;" for these game modes.

2) Skirmish: it seems that music on Skirmish maps only auto-starts with certain map numbers. When I extract map 'Green Acres' SCM01EA and give it number SCM19EA, the music still auto-starts. Give it number SCM20EA or higher: no music. There are stock multiplayer maps that are numbered 20 or higher.

The logic that determines the music in skirmish is in line 90 of tiberiandawn\scenario.cpp

The "< 20" there limits the music to map scm19ea and lower. The song that gets chosen is 19 lines below: THEME_AOI = Act On Instinct.

        if (Scenario < 20 && (!Special.IsJurassic || !AreThingiesEnabled)) {
            if (Scenario != 1 || Whom == HOUSE_GOOD) {
                Play_Movie(IntroMovie);
            }
#ifndef REMASTER_BUILD
            if (Scenario > 1 || Whom == HOUSE_BAD) {
                if (briefing) {
                    PreserveVQAScreen = (Scenario == 1);
                    Play_Movie(BriefMovie);
                }
            }
#else
            if (briefing) {
                PreserveVQAScreen = (Scenario == 1);
                Play_Movie(BriefMovie);
            }
#endif
            Play_Movie(ActionMovie, TransitTheme);
            if (TransitTheme == THEME_NONE) {
                Theme.Queue_Song(THEME_AOI);
            }
        } else {
            Play_Movie(BriefMovie);
            Play_Movie(ActionMovie, TransitTheme);
giulianobelinassi commented 2 years ago

"For reference: In original C&C Gold music auto-starts when entering a multiplayer game. (Skirmish was not an available option in the original game.)"

No, it didn't. I am pretty sure that it was a v1.06 patch thing.

Still, I think this should be fixed nonetheless.

Gerwin2k commented 2 years ago

I did not consider the v1.06 unofficial patch was involved with this. My reference C&C gold is at v1.06 level indeed.

Nyerguds commented 2 years ago

The music was indeed disabled in multiplay originally. This was most likely for two reasons:

I was aware of the first issue, and that prompted me to re-enable it since people mostly have their music on hard disc these days anyway. The patch also enabled random music by default for multiplayer games.

Imagine my surprise when we played games online and noticed that our music was completely synchronised among all players.

And my horror, when I realised the full consequences of that... :facepalm:

I did not consider the v1.06 unofficial patch was involved with this. My reference C&C gold is at v1.06 level indeed.

Yea, you'd be surprised how many things can be seen as regressions when comparing to my patch instead of to the real original :stuck_out_tongue:

If you can't be bothered figuring out how to set up an actual original installation, you should at least double-check issues like this in the 1.06 changelog, so you can correctly mark them as improvements here. http://nyerguds.arsaneus-design.com/cnc95upd/cc95p106/patch106c_r3_en.html