Check track iterators for validity before using

[r2c2.git] / source / designer / manipulator.cpp

diff --git a/source/designer/manipulator.cpp b/source/designer/manipulator.cpp
index 23ea138623c42809726c0df07c644ad15ba52f99..b41b29eeac9ac6e6562df825824f316a817fa3f7 100644 (file)
--- a/source/designer/manipulator.cpp
+++ b/source/designer/manipulator.cpp
@@ -59,12 +59,26 @@ void Manipulator::start_elevate()
        mode = ELEVATE;
 }
 
-void Manipulator::start_extend()
+bool Manipulator::start_extend()
 {
        if(mode)
                cancel();
 
-       mode = EXTEND;
+       bool ok = false;
+       const set<Track *> &stracks = selection.get_tracks();
+       for(set<Track *>::const_iterator i=stracks.begin(); (!ok && i!=stracks.end()); ++i)
+       {
+               const vector<Track *> &links = (*i)->get_links();
+               for(vector<Track *>::const_iterator j=links.begin(); (!ok && j!=links.end()); ++j)
+                       ok = !*j;
+       }
+
+       if(ok)
+               mode = EXTEND;
+       else
+               signal_status.emit("No free endpoints");
+
+       return ok;
 }
 
 void Manipulator::duplicate()
@@ -358,8 +372,8 @@ void Manipulator::button_press(int, int, unsigned btn, unsigned)
 
 void Manipulator::pointer_motion(int x, int y)
 {
-       pointer_y = y;
-       gpointer = designer.map_pointer_to_ground(x, event_source.get_height()-1-y);
+       pointer_y = event_source.get_height()-1-y;
+       gpointer = designer.map_pointer_to_ground(x, pointer_y);
 
        if(mode==MOVE)
        {
@@ -420,7 +434,7 @@ void Manipulator::pointer_motion(int x, int y)
        }
        else if(mode==ELEVATE)
        {
-               float dz = (y-elev_origin)/1000.;
+               float dz = (pointer_y-elev_origin)/1000.;
 
                signal_status.emit(format("Elevation: %+.0fmm (%.0fmm)", dz*1000, (center.z+dz)*1000));