suzanne.soy/FreeCAD
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Part/Attacher: remove a couple of no more valid asserts

Browse Source
This commit is contained in:
Alexander Golubev 2015-09-01 23:46:55 +03:00 committed by Stefan Tröger
parent 797d6d3a11
commit 8bca27463e
1 changed files with 0 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/Mod/Part/App/Attacher.cpp
View File

@ -613,8 +613,6 @@ void AttachEngine::readLinks(const App::PropertyLinkSubList &references,
shapes[i] = &(shape->_Shape); shapes[i] = &(shape->_Shape);
} }
} else if ( geof->isDerivedFrom(App::Plane::getClassTypeId()) ){ } else if ( geof->isDerivedFrom(App::Plane::getClassTypeId()) ){
// TODO Why this assert is here? (2015-08-31, Fat-Zer)
assert(sub[i].length()==0);//no more support for "back"/"front" on planes. Use mapReversed instead.
//obtain Z axis and origin of placement //obtain Z axis and origin of placement
Base::Vector3d norm; Base::Vector3d norm;
geof->Placement.getValue().getRotation().multVec(Base::Vector3d(0.0,0.0,1.0),norm); geof->Placement.getValue().getRotation().multVec(Base::Vector3d(0.0,0.0,1.0),norm);
@ -626,7 +624,6 @@ void AttachEngine::readLinks(const App::PropertyLinkSubList &references,
storage.push_back( builder.Shape() ); storage.push_back( builder.Shape() );
shapes[i] = &(storage[storage.size()-1]); shapes[i] = &(storage[storage.size()-1]);
} else if ( geof->isDerivedFrom(App::Line::getClassTypeId()) ){ } else if ( geof->isDerivedFrom(App::Line::getClassTypeId()) ){
assert(sub[i].length()==0);
//obtain X axis and origin of placement //obtain X axis and origin of placement
//note an inconsistency: App::Line is along local X, PartDesign::DatumLine is along local Z. //note an inconsistency: App::Line is along local X, PartDesign::DatumLine is along local Z.
Base::Vector3d dir; Base::Vector3d dir;