Merge pull request #111 from rdumusc/master

DesktopStreamer: bugfix; always stop streaming when the stream window is closed

Author: tribal-tec

apps/DesktopStreamer/Stream.cpp

@@ -74,14 +74,14 @@ class Stream::Impl
 
 bool processEvents( const bool interact )
 {
-    while( _stream.isRegisteredForEvents() && _stream.hasEvent( ))
+    while( _stream.hasEvent( ))
     {
-        deflect::Event event = _stream.getEvent();
+        const deflect::Event event = _stream.getEvent();
 
         if( event.type == deflect::Event::EVT_CLOSE )
             return false;
 
-        if( !interact )
+        if( !_stream.isRegisteredForEvents() || !interact )
             continue;
 
         switch( event.type )

doc/Changelog.md

@@ -5,6 +5,9 @@ Changelog {#Changelog}
 
 ### 0.11.0 (git master)
 
+* [111](https://github.com/BlueBrain/Deflect/pull/111):
+  DesktopStreamer: bugfix; stop streaming when the server closes the stream
+  and the "remote control" option was not enabled.
 * [110](https://github.com/BlueBrain/Deflect/pull/110):
   DesktopStreamer: Support for streaming multiple windows on OSX disabled by
   default, can be enabled with cmake -DDESKTOPSTREAMER_ENABLE_MULTIWINDOW=ON.