Opened 16 years ago

Closed 16 years ago

Last modified 16 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 Changed 16 years ago by eriktorbjorn

Owner: set to fingolfin

comment:2 Changed 16 years ago by fingolfin

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

comment:3 Changed 16 years ago by fingolfin

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

comment:4 Changed 16 years ago by fingolfin

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