@babblevoice/projectrtp 2.4.15 → 2.4.16

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@babblevoice/projectrtp",
-  "version": "2.4.15",
+  "version": "2.4.16",
   "description": "A scalable Node addon RTP server",
   "main": "index.js",
   "directories": {

package/src/projectrtpchannel.cpp

@@ -1027,22 +1027,20 @@ bool projectrtpchannel::mix( projectrtpchannel::pointer other ) {
   if( nullptr == this->mixer && nullptr != other->mixer ) {
     this->mixer = other->mixer;
     this->mixer->addchannel( shared_from_this() );
-    postdatabacktojsfromthread( shared_from_this(), "mix", "start" );
+
   } else if ( nullptr != this->mixer && nullptr == other->mixer ) {
     other->mixer = this->mixer;
     this->mixer->addchannel( other );
-    postdatabacktojsfromthread( other, "mix", "start" );
+
   } else if( nullptr == this->mixer && nullptr == other->mixer  ) {
     this->mixer = projectchannelmux::create( workercontext );
     other->mixer = this->mixer;
 
     this->mixer->addchannels( shared_from_this(), other );
-    postdatabacktojsfromthread( shared_from_this(), "mix", "start" );
-    postdatabacktojsfromthread( other, "mix", "start" );
   } else {
     /* If we get here this and other are already mixing and should be cleaned up first */
-    postdatabacktojsfromthread( shared_from_this(), "mix", "busy" );
     RELEASESPINLOCK( this->mixerlock );
+    postdatabacktojsfromthread( shared_from_this(), "mix", "busy" );
     return false;
   }
 
@@ -1053,6 +1051,9 @@ bool projectrtpchannel::mix( projectrtpchannel::pointer other ) {
 
   RELEASESPINLOCK( this->mixerlock );
 
+  postdatabacktojsfromthread( shared_from_this(), "mix", "start" );
+  postdatabacktojsfromthread( other, "mix", "start" );
+
   return true;
 }
 

package/test/interface/projectrtpdtmf.js

@@ -808,17 +808,17 @@ describe( "dtmf", function() {
     expect( dtmfcpkcount ).to.be.within( 4, 8 )
 
     expect( receveiedmessages[ 0 ].action ).to.equal( "mix" )
-    //expect( receveiedmessages[ 1 ].action ).to.equal( "mix" )
-    expect( receveiedmessages[ 1 ].action ).to.equal( "telephone-event" )
+    expect( receveiedmessages[ 1 ].action ).to.equal( "mix" )
     expect( receveiedmessages[ 2 ].action ).to.equal( "telephone-event" )
-    expect( receveiedmessages[ 3 ].action ).to.equal( "mix" )
-    expect( receveiedmessages[ 4 ].action ).to.equal( "close" )
+    expect( receveiedmessages[ 3 ].action ).to.equal( "telephone-event" )
+    expect( receveiedmessages[ 4 ].action ).to.equal( "mix" )
+    expect( receveiedmessages[ 5 ].action ).to.equal( "close" )
 
-    //expect( receveiedmessages[ 0 ].event ).to.equal( "start" )
     expect( receveiedmessages[ 0 ].event ).to.equal( "start" )
-    expect( receveiedmessages[ 1 ].event ).to.equal( "4" )
-    expect( receveiedmessages[ 2 ].event ).to.equal( "5" )
-    expect( receveiedmessages[ 3 ].event ).to.equal( "finished" )
+    expect( receveiedmessages[ 1 ].event ).to.equal( "start" )
+    expect( receveiedmessages[ 2 ].event ).to.equal( "4" )
+    expect( receveiedmessages[ 3 ].event ).to.equal( "5" )
+    expect( receveiedmessages[ 4 ].event ).to.equal( "finished" )
 
   } )