performance improvements
+ use const-reference as argument to avoid tmp. copy + use prefix incremation operator instead of postfix increment operator
This commit is contained in:
parent
85170b3d9f
commit
17c8b2c813
|
@ -86,7 +86,7 @@ bool Measurement::has3DReferences()
|
|||
}
|
||||
|
||||
//add a 3D reference (obj+sub) to end of list
|
||||
int Measurement::addReference3D(App::DocumentObject *obj, const std::string subName)
|
||||
int Measurement::addReference3D(App::DocumentObject *obj, const std::string& subName)
|
||||
{
|
||||
return addReference3D(obj,subName.c_str());
|
||||
}
|
||||
|
|
|
@ -58,7 +58,7 @@ public:
|
|||
bool has3DReferences();
|
||||
|
||||
/// Add a reference
|
||||
int addReference3D(App::DocumentObject* obj, const std::string subName);
|
||||
int addReference3D(App::DocumentObject* obj, const std::string& subName);
|
||||
int addReference3D(App::DocumentObject* obj, const char *subName);
|
||||
|
||||
MeasureType getType();
|
||||
|
|
|
@ -167,7 +167,7 @@ std::vector<TopoDS_Edge> DrawProjectSplit::getEdges(TechDrawGeometry::GeometryOb
|
|||
std::vector<splitPoint> splits;
|
||||
std::vector<TopoDS_Edge>::iterator itOuter = origEdges.begin();
|
||||
int iOuter = 0;
|
||||
for (; itOuter != origEdges.end(); itOuter++, iOuter++) {
|
||||
for (; itOuter != origEdges.end(); ++itOuter, iOuter++) {
|
||||
TopoDS_Vertex v1 = TopExp::FirstVertex((*itOuter));
|
||||
TopoDS_Vertex v2 = TopExp::LastVertex((*itOuter));
|
||||
Bnd_Box sOuter;
|
||||
|
@ -183,7 +183,7 @@ std::vector<TopoDS_Edge> DrawProjectSplit::getEdges(TechDrawGeometry::GeometryOb
|
|||
}
|
||||
int iInner = 0;
|
||||
std::vector<TopoDS_Edge>::iterator itInner = faceEdges.begin();
|
||||
for (; itInner != faceEdges.end(); itInner++,iInner++) {
|
||||
for (; itInner != faceEdges.end(); ++itInner,iInner++) {
|
||||
if (iInner == iOuter) {
|
||||
continue;
|
||||
}
|
||||
|
@ -373,7 +373,7 @@ std::vector<TopoDS_Edge> DrawProjectSplit::split1Edge(TopoDS_Edge e, std::vector
|
|||
auto parms2 = parms.begin() + 1;
|
||||
std::vector<double>::iterator psecond = parms2;
|
||||
std::vector<double>::iterator pstop = parms.end();
|
||||
for (; psecond != pstop; pfirst++,psecond++) {
|
||||
for (; psecond != pstop; ++pfirst,++psecond) {
|
||||
try {
|
||||
BRepBuilderAPI_MakeEdge mkEdge(c, *pfirst, *psecond);
|
||||
if (mkEdge.IsDone()) {
|
||||
|
|
|
@ -292,7 +292,7 @@ void DrawViewPart::extractFaces()
|
|||
std::vector<splitPoint> splits;
|
||||
std::vector<TopoDS_Edge>::iterator itOuter = origEdges.begin();
|
||||
int iOuter = 0;
|
||||
for (; itOuter != origEdges.end(); itOuter++, iOuter++) {
|
||||
for (; itOuter != origEdges.end(); ++itOuter, iOuter++) {
|
||||
TopoDS_Vertex v1 = TopExp::FirstVertex((*itOuter));
|
||||
TopoDS_Vertex v2 = TopExp::LastVertex((*itOuter));
|
||||
Bnd_Box sOuter;
|
||||
|
@ -308,7 +308,7 @@ void DrawViewPart::extractFaces()
|
|||
}
|
||||
int iInner = 0;
|
||||
std::vector<TopoDS_Edge>::iterator itInner = faceEdges.begin();
|
||||
for (; itInner != faceEdges.end(); itInner++,iInner++) {
|
||||
for (; itInner != faceEdges.end(); ++itInner,iInner++) {
|
||||
if (iInner == iOuter) {
|
||||
continue;
|
||||
}
|
||||
|
|
|
@ -76,7 +76,7 @@ struct EdgePoints {
|
|||
TopoDS_Edge edge;
|
||||
};
|
||||
|
||||
GeometryObject::GeometryObject(std::string parent) :
|
||||
GeometryObject::GeometryObject(const string& parent) :
|
||||
Scale(1.f),
|
||||
m_parentName(parent),
|
||||
m_isoCount(0)
|
||||
|
|
|
@ -61,7 +61,7 @@ class TechDrawExport GeometryObject
|
|||
{
|
||||
public:
|
||||
/// Constructor
|
||||
GeometryObject(std::string parent);
|
||||
GeometryObject(const std::string& parent);
|
||||
virtual ~GeometryObject();
|
||||
|
||||
void clear();
|
||||
|
|
Loading…
Reference in New Issue
Block a user