From 0cfd0589432798838b66023b01a5ca6ba4fad0ca Mon Sep 17 00:00:00 2001 From: Tobia Tesan Date: Wed, 30 Mar 2016 17:16:41 +0200 Subject: WINTERMUTE: Correctly find .ogg version of .wav files As it was, it didn't reliably work across platforms because it turned some\\windows\\path.wav into some/system/pathogg Note no "." before "ogg"; also since we use the new filename to search for the file inside DCPs, which use Windows naming, we don't want system-specific path format. Fixes #7088 --- engines/wintermute/base/sound/base_sound_manager.cpp | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) (limited to 'engines/wintermute/base/sound/base_sound_manager.cpp') diff --git a/engines/wintermute/base/sound/base_sound_manager.cpp b/engines/wintermute/base/sound/base_sound_manager.cpp index f1e0c3b1f9..74c0086817 100644 --- a/engines/wintermute/base/sound/base_sound_manager.cpp +++ b/engines/wintermute/base/sound/base_sound_manager.cpp @@ -100,15 +100,14 @@ BaseSoundBuffer *BaseSoundMgr::addSound(const Common::String &filename, Audio::M BaseSoundBuffer *sound; Common::String useFilename = filename; + useFilename.toLowercase(); // try to switch WAV to OGG file (if available) - AnsiString ext = PathUtil::getExtension(filename); - if (StringUtil::compareNoCase(ext, "wav")) { - AnsiString path = PathUtil::getDirectoryName(filename); - AnsiString name = PathUtil::getFileNameWithoutExtension(filename); - - AnsiString newFile = PathUtil::combine(path, name + "ogg"); - if (BaseFileManager::getEngineInstance()->hasFile(newFile)) { - useFilename = newFile; + if (useFilename.hasSuffix(".wav")) { + Common::String oggFilename = useFilename; + oggFilename.erase(oggFilename.size() - 4); + oggFilename = oggFilename + ".ogg"; + if (BaseFileManager::getEngineInstance()->hasFile(oggFilename)) { + useFilename = oggFilename; } } -- cgit v1.2.3