From: Carlos Lopez <carlos@pcnuevo.(none)>
Date: Thu, 16 Jul 2009 13:13:22 +0000 (+0200)
Subject: Better solution to find a valid canvas for the displayed valuenodes
X-Git-Url: https://git.pterodactylus.net/?a=commitdiff_plain;ds=sidebyside;h=a109a797a2a1dbae8457cb65c905e44f4847b0ad;p=synfig.git

Better solution to find a valid canvas for the displayed valuenodes
---

diff --git a/synfig-studio/trunk/src/gtkmm/widget_curves.cpp b/synfig-studio/trunk/src/gtkmm/widget_curves.cpp
index cf44f76..6077772 100644
--- a/synfig-studio/trunk/src/gtkmm/widget_curves.cpp
+++ b/synfig-studio/trunk/src/gtkmm/widget_curves.cpp
@@ -504,12 +504,19 @@ Widget_Curves::redraw(GdkEventExpose */*bleh*/)
 	gc->set_rgb_fg_color(Gdk::Color("#0000ff")); // It should be user selectable
 	get_window()->draw_rectangle(gc, false, round_to_int((time_adjustment_->get_value()-t_begin)/dt), 0, 0, h);
 
-	// This is not the best solution but I guess that if the first valuenode
-	// has canvas then show the keyframes. Maybe I can loop all them until I find
-	// a valid canvas and use it for look to the keyframes.
-	synfig::Canvas::Handle canvas(curve_list_.begin()->value_desc.get_canvas());
+	// This try to find a valid vanvas to show the keyframes of those
+	// valuenodes. If not canvas found then no keyframes marks are shown.
+	synfig::Canvas::Handle canvas=0;
+	for(curve_iter=curve_list_.begin();curve_iter!=curve_list_.end();++curve_iter)
+	{
+		canvas=curve_iter->value_desc.get_canvas();
+		if(canvas)
+			break;
+	}
+
 	if(canvas)
 	{
+	// Draw vertical lines for the keyframes marks.
 		const synfig::KeyframeList& keyframe_list(canvas->keyframe_list());
 		synfig::KeyframeList::const_iterator iter;