Opened 17 years ago

Closed 17 years ago

Last modified 17 years ago

#1341 closed defect

COMI : sound glitches on MSVC6 / EVC 3-4

Reported by: SF/arisme Owned by: SF/jamieson630
Priority: normal Component: Engine: SCUMM
Keywords: Cc:
Game: Monkey Island 3


Sounds quite horrible (like a bad sample rate) when compiled with Visual C 6 or Embedded Visual C 3/4 - surprinsingly the compiled daily build is fine (probably compiled with Visual C 7). Tested with the latest CVS (14th December 2003)

Assigning to Jamieson, I'm having a look at it too ...

Ticket imported from: #860067. Ticket imported from: bugs/1341.

Change History (8)

comment:1 by fingolfin, 17 years ago

The daily builds are created with mingw, a freely downloadable tool chain whihc you could use, too.

So it seems the audio system once again exposes one of the hundreds of bugs in MSVC6. I really wonder why we keep spending so much energy to support this pile of crap. In fact I wonder how MSVC is able to compile a simple "Hello World" considering how many problems we have exposed in it so far <sigh>

comment:2 by SF/arisme, 17 years ago

Perhaps because Embedded Visual C has the same kind of bugs and is the common environment to generate Pocket PC/Handheld PC executables ...

but I could try to switch to a gcc cross-compiler just to see how it performs on different platforms - I'm almost confident about ARM/CE support (even if EVC could make a better use of XScale extensions, I don't know ...), but not so sure about MIPS/CE or SH3/CE support, and I doubt the WinCE port users would like this new alpha test either :)

comment:3 by fingolfin, 17 years ago

&/$%. Why can't MS, like, release a bug fix update for EVC then, I wonder? <sigh>

comment:4 by SF/arisme, 17 years ago

Status: newpending

comment:5 by SF/arisme, 17 years ago

OK located, the bug lies in how template functions are managed - in our case, makeLinearInputStream and makeWrappedInputStream are not correctly generated with VC6 or EVC.

Suggestions :

* Copy-and-pasting (evil) to specialize the functions with <true> (it actually works ... no comments) * Put stereo as a parameter

Your choice Max :)

comment:6 by fingolfin, 17 years ago

I put a change into CVS which might fix this. Please try.

comment:7 by SF/arisme, 17 years ago

that fixed it, thanks

comment:8 by SF/arisme, 17 years ago

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