[Solved]small line problem

deshan

25-02-2010 11:25:40

Hi sticky
I have just downloaded latest svn trunk and found small error in
bool OgreOggSoundManager::attachEffectToSound

error is
Error 1 error C2664: 'OgreOggSound::OgreOggSoundManager::_attachEffectToSoundImpl' : cannot convert parameter 1 from 'const std::string' to 'OgreOggSound::OgreOggISound *' d:\config\ogreoggsound(02-25)\trunk\src\ogreoggsoundmanager.cpp 1137

after changing return _attachEffectToSoundImpl(sName,... to return _attachEffectToSoundImpl(sound,
It complied fine. Not really a big issue.

And another thing I havent found any prebuild libs in svn download.

stickymango

25-02-2010 13:51:17

Ah bugger thanks for that, I'll fix it.

I removed the prebuilt libs mainly because I couldn't decide which versions to supply :lol: Up until recently I was still using VC8 for my projects, but have now upgraded to vc9, but also there's the OGRE version transitioning period at the moment, I mainly still use OGRE 1.6.2!! :shock: but obviously there are updates to this, as well as 1.7 RC, so I thought I'd just let the user build against whatever version they currently have.. :wink:

stickymango

25-02-2010 14:16:46

Sorted.

deshan

24-03-2010 04:00:53

Hi sticky
I may be wrong but found problem compiling in non boost mode in OgreOggSoundManager in current svn(checked today). Following are the things i did to make it compile
some places in the code missed to wrap like following
#if OGGSOUND_THREADED
mNoLock = ...;
#endif

lines around
2265
2259
2144
2135
2112

And lines around 2112 and 2149
change #ifdef OGGSOUND_THREADED
to #if OGGSOUND_THREADED

stickymango

24-03-2010 09:21:17

Thanks for the heads up, I shall check it out...

Fixed.

kornerr

03-04-2010 14:33:22

Patch for some typo in the last revision:
Index: src/ogreoggsoundmanager.cpp
===================================================================
--- src/ogreoggsoundmanager.cpp (revision 301)
+++ src/ogreoggsoundmanager.cpp (working copy)
@@ -231,7 +231,7 @@
return false;
}
alcGetIntegerv(device, ALC_MAJOR_VERSION, sizeof(majorVersion), &majorVersion);
- error = alcGetError();
+ error = alcGetError(device);
if (error != ALC_NO_ERROR)
{
LogManager::getSingleton().logMessage("Unable to get OpenAL Major Version number", Ogre::LML_CRITICAL);

kornerr

04-04-2010 10:31:51

The patch from "Linux support" thread contains this typo fix.

stickymango

06-04-2010 10:08:21

Thanks, sorted.