X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=gtk2_ardour%2Feditor_cursors.cc;h=239d27d4f98dd413358e755f9100cc016c083117;hb=1315ee3dff1e4966ab28bd47e81b5f003cac6e40;hp=78552d6451ee157005bdfef5829a1ff315daf59b;hpb=077b5779b9392ed8dae243700e03ac79c12a7180;p=ardour.git

diff --git a/gtk2_ardour/editor_cursors.cc b/gtk2_ardour/editor_cursors.cc
index 78552d6451..239d27d4f9 100644
--- a/gtk2_ardour/editor_cursors.cc
+++ b/gtk2_ardour/editor_cursors.cc
@@ -15,7 +15,6 @@
     along with this program; if not, write to the Free Software
     Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 
-    $Id$
 */
 
 #include <cstdlib>
@@ -28,33 +27,30 @@
 
 using namespace sigc;
 using namespace ARDOUR;
+using namespace PBD;
 using namespace Gtk;
 
-Editor::Cursor::Cursor (Editor& ed, const string& color, bool (Editor::*callbck)(GdkEvent*,ArdourCanvas::Item*))
-	: editor (ed), length(1.0)
+Editor::Cursor::Cursor (Editor& ed, bool (Editor::*callbck)(GdkEvent*,ArdourCanvas::Item*))
+	: editor (ed),
+	  canvas_item (*editor.cursor_group),
+	  length(1.0)
 {
 	
 	/* "randomly" initialize coords */
+	
+	points.push_back(Gnome::Art::Point(-9383839.0, 0.0));
+	points.push_back(Gnome::Art::Point(1.0, 0.0));
 
-	points->push_back(Gnome::Art::Point(-9383839.0, 0.0));
-	points->push_back(Gnome::Art::Point(1.0, 0.0));
-	Gnome::Canvas::Group *group = editor.cursor_group;
-
-	// cerr << "set cursor points, nc = " << points->num_points << endl;
-
-	canvas_item = new Gnome::Canvas::Line (*group, *points);
-	canvas_item->set_property ("fill_color", color.c_str());
-	canvas_item->set_property ("width_pixels", 1);
-	canvas_item->set_property ("first_arrowhead", (gboolean) TRUE);
-	canvas_item->set_property ("last_arrowhead", (gboolean) TRUE);
-	canvas_item->set_property ("arrow_shape_a", 11.0);
-	canvas_item->set_property ("arrow_shape_b", 0.0);
-	canvas_item->set_property ("arrow_shape_c", 9.0);
-
-	// cerr << "cursor line @ " << canvas_item << endl;
+	canvas_item.property_points() = points;
+	canvas_item.property_width_pixels() = 1;
+	canvas_item.property_first_arrowhead() = TRUE;
+	canvas_item.property_last_arrowhead() = TRUE;
+	canvas_item.property_arrow_shape_a() = 11.0;
+	canvas_item.property_arrow_shape_b() = 0.0;
+	canvas_item.property_arrow_shape_c() = 9.0;
 
-	canvas_item->set_data ("cursor", this);
-	canvas_item->signal_event().connect (bind (mem_fun (ed, callbck), canvas_item));
+	canvas_item.set_data ("cursor", this);
+	canvas_item.signal_event().connect (bind (mem_fun (ed, callbck), &canvas_item));
 
 	current_frame = 1; /* force redraw at 0 */
 }
@@ -62,53 +58,46 @@ Editor::Cursor::Cursor (Editor& ed, const string& color, bool (Editor::*callbck)
 Editor::Cursor::~Cursor ()
 
 {
-        gtk_object_destroy (GTK_OBJECT(canvas_item));
-	gnome_canvas_points_unref (points->gobj());
 }
 
 void
-Editor::Cursor::set_position (jack_nframes_t frame)
+Editor::Cursor::set_position (nframes_t frame)
 {
 	double new_pos =  editor.frame_to_unit (frame);
 
 	if (editor.session == 0) {
-		canvas_item->hide();
+		canvas_item.hide();
 	} else {
-		canvas_item->show();
+		canvas_item.show();
 	}
 
 	current_frame = frame;
 
-	if (new_pos == points->front().get_x()) {
+	if (new_pos != points.front().get_x()) {
 
-		/* change in position is not visible, so just raise it */
+		points.front().set_x (new_pos);
+		points.back().set_x (new_pos);
 
-		canvas_item->raise_to_top();
-		return;
-	} 
+		canvas_item.property_points() = points;
 
-	points->front().set_x(new_pos);
-	points->back().set_x(new_pos);
+		ArdourCanvas::Points p = canvas_item.property_points();
+	}
 
-	// cerr << "set cursor2 al points, nc = " << points->num_points << endl;
-	canvas_item->set_property ("points", points);
-	canvas_item->raise_to_top();
+	canvas_item.raise_to_top();
 }
 
 void
 Editor::Cursor::set_length (double units)
 {
 	length = units; 
-	points->back().set_x (points->front().get_y() + length);
-	// cerr << "set cursor3 al points, nc = " << points->num_points << endl;
-	canvas_item->set_property("points", points);
+	points.back().set_y (points.front().get_y() + length);
+	canvas_item.property_points() = points;
 }
 
 void 
 Editor::Cursor::set_y_axis (double position)
 {
-        points->front().set_y (position);
-	points->back().set_x (position + length);
-	// cerr << "set cursor4 al points, nc = " << points->num_points << endl;
-	canvas_item->set_property("points", points);
+        points.front().set_y (position);
+	points.back().set_y (position + length);
+	canvas_item.property_points() = points;
 }