kindred/create
1
1
Fork 0
Code Issues 30 Pull Requests Actions Packages Projects 9 Releases 2 Wiki Activity

[Sketcher] Use switch-case in getPointForGeometry()

Browse Source 
Minor readability improvement.
This commit is contained in:
Ajinkya Dahale
2024-11-23 18:39:23 +05:30
parent 5262482bc4
commit 0b32db53ed
1 changed files with 56 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

72
src/Mod/Sketcher/App/SketchObject.cpp
View File

@@ -1401,10 +1401,16 @@ Base::Vector3d SketchObject::getPointForGeometry<>(const Part::GeomPoint *geomPo
template <>
Base::Vector3d SketchObject::getPointForGeometry<>(const Part::GeomLineSegment *lineSeg, PointPos PosId)
{
if (PosId == PointPos::start)
switch (PosId) {
case PointPos::start: {
return lineSeg->getStartPoint();
else if (PosId == PointPos::end)
}
case PointPos::end: {
return lineSeg->getEndPoint();
}
default:
break;
}
return Base::Vector3d();
}
@@ -1429,58 +1435,92 @@ Base::Vector3d SketchObject::getPointForGeometry<>(const Part::GeomEllipse *elli
template <>
Base::Vector3d SketchObject::getPointForGeometry<>(const Part::GeomArcOfCircle *aoc, PointPos PosId)
{
if (PosId == PointPos::start)
switch (PosId) {
case PointPos::start: {
return aoc->getStartPoint(/*emulateCCW=*/true);
else if (PosId == PointPos::end)
}
case PointPos::end: {
return aoc->getEndPoint(/*emulateCCW=*/true);
else if (PosId == PointPos::mid)
}
case PointPos::mid: {
return aoc->getCenter();
}
default:
break;
}
return Base::Vector3d();
}
template <>
Base::Vector3d SketchObject::getPointForGeometry<>(const Part::GeomArcOfEllipse *aoe, PointPos PosId)
{
if (PosId == PointPos::start)
switch (PosId) {
case PointPos::start: {
return aoe->getStartPoint(/*emulateCCW=*/true);
else if (PosId == PointPos::end)
}
case PointPos::end: {
return aoe->getEndPoint(/*emulateCCW=*/true);
else if (PosId == PointPos::mid)
}
case PointPos::mid: {
return aoe->getCenter();
}
default:
break;
}
return Base::Vector3d();
}
template <>
Base::Vector3d SketchObject::getPointForGeometry<>(const Part::GeomArcOfHyperbola *aoh, PointPos PosId)
{
if (PosId == PointPos::start)
switch (PosId) {
case PointPos::start: {
return aoh->getStartPoint();
else if (PosId == PointPos::end)
}
case PointPos::end: {
return aoh->getEndPoint();
else if (PosId == PointPos::mid)
}
case PointPos::mid: {
return aoh->getCenter();
}
default:
break;
}
return Base::Vector3d();
}
template <>
Base::Vector3d SketchObject::getPointForGeometry<>(const Part::GeomArcOfParabola *aop, PointPos PosId)
{
if (PosId == PointPos::start)
switch (PosId) {
case PointPos::start: {
return aop->getStartPoint();
else if (PosId == PointPos::end)
}
case PointPos::end: {
return aop->getEndPoint();
else if (PosId == PointPos::mid)
}
case PointPos::mid: {
return aop->getCenter();
}
default:
break;
}
return Base::Vector3d();
}
template <>
Base::Vector3d SketchObject::getPointForGeometry<>(const Part::GeomBSplineCurve *bsp, PointPos PosId)
{
if (PosId == PointPos::start)
switch (PosId) {
case PointPos::start: {
return bsp->getStartPoint();
else if (PosId == PointPos::end)
}
case PointPos::end: {
return bsp->getEndPoint();
}
default:
break;
}
return Base::Vector3d();
}