SCI: SQ3 German Amiga crashes on audio init

[sluicebox]

SQ3 German Amiga fails an assertion on load. I went back a couple ScummVM versions with same results.

It looks to me like the SCI version detection is coming up with the wrong values, though I don't know much about SCI versions, so maybe they're right and it's just a weird game.

detectMapVersion(): kResVersionSci1Middle (seems wrong)
detectVolVersion(): kResVersionSci0Sci1Early (seems right)
detectSciVersion(): SCI_VERSION_1_MIDDLE (seems wrong, a result of the map version)

SCI Viewer and SCI Companion can read the resources successfully.

This is the final version of SQ3. It's dated June 1991, two years after English Amiga and six months after German DOS.

[sluicebox]

The map/vol/sci version detection looks to be correct. The problem is occurring during audio initialization. Disabling audio allows the loading to get farther before a different error occurs, #10926, fix pending. Once that's merged it will load with audio disabled. Updating ticket title.

[sluicebox]

​https://github.com/scummvm/scummvm/pull/1562

MidiDriver_AmigaMac is attempting to determine the platform based on the existence of resources. SQ3 German Amiga happens to contain a patch 7 resource, which the code treats as proof of Macintosh, and uses that instead of the correct patch 5.

There's no need for this component to try to figure out the platform, it's already known, so now it uses the known value.

Tested against all my Amiga and Mac versions.

[Filippos Karapetis <bluegr@…>]

In 3209bfb7:

SCI: Remove MidiDriver_AmigaMac platform inference

Tell MidiDriver_AmigaMac the game's platform instead of it
attempting to infer this by which patch resources exist.

Fixes bug #10925 where SQ3 German Amiga is treated as Mac because
it happens to contain a patch 7 resource.