Opened 17 years ago

Closed 17 years ago

Last modified 17 years ago

#938 closed defect

FT: Intro sound regression

Reported by: eriktorbjorn Owned by: fingolfin
Priority: normal Component: Engine: SCUMM
Keywords: Cc:
Game: Full Throttle

Description

The Full Throttle intro doesn't work as well as it used to in recent CVS snapshots. There are numerous drop-outs in the sound, and at least one sound (I only watched the beginning of it) repeats.

I think it may have started with revision 1.14 of smush_mixer.cpp, where the buffer size was decreased from 2,000,000 to 100,000. If I turn on debugging output I get lots of "mixer full" messages. The messages don't go away completely even if I revert the patch, though I no longer hear any sound errors.

But I don't *want* to revert the patch, because I agree with the reasoning behind it. I just don't know why it doesn't work in reality.

Ticket imported from: #765800. Ticket imported from: bugs/938.

Change History (4)

comment:1 by eriktorbjorn, 17 years ago

Owner: set to fingolfin

comment:2 by fingolfin, 17 years ago

The problem seems to be that the wrap-around code is buggy / odd... will look into that.

comment:3 by fingolfin, 17 years ago

I increased the buffer size again some time ago, which avoids any problems, it seems.

comment:4 by fingolfin, 17 years ago

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