add coordinate space comment
authorPaul Davis <paul@linuxaudiosystems.com>
Mon, 28 Oct 2013 16:27:03 +0000 (12:27 -0400)
committerPaul Davis <paul@linuxaudiosystems.com>
Mon, 28 Oct 2013 16:27:03 +0000 (12:27 -0400)
libs/canvas/lookup_table.cc

index 9697ba81039ef6ac577945f147120598bb7ab889..776d873de22b5b98dd42d1e5f2289d983e894c79 100644 (file)
@@ -49,18 +49,19 @@ DumbLookupTable::get (Rect const &)
        return vitems;
 }
 
-/* XXX: what coordinate system is the point in? parent of our group I think */
 vector<Item *>
 DumbLookupTable::items_at_point (Duple point) const
 {
+       /* Point is in canvas coordinate system */
+
        list<Item *> items = _group.items ();
        vector<Item *> vitems;
 
        for (list<Item *>::const_iterator i = items.begin(); i != items.end(); ++i) {
                boost::optional<Rect> item_bbox = (*i)->bounding_box ();
                if (item_bbox) {
-                       Rect parent_bbox = (*i)->item_to_parent (item_bbox.get ());
-                       if (parent_bbox.contains (point)) {
+                       Rect canvas_bbox = (*i)->item_to_canvas (item_bbox.get ());
+                       if (canvas_bbox.contains (point)) {
                                vitems.push_back (*i);
                        }
                }