Make the view-manager and sound-manager independent.

[flightgear.git] / src / Sound / fg_fx.cxx

diff --git a/src/Sound/fg_fx.cxx b/src/Sound/fg_fx.cxx
index 94b7f6b98a5f84f1d2526e7b8d4ae0d3b5263222..b9cd492e4a6bf0d0c757bc60daa0dbb0de85c2cf 100644 (file)
--- a/src/Sound/fg_fx.cxx
+++ b/src/Sound/fg_fx.cxx
@@ -53,7 +53,6 @@ FGFX::FGFX ( SGSoundMgr *smgr, const string &refname, SGPropertyNode *props ) :
         _enabled->setBoolValue(fgGetBool("/sim/sound/effects/enabled"));
          _volume = _props->getNode("/sim/sound/aimodels/volume", true);
         _volume->setFloatValue(fgGetFloat("/sim/sound/effects/volume"));
-_volume->setFloatValue(0.1f);
     }
 
     _avionics_enabled = _props->getNode("sim/sound/avionics/enabled", true);
@@ -65,7 +64,7 @@ _volume->setFloatValue(0.1f);
     SGSampleGroup::_refname = refname;
     SGSampleGroup::_smgr->add(this, refname);
 
-    if (_avionics_enabled->getBoolValue())
+    if (!_is_aimodel)
     {
         _avionics = _smgr->find("avionics", true);
         _avionics->tie_to_listener();
@@ -113,7 +112,7 @@ FGFX::init()
     }
 
     node = root.getNode("fx");
-    if(node && !_is_aimodel) {
+    if(node) {
         for (int i = 0; i < node->nChildren(); ++i) {
             SGXmlSound *soundfx = new SGXmlSound();
   
@@ -144,23 +143,18 @@ FGFX::reinit()
 void
 FGFX::update (double dt)
 {
-    bool active = _avionics_ext->getBoolValue() ||
-                  _internal->getBoolValue();
-
-    if (_avionics_enabled->getBoolValue()) {
-        if (!_avionics) {
-            _avionics = _smgr->find("avionics", true);
-            _avionics->tie_to_listener();
+    if ( _enabled->getBoolValue() ) {
+        if ( _avionics_enabled->getBoolValue())
+        {
+            if (_avionics_ext->getBoolValue() || _internal->getBoolValue()) {
+                // avionics sound is enabled
+                _avionics->resume(); // no-op if already in resumed state
+                _avionics->set_volume( _avionics_volume->getFloatValue() );
+            }
+            else
+                _avionics->suspend();
         }
 
-        if ( active )
-            _avionics->resume(); // no-op if already in resumed state
-        else
-            _avionics->suspend();
-        _avionics->set_volume( _avionics_volume->getFloatValue() );
-    }
-
-    if ( _enabled->getBoolValue() ) {
         set_volume( _volume->getDoubleValue() );
         resume();