[ekiga] x-videooutput: refactor sync()



commit a41c5e0283ed089fb8e239083196605f6f3ef90c
Author: VÃctor Manuel JÃquez Leal <vjaquez igalia com>
Date:   Wed Sep 5 16:05:30 2012 -0500

    x-videooutput: refactor sync()
    
    Make more readable the method using a local boolean variable that
    represents no synchronization demand by any video stream.

 .../x-videooutput/videooutput-manager-x.cpp        |    8 ++++++--
 1 files changed, 6 insertions(+), 2 deletions(-)
---
diff --git a/lib/engine/components/x-videooutput/videooutput-manager-x.cpp b/lib/engine/components/x-videooutput/videooutput-manager-x.cpp
index 64f0835..0f51dff 100644
--- a/lib/engine/components/x-videooutput/videooutput-manager-x.cpp
+++ b/lib/engine/components/x-videooutput/videooutput-manager-x.cpp
@@ -457,11 +457,15 @@ GMVideoOutputManager_x::display_pip_frames (const char *local_frame,
 void
 GMVideoOutputManager_x::sync (UpdateRequired sync_required)
 {
-  if (rxWindow && (sync_required.remote || (!sync_required.remote && !sync_required.local))) {
+  bool none_required = ( !sync_required.remote &&
+                         !sync_required.local &&
+                         !sync_required.extended );
+
+  if (rxWindow && (sync_required.remote || none_required)) {
     rxWindow->Sync();
   }
 
-  if (lxWindow && (sync_required.local  || (!sync_required.remote && !sync_required.local))) {
+  if (lxWindow && (sync_required.local || none_required)) {
     lxWindow->Sync();
   }
 }



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]