fix up thinko in recent raise_to_top/lower_to_bottom optimization
[ardour.git] / libs / canvas / rectangle.cc
index a7683143641ee108978ea88380b95ff7c73d0359..06a41e074c0fe6514c58b40ca12b7cdfc4778bb3 100644 (file)
@@ -1,8 +1,28 @@
+/*
+    Copyright (C) 2011-2013 Paul Davis
+    Author: Carl Hetherington <cth@carlh.net>
+
+    This program is free software; you can redistribute it and/or modify
+    it under the terms of the GNU General Public License as published by
+    the Free Software Foundation; either version 2 of the License, or
+    (at your option) any later version.
+
+    This program is distributed in the hope that it will be useful,
+    but WITHOUT ANY WARRANTY; without even the implied warranty of
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+    GNU General Public License for more details.
+
+    You should have received a copy of the GNU General Public License
+    along with this program; if not, write to the Free Software
+    Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+*/
+
 #include <iostream>
 #include <cairomm/context.h>
 #include "pbd/stacktrace.h"
-#include "pbd/xml++.h"
 #include "pbd/compose.h"
+
+#include "canvas/canvas.h"
 #include "canvas/rectangle.h"
 #include "canvas/debug.h"
 #include "canvas/utils.h"
@@ -16,7 +36,6 @@ Rectangle::Rectangle (Group* parent)
        , Fill (parent)
        , _outline_what ((What) (LEFT | RIGHT | TOP | BOTTOM))
 {
-
 }
 
 Rectangle::Rectangle (Group* parent, Rect const & rect)
@@ -30,41 +49,69 @@ Rectangle::Rectangle (Group* parent, Rect const & rect)
 }
 
 void
-Rectangle::render (Rect const & /*area*/, Cairo::RefPtr<Cairo::Context> context) const
+Rectangle::render (Rect const & area, Cairo::RefPtr<Cairo::Context> context) const
 {
-       Rect plot = _rect;
+       Rect self = item_to_window (_rect);
+       boost::optional<Rect> r = self.intersection (area);
+
+       if (!r) {
+               std::cerr << whatami() << '/' << name << " not covered by render area! ... " << self << " vs. " << area << std::endl;
+               return;
+       }
 
-       plot.x1 = min (plot.x1, CAIRO_MAX);
-       plot.y1 = min (plot.y1, CAIRO_MAX);
+       Rect draw = r.get ();
 
        if (_fill) {
-               setup_fill_context (context);
-               context->rectangle (plot.x0, plot.y0, plot.width(), plot.height());
+               if (_stops.empty()) {
+                       setup_fill_context (context);
+               } else {
+                       setup_gradient_context (context, self, Duple (draw.x0, draw.y0));
+               }
+               context->rectangle (draw.x0, draw.y0, draw.width(), draw.height());
                context->fill ();
        }
-
+       
        if (_outline) {
-               if (_outline_what & LEFT) {
-                       context->move_to (plot.x0, plot.y0);
-                       context->line_to (plot.x0, plot.y1);
-               }
-               
-               if (_outline_what & BOTTOM) {
-                       context->move_to (plot.x0, plot.y1);
-                       context->line_to (plot.x1, plot.y1);
-               }
-               
-               if (_outline_what & RIGHT) {
-                       context->move_to (plot.x1, plot.y0);
-                       context->line_to (plot.x1, plot.y1);
-               }
-               
-               if (_outline_what & TOP) {
-                       context->move_to (plot.x0, plot.y0);
-                       context->line_to (plot.x0, plot.y1);
+
+               setup_outline_context (context);
+
+               if (_outline_what == What (LEFT|RIGHT|BOTTOM|TOP)) {
+                       
+                       context->rectangle (self.x0 + 0.5, self.y0 + 0.5, self.width(), self.height());
+
+               } else {
+
+                       context->set_line_cap (Cairo::LINE_CAP_SQUARE);
+                       
+                       /* see the cairo FAQ on single pixel lines to see why we do
+                        * this expansion of the perimeter.
+                        */
+
+                       if (_outline_what & LEFT) {
+                               /* vertical line: move x-coordinate by 0.5 pixels */
+                               context->move_to (self.x0 + 0.5, self.y0);
+                               context->line_to (self.x0 + 0.5, self.y1);
+                       }
+                       
+                       if (_outline_what & BOTTOM) {
+                               /* horizontal line: move y-coordinate by 0.5 pixels */
+                               context->move_to (self.x0, self.y1 - 0.5);
+                               context->line_to (self.x1, self.y1 - 0.5);
+                       }
+                       
+                       if (_outline_what & RIGHT) {
+                               /* vertical line: move x-coordinate by 0.5 pixels */
+                               context->move_to (self.x1 - 0.5, self.y0);
+                               context->line_to (self.x1 - 0.5, self.y1);
+                       }
+                       
+                       if (_outline_what & TOP) {
+                               /* horizontal line: move y-coordinate by 0.5 pixels */
+                               context->move_to (self.x0, self.y0 + 0.5);
+                               context->line_to (self.x1, self.y0 + 0.5);
+                       }
                }
                
-               setup_outline_context (context);
                context->stroke ();
        }
 }
@@ -72,9 +119,19 @@ Rectangle::render (Rect const & /*area*/, Cairo::RefPtr<Cairo::Context> context)
 void
 Rectangle::compute_bounding_box () const
 {
-       Rect r = _rect.fix ();
-       _bounding_box = boost::optional<Rect> (r.expand (_outline_width / 2));
-       
+       if (!_rect.empty()) {
+               Rect r = _rect.fix ();
+
+               /* our outlines are always inside our coordinates, but we have
+                * to ensure that our bounding box fully *contains* the
+                * rectangle
+                *
+                * XXX: or something like that, waffle.
+                *
+                */
+               _bounding_box = r.expand (1.0);
+       }
+
        _bounding_box_dirty = false;
 }
 
@@ -163,39 +220,3 @@ Rectangle::set_outline_what (int what)
        set_outline_what ((What) what);
 }
 
-XMLNode *
-Rectangle::get_state () const
-{
-       XMLNode* node = new XMLNode ("Rectangle");
-#ifdef CANVAS_DEBUG
-       if (!name.empty ()) {
-               node->add_property ("name", name);
-       }
-#endif 
-       node->add_property ("x0", string_compose ("%1", _rect.x0));
-       node->add_property ("y0", string_compose ("%1", _rect.y0));
-       node->add_property ("x1", string_compose ("%1", _rect.x1));
-       node->add_property ("y1", string_compose ("%1", _rect.y1));
-       node->add_property ("outline-what", string_compose ("%1", _outline_what));
-
-       add_item_state (node);
-       add_outline_state (node);
-       add_fill_state (node);
-       return node;
-}
-
-void
-Rectangle::set_state (XMLNode const * node)
-{
-       _rect.x0 = atof (node->property("x0")->value().c_str());
-       _rect.y0 = atof (node->property("y0")->value().c_str());
-       _rect.x1 = atof (node->property("x1")->value().c_str());
-       _rect.y1 = atof (node->property("y1")->value().c_str());
-       _outline_what = (What) atoi (node->property("outline-what")->value().c_str());
-
-       set_item_state (node);
-       set_outline_state (node);
-       set_fill_state (node);
-
-       _bounding_box_dirty = true;
-}