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

Sketcher: Refactor GeoDef and ConstrDef structs

Browse Source 
- Fix comment for mid-point
- Use in-class member initializers instead of constructor
- Remove redundant default constructor (got redundant after initializers)
- Made comments doxygen compatible
This commit is contained in:
Benjamin Nauck
2025-06-12 09:09:02 +02:00
committed by Chris Hennes
parent acac7ac51b
commit ff8e1efeba
1 changed files with 11 additions and 26 deletions
Download Patch File Download Diff File Expand all files Collapse all files

37
src/Mod/Sketcher/App/Sketch.h
View File

@@ -538,36 +538,21 @@ private:
/// container element to store and work with the geometric elements of this sketch
struct GeoDef
{
GeoDef()
: geo(nullptr)
, type(None)
, external(false)
, index(-1)
, startPointId(-1)
, midPointId(-1)
, endPointId(-1)
{}
Part::Geometry* geo; // pointer to the geometry
GeoType type; // type of the geometry
bool external; // flag for external geometries
int index; // index in the corresponding storage vector (Lines, Arcs, Circles, ...)
int startPointId; // index in Points of the start point of this geometry
int midPointId; // index in Points of the start point of this geometry
int endPointId; // index in Points of the end point of this geometry
Part::Geometry* geo {}; ///< Pointer to the geometry
GeoType type = None; ///< Type of the geometry
bool external = false; ///< Flag for external geometries
int index = -1; ///< Index in the corresponding storage vector (Lines, Arcs, ...)
int startPointId = -1; ///< Index in Points of the start point of this geometry
int midPointId = -1; ///< Index in Points of the mid point of this geometry
int endPointId = -1; ///< Index in Points of the end point of this geometry
};
/// container element to store and work with the constraints of this sketch
struct ConstrDef
{
ConstrDef()
: constr(nullptr)
, driving(true)
, value(nullptr)
, secondvalue(nullptr)
{}
Constraint* constr; // pointer to the constraint
bool driving;
double* value;
double* secondvalue; // this is needed for SnellsLaw
Constraint* constr {}; ///< Pointer to the constraint
bool driving = true;
double* value {};
double* secondvalue {}; ///< Needed for SnellsLaw
};
std::vector<GeoDef> Geoms;