dragging(false), draggingHandle1(false), draggingHandle2(false)//, needUpdate(false)
{
type = OTContainer;
+ state = OSInactive;
}
// Use overloaded assignment operator
*this = copy;
type = OTContainer;
+ state = OSInactive;
}
// Small problem with this approach: if the copied object goes out of scope,
// all of the objects we copied in here will be deleted. D'oh!
- for(uint i=0; i<from.objects.size(); i++)
+ // For this COPY constructor to be meaningful, we have to actually COPY the
+ // objects in this Container, not just MOVE a copy of the POINTER! D-:
+ std::vector<Object *>::const_iterator i;
+
+// for(uint i=0; i<from.objects.size(); i++)
+ for(i=from.objects.begin(); i!=from.objects.end(); i++)
{
- Object * object = from.objects[i];
+// Object * object = from.objects[i];
+ Object * object = (*i)->Copy();
- // Need to copy the object here...
+ // Need to actually COPY the object here, not copy the pointer only!!
+ // (which we do now, above :-P)
objects.push_back(object);
}
{
QRectF boundary;
+//int a=1;
for(std::vector<Object *>::iterator i=objects.begin(); i!=objects.end(); i++)
{
+//printf("Containter::Draw item #%i [%X]...\n", a++, *i);
(*i)->Draw(painter);
boundary = boundary.united((*i)->Extents());
}
painter->SetPen(QPen(Qt::blue, 2.0, Qt::DashLine));
painter->SetBrush(QBrush(Qt::NoBrush));
- painter->DrawRect(boundary);
+ painter->DrawPaddedRect(boundary);
}
}
}
}
+ if (snapToGrid)
+ point = SnapPointToGrid(point);
+
// We check to see if the container we're trying to access is the
// DrawingView's document. If so, we ignore the state of the container.
// Otherwise, we care about the state of the container. :-)
fprintf(file, "ENDCONTAINER\n");
}
+
+/*virtual*/ Object * Container::Copy(void)
+{
+#warning "!!! This doesn't take care of attached Dimensions !!!"
+/*
+This is a real problem. While having a pointer in the Dimension to this line's points
+is fast & easy, it creates a huge problem when trying to replicate an object like this.
+
+Maybe a way to fix that then, is to have reference numbers instead of pointers. That
+way, if you copy them, ... you might still have problems. Because you can't be sure if
+a copy will be persistant or not, you then *definitely* do not want them to have the
+same reference number.
+*/
+ Container * c = new Container(position, parent);
+ *c = *this;
+ return c;
+}
+