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

Console: rename PascalCase named methods to camelCase

Browse Source
This commit is contained in:
bofdahof
2025-03-30 00:50:27 +10:00
committed by Kacper Donat
parent 1dbc0638c3
commit ba2c2ca5ad
497 changed files with 2423 additions and 2425 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -278,7 +278,7 @@ App::DocumentObjectExecReturn* SketchObject::execute()
catch (const Base::Exception&) {
// 9/16/24: We used to clear the constraints here, but we no longer want to do that
// as missing reference geometry is not considered an error while we sort out sketcher UI.
// Base::Console().Error("%s\nClear constraints to external geometry\n", e.what());
// Base::Console().error("%s\nClear constraints to external geometry\n", e.what());
// we cannot trust the constraints of external geometries, so remove them
// delConstraintsToExternal();
}
@@ -577,13 +577,13 @@ int SketchObject::solve(bool updateGeoAfterSolving /*=true*/)
}
if (lastHasMalformedConstraints) {
Base::Console().Error(
Base::Console().error(
this->getFullLabel(),
QT_TRANSLATE_NOOP("Notifications", "The Sketch has malformed constraints!") "\n");
}
if (lastHasPartialRedundancies) {
Base::Console().Warning(
Base::Console().warning(
this->getFullLabel(),
QT_TRANSLATE_NOOP("Notifications",
"The Sketch has partially redundant constraints!") "\n");
@@ -1188,7 +1188,7 @@ void SketchObject::setConstraintExpression(int constNum, const std::string& newE
setExpression(path, std::move(expr));
}
catch (const Base::Exception&) {
Base::Console().Error("Failed to set constraint expression.");
Base::Console().error("Failed to set constraint expression.");
}
}
}
@@ -4000,7 +4000,7 @@ bool SketchObject::isExternalAllowed(App::Document* pDoc, App::DocumentObject* p
}
}
catch (Base::Exception& e) {
Base::Console().Warning(
Base::Console().warning(
"Probably, there is a circular reference in the document. Error: %s\n", e.what());
return true;// prohibiting this reference won't remove the problem anyway...
}
@@ -4072,7 +4072,7 @@ bool SketchObject::isCarbonCopyAllowed(App::Document* pDoc, App::DocumentObject*
}
}
catch (Base::Exception& e) {
Base::Console().Warning(
Base::Console().warning(
"Probably, there is a circular reference in the document. Error: %s\n", e.what());
return true;// prohibiting this reference won't remove the problem anyway...
}
@@ -4603,7 +4603,7 @@ std::vector<Part::Geometry*> SketchObject::getSymmetric(const std::vector<int>&
isStartEndInverted.insert(std::make_pair(geoId, false));
}
else {
Base::Console().Error("Unsupported Geometry!! Just copying it.\n");
Base::Console().error("Unsupported Geometry!! Just copying it.\n");
isStartEndInverted.insert(std::make_pair(geoId, false));
}
@@ -4624,7 +4624,7 @@ std::vector<Part::Geometry*> SketchObject::getSymmetric(const std::vector<int>&
}
else {
if (refPosId == Sketcher::PointPos::none) {
Base::Console().Error("Wrong PointPosId.\n");
Base::Console().error("Wrong PointPosId.\n");
return {};
}
refpoint = getPoint(georef, refPosId);
@@ -4762,7 +4762,7 @@ std::vector<Part::Geometry*> SketchObject::getSymmetric(const std::vector<int>&
isStartEndInverted.insert(std::make_pair(geoId, false));
}
else {
Base::Console().Error("Unsupported Geometry!! Just copying it.\n");
Base::Console().error("Unsupported Geometry!! Just copying it.\n");
isStartEndInverted.insert(std::make_pair(geoId, false));
}
@@ -5023,7 +5023,7 @@ int SketchObject::addCopy(const std::vector<int>& geoIdList, const Base::Vector3
iterfirstpoint = scp;
}
else {
Base::Console().Error("Unsupported Geometry!! Just skipping it.\n");
Base::Console().error("Unsupported Geometry!! Just skipping it.\n");
continue;
}
@@ -6413,7 +6413,7 @@ bool SketchObject::convertToNURBS(int GeoId)
}
}
catch (const Base::Exception& e) {
Base::Console().Error("%s\n", e.what());
Base::Console().error("%s\n", e.what());
// revert to original values
return false;
}
@@ -6499,7 +6499,7 @@ bool SketchObject::increaseBSplineDegree(int GeoId, int degreeincrement /*= 1*/)
bspline->increaseDegree(cdegree + degreeincrement);
}
catch (const Base::Exception& e) {
Base::Console().Error("%s\n", e.what());
Base::Console().error("%s\n", e.what());
return false;
}
@@ -6545,7 +6545,7 @@ bool SketchObject::decreaseBSplineDegree(int GeoId, int degreedecrement /*= 1*/)
bspline->approximate(Precision::Confusion(), 20, maxdegree, GeomAbs_C0);
}
catch (const Base::Exception& e) {
Base::Console().Error("%s\n", e.what());
Base::Console().error("%s\n", e.what());
return false;
}
@@ -6644,7 +6644,7 @@ bool SketchObject::modifyBSplineKnotMultiplicity(int GeoId, int knotIndex, int m
}
}
catch (const Base::Exception& e) {
Base::Console().Error("%s\n", e.what());
Base::Console().error("%s\n", e.what());
return false;
}
@@ -6789,7 +6789,7 @@ bool SketchObject::insertBSplineKnot(int GeoId, double param, int multiplicity)
bspline->insertKnot(param, multiplicity);
}
catch (const Base::Exception& e) {
Base::Console().Error("%s\n", e.what());
Base::Console().error("%s\n", e.what());
return false;
}
@@ -6972,7 +6972,7 @@ int SketchObject::carbonCopy(App::DocumentObject* pObj, bool construction)
if (Objects.size() != SubElements.size() || sObjects.size() != sSubElements.size()) {
assert(0 /*counts of objects and subelements in external geometry links do not match*/);
Base::Console().Error("Internal error: counts of objects and subelements in external "
Base::Console().error("Internal error: counts of objects and subelements in external "
"geometry links do not match\n");
return -1;
}
@@ -6982,7 +6982,7 @@ int SketchObject::carbonCopy(App::DocumentObject* pObj, bool construction)
int i = 0;
for (auto& obj : Objects) {
if (obj == sobj && SubElements[i] == sSubElements[si]) {
Base::Console().Error(
Base::Console().error(
"Link to %s already exists in this sketch. Delete the link and try again\n",
sSubElements[si].c_str());
return -1;
@@ -7003,7 +7003,7 @@ int SketchObject::carbonCopy(App::DocumentObject* pObj, bool construction)
rebuildExternalGeometry();
}
catch (const Base::Exception& e) {
Base::Console().Error("%s\n", e.what());
Base::Console().error("%s\n", e.what());
// revert to original values
ExternalGeometry.setValues(originalObjects, originalSubElements);
return -1;
@@ -7173,7 +7173,7 @@ int SketchObject::addExternal(App::DocumentObject *Obj, const char* SubName, boo
if (Objects.size() != SubElements.size()) {
assert(0 /*counts of objects and subelements in external geometry links do not match*/);
Base::Console().Error("Internal error: counts of objects and subelements in external "
Base::Console().error("Internal error: counts of objects and subelements in external "
"geometry links do not match\n");
return -1;
}
@@ -7184,7 +7184,7 @@ int SketchObject::addExternal(App::DocumentObject *Obj, const char* SubName, boo
if (Types[i] == (int)ExtType::Both
|| (Types[i] == (int)ExtType::Projection && !intersection)
|| (Types[i] == (int)ExtType::Intersection && intersection)) {
Base::Console().Error("Link to %s already exists in this sketch.\n", SubName);
Base::Console().error("Link to %s already exists in this sketch.\n", SubName);
return -1;
}
// Case where projections are already there when adding intersections.
@@ -7209,7 +7209,7 @@ int SketchObject::addExternal(App::DocumentObject *Obj, const char* SubName, boo
rebuildExternalGeometry(ext);
}
catch (const Base::Exception& e) {
Base::Console().Error("%s\n", e.what());
Base::Console().error("%s\n", e.what());
// revert to original values
ExternalGeometry.setValues(originalObjects, originalSubElements);
return -1;
@@ -7401,7 +7401,7 @@ int SketchObject::delAllExternal()
rebuildExternalGeometry();
}
catch (const Base::Exception& e) {
Base::Console().Error("%s\n", e.what());
Base::Console().error("%s\n", e.what());
// revert to original values
ExternalGeometry.setValues(originalObjects, originalSubElements);
for (Constraint* it : newConstraints)
@@ -7779,12 +7779,12 @@ void SketchObject::validateExternalLinks()
}
catch (Base::IndexError& indexError) {
removeBadLink = true;
Base::Console().Warning(
Base::Console().warning(
this->getFullLabel(), (indexError.getMessage() + "\n").c_str());
}
catch (Base::ValueError& valueError) {
removeBadLink = true;
Base::Console().Warning(
Base::Console().warning(
this->getFullLabel(), (valueError.getMessage() + "\n").c_str());
}
catch (Standard_Failure&) {
@@ -8540,7 +8540,7 @@ std::vector<TopoDS_Shape> projectShape(const TopoDS_Shape& inShape, const gp_Ax3
brep_hlr->Hide();
}
catch (const Standard_Failure& e) {
Base::Console().Error("GO::projectShape - OCC error - %s - while projecting shape\n",
Base::Console().error("GO::projectShape - OCC error - %s - while projecting shape\n",
e.GetMessageString());
throw Base::RuntimeError("SketchObject::projectShape - OCC error");
}
@@ -9955,7 +9955,7 @@ void SketchObject::onChanged(const App::Property* prop)
acceptGeometry();
}
else {
Base::Console().Error(
Base::Console().error(
this->getFullLabel() + " SketchObject::onChanged ",
QT_TRANSLATE_NOOP("Notifications", "Unmanaged change of Geometry Property "
"results in invalid constraint indices") "\n");
@@ -9986,7 +9986,7 @@ void SketchObject::onChanged(const App::Property* prop)
}
}
else {
Base::Console().Error(
Base::Console().error(
this->getFullLabel() + " SketchObject::onChanged ",
QT_TRANSLATE_NOOP("Notifications", "Unmanaged change of Constraint "
"Property results in invalid constraint indices") "\n");
@@ -10614,7 +10614,7 @@ void SketchObject::migrateSketch()
Constraints.setValues(std::move(newconstraints));
Base::Console().Critical(
Base::Console().critical(
this->getFullName(),
QT_TRANSLATE_NOOP("Notifications",
"Parabolas were migrated. Migrated files won't open in previous "
@@ -10675,13 +10675,13 @@ int SketchObject::changeConstraintsLocking(bool bLock)
cntSuccess++;
newVals[i] = constNew;
Base::Console().Log("Constraint%i will be affected\n", i + 1);
Base::Console().log("Constraint%i will be affected\n", i + 1);
}
}
this->Constraints.setValues(std::move(newVals));
Base::Console().Log("ChangeConstraintsLocking: affected %i of %i tangent/perp constraints\n",
Base::Console().log("ChangeConstraintsLocking: affected %i of %i tangent/perp constraints\n",
cntSuccess,
cntToBeAffected);
@@ -10768,13 +10768,13 @@ int SketchObject::port_reversedExternalArcs(bool justAnalyze)
if (affected) {
cntToBeAffected++;
newVals[ic] = constNew;
Base::Console().Log("Constraint%i will be affected\n", ic + 1);
Base::Console().log("Constraint%i will be affected\n", ic + 1);
};
}
if (!justAnalyze) {
this->Constraints.setValues(std::move(newVals));
Base::Console().Log("Swapped start/end of reversed external arcs in %i constraints\n",
Base::Console().log("Swapped start/end of reversed external arcs in %i constraints\n",
cntToBeAffected);
}
@@ -10880,7 +10880,7 @@ bool SketchObject::AutoLockTangencyAndPerpty(Constraint* cstr, bool bForce, bool
}
catch (Base::Exception& e) {
// failure to determine tangency type is not a big deal, so a warning.
Base::Console().Warning("Error in AutoLockTangency. %s \n", e.what());
Base::Console().warning("Error in AutoLockTangency. %s \n", e.what());
return false;
}
return true;