[ekiga] Code cleanup: make code more robust



commit 5b3448eabd7cce018a274239e88269f666c56bd6
Author: Eugen Dedu <Eugen Dedu pu-pm univ-fcomte fr>
Date:   Thu Sep 15 22:12:58 2011 +0200

    Code cleanup: make code more robust

 lib/engine/components/opal/opal-call-manager.cpp |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)
---
diff --git a/lib/engine/components/opal/opal-call-manager.cpp b/lib/engine/components/opal/opal-call-manager.cpp
index 762fd6c..b3c54d0 100644
--- a/lib/engine/components/opal/opal-call-manager.cpp
+++ b/lib/engine/components/opal/opal-call-manager.cpp
@@ -641,13 +641,15 @@ void CallManager::get_video_options (CallManager::VideoOptions & options) const
     OpalMediaFormat media_format = media_formats_list [i];
     if (media_format.GetMediaType () == OpalMediaType::Video ()) {
 
-      int j = 0;
+      int j;
       for (j = 0; j < NB_VIDEO_SIZES; j++) {
 
         if (Ekiga::VideoSizes [j].width == media_format.GetOptionInteger (OpalVideoFormat::FrameWidthOption ())
-            && Ekiga::VideoSizes [j].width == media_format.GetOptionInteger (OpalVideoFormat::FrameWidthOption ()))
+            && Ekiga::VideoSizes [j].height == media_format.GetOptionInteger (OpalVideoFormat::FrameHeightOption ()))
           break;
       }
+      if (j >= NB_VIDEO_SIZES)
+        g_error ("Cannot find video size");
       options.size = j;
 
       options.maximum_frame_rate =



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