Opened 16 years ago

Closed 16 years ago

Last modified 16 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

Description

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 Changed 16 years ago by fingolfin

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 Changed 16 years ago by SF/arisme

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 Changed 16 years ago by fingolfin

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

comment:4 Changed 16 years ago by SF/arisme

Status: newpending

comment:5 Changed 16 years ago by SF/arisme

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 Changed 16 years ago by fingolfin

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

comment:7 Changed 16 years ago by SF/arisme

that fixed it, thanks

comment:8 Changed 16 years ago by SF/arisme

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