[vlmc-devel] commit: WorkflowRenderer: avoid multiple SIGNAL reconnections. ( Hugo Beauzée-Luyssen )

git at videolan.org git at videolan.org
Thu May 27 22:50:43 CEST 2010


vlmc | branch: master | Hugo Beauzée-Luyssen <beauze.h at gmail.com> | Thu May 27 21:09:13 2010 +0200| [9ec88dcd12d898922827cc4cd1a5245fbc97fecf] | committer: Hugo Beauzée-Luyssen 

WorkflowRenderer: avoid multiple SIGNAL reconnections.

> http://git.videolan.org/gitweb.cgi/vlmc.git/?a=commit;h=9ec88dcd12d898922827cc4cd1a5245fbc97fecf
---

 src/Renderer/WorkflowRenderer.cpp |   12 ++++++------
 1 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/Renderer/WorkflowRenderer.cpp b/src/Renderer/WorkflowRenderer.cpp
index 8bba6b5..0038732 100644
--- a/src/Renderer/WorkflowRenderer.cpp
+++ b/src/Renderer/WorkflowRenderer.cpp
@@ -65,6 +65,11 @@ void    WorkflowRenderer::initializeRenderer()
              this, SIGNAL( frameChanged( qint64, MainWorkflow::FrameChangedReason ) ) );
     connect( m_mainWorkflow, SIGNAL( lengthChanged( qint64 ) ),
              this, SLOT(mainWorkflowLenghtChanged(qint64) ) );
+    //Media player part: to update PreviewWidget
+    connect( m_mediaPlayer, SIGNAL( playing() ),    this,   SIGNAL( playing() ), Qt::DirectConnection );
+    connect( m_mediaPlayer, SIGNAL( paused() ),     this,   SIGNAL( paused() ), Qt::DirectConnection );
+    //FIXME:: check if this doesn't require Qt::QueuedConnection
+    connect( m_mediaPlayer, SIGNAL( stopped() ),    this,   SIGNAL( endReached() ) );
 }
 
 WorkflowRenderer::~WorkflowRenderer()
@@ -224,15 +229,10 @@ void        WorkflowRenderer::startPreview()
         m_outputFps = outputFps();
         setupRenderer( m_width, m_height, m_outputFps );
     }
+    //Deactivating vlc's keyboard inputs.
     m_mediaPlayer->setKeyInput( false );
     m_mediaPlayer->setMedia( m_media );
 
-    //Media player part: to update PreviewWidget
-    connect( m_mediaPlayer, SIGNAL( playing() ),    this,   SIGNAL( playing() ), Qt::DirectConnection );
-    connect( m_mediaPlayer, SIGNAL( paused() ),     this,   SIGNAL( paused() ), Qt::DirectConnection );
-    //FIXME:: check if this doesn't require Qt::QueuedConnection
-    connect( m_mediaPlayer, SIGNAL( stopped() ),    this,   SIGNAL( endReached() ) );
-
     m_mainWorkflow->setFullSpeedRender( false );
     m_mainWorkflow->startRender( m_width, m_height );
     m_isRendering = true;



More information about the Vlmc-devel mailing list