diff libinterp/corefcn/graphics.cc @ 18711:f684f7075aee

Don't fetch lots of axes state information for rendering if object is not visible. * gl-render.cc (opengl_renderer::draw_axes_planes): Check axes color and visibility and return immediately from function if there is nothing to do. * gl-render.cc (opengl_renderer::draw_axes_boxes): Check axes visibility and return immediately from function if there is nothing to do. * gl-render.cc (opengl_renderer::draw_axes_x_grid, drax_axes_y_grid, draw_axes_z_grid): Check axes state and visibility before bothering to fetch any more state data. * gl-render.cc (miscellaneous): Clean up comments. Change lines to wrap at < 80 chars. * graphics.cc (axes::properties::update_axes_layout): Use std::swap instead of intermediate temporary variable for clearer coding style.
author Rik <rik@octave.org>
date Fri, 25 Apr 2014 11:18:04 -0700
parents e42843ceff48
children ee7b23a48947
line wrap: on
line diff
--- a/libinterp/corefcn/graphics.cc	Fri Apr 25 10:02:10 2014 -0700
+++ b/libinterp/corefcn/graphics.cc	Fri Apr 25 11:18:04 2014 -0700
@@ -5172,18 +5172,14 @@
     {
       if (xaxislocation_is ("top"))
         {
-          double tmp = yPlane;
-          yPlane = yPlaneN;
-          yPlaneN = tmp;
+          std::swap (yPlane, yPlaneN);
           x2Dtop = true;
         }
       ypTick = yPlaneN;
       ypTickN = yPlane;
       if (yaxislocation_is ("right"))
         {
-          double tmp = xPlane;
-          xPlane = xPlaneN;
-          xPlaneN = tmp;
+          std::swap (xPlane, xPlaneN);
           y2Dright = true;
         }
       xpTick = xPlaneN;