Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#5702 closed defect (outdated)

Assertion in Audio::Timestamp::addIntern() with MI1 CD

Reported by: SF/doc_wagon Owned by: digitall
Priority: normal Component: Engine: SCUMM
Keywords: Cc:
Game: Monkey Island 1

Description

Hi,

I just recompiled ScummVM with today's SVN and now I'm hitting this assertion every time (reproducably) in Monkey 1 VGA CD (english):

scummvm_debug.exe!_wassert(const wchar_t * expr=0x011fb6fc, const wchar_t * filename=0x011fb6b0, unsigned int lineno=151) Zeile 335 C
> scummvm_debug.exe!Audio::Timestamp::addIntern(const Audio::Timestamp & ts={...}) Zeile 151 + 0x26 Bytes C++
scummvm_debug.exe!Audio::Timestamp::operator-(const Audio::Timestamp & ts={...}) Zeile 175 C++
scummvm_debug.exe!Audio::SubSeekableAudioStream::SubSeekableAudioStream(Audio::SeekableAudioStream * parent=0x034392b8, Audio::Timestamp start={...}, Audio::Timestamp end={...}, DisposeAfterUse::Flag disposeAfterUse=YES) Zeile 232 + 0x96 Bytes C++
scummvm_debug.exe!Audio::makeLoopingAudioStream(Audio::SeekableAudioStream * stream=0x034392b8, Audio::Timestamp start={...}, Audio::Timestamp end={...}, unsigned int loops=1) Zeile 160 + 0x62 Bytes C++
scummvm_debug.exe!DefaultAudioCDManager::play(int track=17, int numLoops=1, int startFrame=7173, int duration=0, bool only_emulate=false) Zeile 75 + 0x7f Bytes C++
scummvm_debug.exe!Scumm::Sound::playCDTrack(int track=17, int numLoops=1, int startFrame=7173, int duration=0) Zeile 1031 + 0x4b Bytes C++
scummvm_debug.exe!Scumm::Sound::playSound(int soundID=108) Zeile 338 C++
scummvm_debug.exe!Scumm::Sound::processSoundQueues() Zeile 139 C++
scummvm_debug.exe!Scumm::Sound::processSound() Zeile 125 + 0xf Bytes C++
scummvm_debug.exe!Scumm::ScummEngine::scummLoop_handleSound() Zeile 2390 C++
scummvm_debug.exe!Scumm::ScummEngine::scummLoop(int delta=6) Zeile 2171 + 0xf Bytes C++
scummvm_debug.exe!Scumm::ScummEngine::go() Zeile 1971 + 0x13 Bytes C++
scummvm_debug.exe!Scumm::ScummEngine::run() Zeile 486 C++
scummvm_debug.exe!runGame(const PluginSubclass<MetaEngine> * plugin=0x02550e58, OSystem & system={...}, const Common::String & edebuglevels={...}) Zeile 213 + 0xf Bytes C++
scummvm_debug.exe!scummvm_main(int argc=2, const char * const * argv=0x003f3c90) Zeile 423 + 0x14 Bytes C++
scummvm_debug.exe!SDL_main(int argc=2, char * * argv=0x003f3c90) Zeile 63 + 0xd Bytes C++
scummvm_debug.exe!WinMain(HINSTANCE__ * __formal=0x00400000, HINSTANCE__ * __formal=0x00400000, HINSTANCE__ * __formal=0x00400000, HINSTANCE__ * __formal=0x00400000) Zeile 47 + 0x12 Bytes C++
scummvm_debug.exe!__tmainCRTStartup() Zeile 263 + 0x2c Bytes C
scummvm_debug.exe!WinMainCRTStartup() Zeile 182 C
kernel32.dll!_BaseProcessStart@4() + 0x23 Bytes

these are the variables in question:

ts = {_secs=-96 _numFrames=1080 _framerate=3000 ...}
_framerate = 441000

apparently, something is wrong with the frame rate.

This is on Win32 with MSVC 2008 (everything self-compiled), and I'm using FLA audio since MP3 doesn't seem to work for me anymore.

If I click on "continue" everything works and the game plays normal from that point on (at least as far as I've played... about 5 minutes in)

I've uploaded a coredump here if that is of any help: cornelia.regengedanken.de/~mdrueing/temp/scummvm_debug_dump.zip

Ticket imported from: #3303041. Ticket imported from: bugs/5702.

Change History (7)

comment:1 by SF/doc_wagon, 8 years ago

typo in description -- fixed

comment:2 by SF/doc_wagon, 8 years ago

Summary: Assertion in Audio::Timestamp::addInternal() with MI1 CDAssertion in Audio::Timestamp::addIntern() with MI1 CD

comment:3 by SF/doc_wagon, 8 years ago

I probably should have mentioned where exactly that happens: As soon as I skip the intro by pressing "ESC" and before Guybrush enters the lookout.

comment:4 by digitall, 8 years ago

doc_wagon: Not sure if you are slightly confused. Development has now moved to Git. The latest code can be found here:
https://github.com/scummvm/scummvm
The SF.net SVN is now read only and kept only for reference.

If you are using the latest Git master code or branch-1-3-0, then there have been a number of changes to SCUMM Imuse today and this may be corrected with the latest so please retest.

Also, please attach the output of ./scummvm --version as it is unclear of the exact code revision otherwise.

comment:5 by SF/doc_wagon, 8 years ago

Status: newclosed

comment:6 by SF/doc_wagon, 8 years ago

You're right, I didn't notice. I don't follow the project that closely and I just did the usual "svn update -> compile" ritual.

Seems fixed in current git.

comment:7 by wjp, 8 years ago

Owner: set to digitall
Resolution: outdated
Note: See TracTickets for help on using tickets.