Check that SceneManager actually exists before cleaning dangling sounds

[sercero]

This fix is for #15, I thought that using mSoundManager->setSceneManager(nullptr); fixed the problem but it still shows when using the debugger.

So I made this change that is not perfect because it is calling getName() on a SceneManager that might not exist but it works even if the SceneManager does not exist.

With this change it is not even needed to use the setSceneManager(nullptr) workaround.

[paroj]

I cannot reproduce the crash neither with boost threads nor with the non-threaded version. Maybe this was only introduced by the std thread port?

[sercero]

I cannot reproduce the crash neither with boost threads nor with the non-threaded version. Maybe this was only introduced by the std thread port?

This happens with the current master branch.

You have to do something like this to reproduce the issue:

    mSceneMgr->clearScene();
    mRoot->destroySceneManager(mSceneMgr);

    // mSoundManager->setSceneManager(nullptr);

    //Remove ourself as a Window listener
    Ogre::WindowEventUtilities::removeWindowEventListener(mWindow, this);
    windowClosed(mWindow);
    mRoot->shutdown();

    delete mRoot;

The issue is that is it possible to destroy the SceneManager that OgreOggSound is using before terminating the plugin.

[paroj]

does 346407fd3eb8ef3d194a364e989432ea18cd82ee also work for you?

[sercero]

Hello @paroj, yes that changed fixed it.

Thanks.

[paroj]

replaced by #18