tweak whitespace and add coordinate space comment
authorPaul Davis <paul@linuxaudiosystems.com>
Mon, 28 Oct 2013 16:26:40 +0000 (12:26 -0400)
committerPaul Davis <paul@linuxaudiosystems.com>
Mon, 28 Oct 2013 16:26:40 +0000 (12:26 -0400)
libs/canvas/group.cc

index 37da7c4bfce49974be8566ce97e1a60df047a6df..59dad89dc5740481462f23aa35799caf44193ff7 100644 (file)
@@ -148,7 +148,9 @@ Group::compute_bounding_box () const
        bool have_one = false;
 
        for (list<Item*>::const_iterator i = _items.begin(); i != _items.end(); ++i) {
+
                boost::optional<Rect> item_bbox = (*i)->bounding_box ();
+
                if (!item_bbox) {
                        continue;
                }
@@ -285,17 +287,23 @@ Group::child_changed ()
 void
 Group::add_items_at_point (Duple const point, vector<Item const *>& items) const
 {
+       /* Point is in canvas coordinate system */
+
        boost::optional<Rect> const bbox = bounding_box ();
 
        if (!bbox || !bbox.get().contains (point)) {
                return;
        }
 
+       /* this adds this group itself to the list of items at point */
        Item::add_items_at_point (point, items);
        
+
+       /* now recurse and add any items within our group that contain point */
+
        ensure_lut ();
-       
        vector<Item*> our_items = _lut->items_at_point (point);
+
        for (vector<Item*>::iterator i = our_items.begin(); i != our_items.end(); ++i) {
                (*i)->add_items_at_point (point - (*i)->position(), items);
        }