Author: Carlos Lopez <[email protected]>
Date:   Fri Mar  1 18:22:39 2013 +0100

Enclose statements inside if-else sections for better readability.

---

 synfig-studio/src/gui/docks/dock_navigator.cpp |   95 ++++++++++++------------
 1 files changed, 48 insertions(+), 47 deletions(-)

diff --git a/synfig-studio/src/gui/docks/dock_navigator.cpp 
b/synfig-studio/src/gui/docks/dock_navigator.cpp
index 5c1975a..ddb635f 100644
--- a/synfig-studio/src/gui/docks/dock_navigator.cpp
+++ b/synfig-studio/src/gui/docks/dock_navigator.cpp
@@ -189,58 +189,59 @@ void studio::Widget_NavView::on_finish_render()
                if(cairo_surface_status(surf))
                        return;
                Target_Cairo::gamma_filter(surf);
-               queue_draw();
-               return;
        }
-       //convert it into our pixmap
-       PixelFormat pf(PF_RGB);
-
-       if(!*surface)
+       else
        {
-               synfig::warning("dock_navigator: Bad surface");
-               return;
-       }
+               //convert it into our pixmap
+               PixelFormat pf(PF_RGB);
 
-       int w = 0, h = 0;
-       int dw = surface->get_w();
-       int dh = surface->get_h();
+               if(!*surface)
+               {
+                       synfig::warning("dock_navigator: Bad surface");
+                       return;
+               }
 
-       if(prev)
-       {
-               w = prev->get_width();
-               h = prev->get_height();
-       }
+               int w = 0, h = 0;
+               int dw = surface->get_w();
+               int dh = surface->get_h();
 
-       if(w != dw || h != dh || !prev)
-       {
-               const int total_bytes(dw*dh*synfig::channels(pf));
-
-               //synfig::warning("Nav: Updating the pixbuf to be the right 
size, etc. (%d bytes)", total_bytes);
-
-               prev.clear();
-               guint8 *bytes = new guint8[total_bytes]; //24 bits per pixel
-
-               //convert into our buffered dataS
-               //synfig::warning("Nav: converting color format into buffer");
-               convert_color_format((unsigned char *)bytes, (*surface)[0], 
dw*dh, pf, App::gamma);
-
-               prev =
-               Gdk::Pixbuf::create_from_data(
-                       bytes,  // pointer to the data
-                       Gdk::COLORSPACE_RGB, // the colorspace
-                       ((pf&PF_A)==PF_A), // has alpha?
-                       8, // bits per sample
-                       dw,     // width
-                       dh,     // height
-                       dw*synfig::channels(pf), // stride (pitch)
-                       sigc::ptr_fun(freegu8)
-               );
-       }
-       else
-       {
-               if(prev) //just in case we're stupid
+               if(prev)
                {
-                       convert_color_format((unsigned char 
*)prev->get_pixels(), (*surface)[0], dw*dh, pf, App::gamma);
+                       w = prev->get_width();
+                       h = prev->get_height();
+               }
+
+               if(w != dw || h != dh || !prev)
+               {
+                       const int total_bytes(dw*dh*synfig::channels(pf));
+
+                       //synfig::warning("Nav: Updating the pixbuf to be the 
right size, etc. (%d bytes)", total_bytes);
+
+                       prev.clear();
+                       guint8 *bytes = new guint8[total_bytes]; //24 bits per 
pixel
+
+                       //convert into our buffered dataS
+                       //synfig::warning("Nav: converting color format into 
buffer");
+                       convert_color_format((unsigned char *)bytes, 
(*surface)[0], dw*dh, pf, App::gamma);
+
+                       prev =
+                       Gdk::Pixbuf::create_from_data(
+                               bytes,  // pointer to the data
+                               Gdk::COLORSPACE_RGB, // the colorspace
+                               ((pf&PF_A)==PF_A), // has alpha?
+                               8, // bits per sample
+                               dw,     // width
+                               dh,     // height
+                               dw*synfig::channels(pf), // stride (pitch)
+                               sigc::ptr_fun(freegu8)
+                       );
+               }
+               else
+               {
+                       if(prev) //just in case we're stupid
+                       {
+                               convert_color_format((unsigned char 
*)prev->get_pixels(), (*surface)[0], dw*dh, pf, App::gamma);
+                       }
                }
        }
        queue_draw();
@@ -294,7 +295,7 @@ bool studio::Widget_NavView::on_expose_draw(GdkEventExpose 
*/*exp*/)
                        w = prev->get_width();
                        h = prev->get_height();
                }
-               else
+               if(studio::App::navigator_uses_cairo)
                {
                        w=cairo_image_surface_get_width(*cairo_surface.get());
                        h=cairo_image_surface_get_height(*cairo_surface.get());


------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_feb
_______________________________________________
Synfig-devl mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/synfig-devl

Reply via email to