Merge pull request #20835 from oursland/python-binding-generator-const

python: Make const methods declared const in C++.
This commit is contained in:
Chris Hennes
2025-04-21 10:58:21 -05:00
committed by GitHub
73 changed files with 554 additions and 553 deletions

View File

@@ -210,7 +210,7 @@ void CommandPy::setParameters(Py::Dict arg)
// GCode methods
PyObject* CommandPy::toGCode(PyObject* args)
PyObject* CommandPy::toGCode(PyObject* args) const
{
if (PyArg_ParseTuple(args, "")) {
return PyUnicode_FromString(getCommandPtr()->toGCode().c_str());

View File

@@ -139,7 +139,7 @@ Py::Object PathPy::getBoundBox() const
// specific methods
PyObject* PathPy::copy(PyObject* args)
PyObject* PathPy::copy(PyObject* args) const
{
if (PyArg_ParseTuple(args, "")) {
return new PathPy(new Path::Toolpath(*getToolpathPtr()));
@@ -192,7 +192,7 @@ PyObject* PathPy::deleteCommand(PyObject* args)
Py_Error(PyExc_TypeError, "Wrong parameters - expected an integer (optional)");
}
PyObject* PathPy::getCycleTime(PyObject* args)
PyObject* PathPy::getCycleTime(PyObject* args) const
{
double hFeed, vFeed, hRapid, vRapid;
if (PyArg_ParseTuple(args, "dddd", &hFeed, &vFeed, &hRapid, &vRapid)) {
@@ -203,7 +203,7 @@ PyObject* PathPy::getCycleTime(PyObject* args)
// GCode methods
PyObject* PathPy::toGCode(PyObject* args)
PyObject* PathPy::toGCode(PyObject* args) const
{
if (PyArg_ParseTuple(args, "")) {
std::string result = getToolpathPtr()->toGCode();

View File

@@ -168,7 +168,7 @@ Py::Object VoronoiCellPy::getIncidentEdge() const
return Py::asObject(new VoronoiEdgePy(new VoronoiEdge(c->dia, c->ptr->incident_edge())));
}
PyObject* VoronoiCellPy::containsPoint(PyObject* args)
PyObject* VoronoiCellPy::containsPoint(PyObject* args) const
{
VoronoiCell* c = getVoronoiCellFromPy(this, args);
PyObject* chk = c->ptr->contains_point() ? Py_True : Py_False;
@@ -176,7 +176,7 @@ PyObject* VoronoiCellPy::containsPoint(PyObject* args)
return chk;
}
PyObject* VoronoiCellPy::containsSegment(PyObject* args)
PyObject* VoronoiCellPy::containsSegment(PyObject* args) const
{
VoronoiCell* c = getVoronoiCellFromPy(this, args);
PyObject* chk = c->ptr->contains_segment() ? Py_True : Py_False;
@@ -184,7 +184,7 @@ PyObject* VoronoiCellPy::containsSegment(PyObject* args)
return chk;
}
PyObject* VoronoiCellPy::isDegenerate(PyObject* args)
PyObject* VoronoiCellPy::isDegenerate(PyObject* args) const
{
VoronoiCell* c = getVoronoiCellFromPy(this, args);
PyObject* chk = c->ptr->is_degenerate() ? Py_True : Py_False;
@@ -192,7 +192,7 @@ PyObject* VoronoiCellPy::isDegenerate(PyObject* args)
return chk;
}
PyObject* VoronoiCellPy::getSource(PyObject* args)
PyObject* VoronoiCellPy::getSource(PyObject* args) const
{
double z = 0;
if (!PyArg_ParseTuple(args, "|d", &z)) {

View File

@@ -397,7 +397,7 @@ Py::Object VoronoiEdgePy::getCell() const
}
PyObject* VoronoiEdgePy::isFinite(PyObject* args)
PyObject* VoronoiEdgePy::isFinite(PyObject* args) const
{
VoronoiEdge* e = getVoronoiEdgeFromPy(this, args);
PyObject* chk = e->ptr->is_finite() ? Py_True : Py_False;
@@ -405,7 +405,7 @@ PyObject* VoronoiEdgePy::isFinite(PyObject* args)
return chk;
}
PyObject* VoronoiEdgePy::isInfinite(PyObject* args)
PyObject* VoronoiEdgePy::isInfinite(PyObject* args) const
{
VoronoiEdge* e = getVoronoiEdgeFromPy(this, args);
PyObject* chk = e->ptr->is_infinite() ? Py_True : Py_False;
@@ -413,7 +413,7 @@ PyObject* VoronoiEdgePy::isInfinite(PyObject* args)
return chk;
}
PyObject* VoronoiEdgePy::isLinear(PyObject* args)
PyObject* VoronoiEdgePy::isLinear(PyObject* args) const
{
VoronoiEdge* e = getVoronoiEdgeFromPy(this, args);
PyObject* chk = e->ptr->is_linear() ? Py_True : Py_False;
@@ -421,7 +421,7 @@ PyObject* VoronoiEdgePy::isLinear(PyObject* args)
return chk;
}
PyObject* VoronoiEdgePy::isCurved(PyObject* args)
PyObject* VoronoiEdgePy::isCurved(PyObject* args) const
{
VoronoiEdge* e = getVoronoiEdgeFromPy(this, args);
PyObject* chk = e->ptr->is_curved() ? Py_True : Py_False;
@@ -429,7 +429,7 @@ PyObject* VoronoiEdgePy::isCurved(PyObject* args)
return chk;
}
PyObject* VoronoiEdgePy::isPrimary(PyObject* args)
PyObject* VoronoiEdgePy::isPrimary(PyObject* args) const
{
VoronoiEdge* e = getVoronoiEdgeFromPy(this, args);
PyObject* chk = e->ptr->is_primary() ? Py_True : Py_False;
@@ -437,7 +437,7 @@ PyObject* VoronoiEdgePy::isPrimary(PyObject* args)
return chk;
}
PyObject* VoronoiEdgePy::isSecondary(PyObject* args)
PyObject* VoronoiEdgePy::isSecondary(PyObject* args) const
{
VoronoiEdge* e = getVoronoiEdgeFromPy(this, args);
PyObject* chk = e->ptr->is_secondary() ? Py_True : Py_False;
@@ -445,7 +445,7 @@ PyObject* VoronoiEdgePy::isSecondary(PyObject* args)
return chk;
}
PyObject* VoronoiEdgePy::isBorderline(PyObject* args)
PyObject* VoronoiEdgePy::isBorderline(PyObject* args) const
{
VoronoiEdge* e = getVoronoiEdgeFromPy(this, args);
PyObject* chk = Py_False;
@@ -464,7 +464,7 @@ PyObject* VoronoiEdgePy::isBorderline(PyObject* args)
return chk;
}
PyObject* VoronoiEdgePy::toShape(PyObject* args)
PyObject* VoronoiEdgePy::toShape(PyObject* args) const
{
double z0 = 0.0;
double z1 = std::numeric_limits<double>::max();
@@ -679,7 +679,7 @@ PyObject* VoronoiEdgePy::toShape(PyObject* args)
}
PyObject* VoronoiEdgePy::getDistances(PyObject* args)
PyObject* VoronoiEdgePy::getDistances(PyObject* args) const
{
VoronoiEdge* e = getVoronoiEdgeFromPy(this, args);
Py::List list;
@@ -687,7 +687,7 @@ PyObject* VoronoiEdgePy::getDistances(PyObject* args)
return Py::new_reference_to(list);
}
PyObject* VoronoiEdgePy::getSegmentAngle(PyObject* args)
PyObject* VoronoiEdgePy::getSegmentAngle(PyObject* args) const
{
using std::numbers::pi;

View File

@@ -120,7 +120,7 @@ PyObject* VoronoiPy::construct(PyObject* args)
return Py_None;
}
PyObject* VoronoiPy::numCells(PyObject* args)
PyObject* VoronoiPy::numCells(PyObject* args) const
{
if (!PyArg_ParseTuple(args, "")) {
throw Py::RuntimeError("no arguments accepted");
@@ -128,7 +128,7 @@ PyObject* VoronoiPy::numCells(PyObject* args)
return PyLong_FromLong(getVoronoiPtr()->numCells());
}
PyObject* VoronoiPy::numEdges(PyObject* args)
PyObject* VoronoiPy::numEdges(PyObject* args) const
{
if (!PyArg_ParseTuple(args, "")) {
throw Py::RuntimeError("no arguments accepted");
@@ -136,7 +136,7 @@ PyObject* VoronoiPy::numEdges(PyObject* args)
return PyLong_FromLong(getVoronoiPtr()->numEdges());
}
PyObject* VoronoiPy::numVertices(PyObject* args)
PyObject* VoronoiPy::numVertices(PyObject* args) const
{
if (!PyArg_ParseTuple(args, "")) {
throw Py::RuntimeError("no arguments accepted");
@@ -297,7 +297,7 @@ PyObject* VoronoiPy::resetColor(PyObject* args)
return Py_None;
}
PyObject* VoronoiPy::getPoints(PyObject* args)
PyObject* VoronoiPy::getPoints(PyObject* args) const
{
double z = 0;
if (!PyArg_ParseTuple(args, "|d", &z)) {
@@ -312,7 +312,7 @@ PyObject* VoronoiPy::getPoints(PyObject* args)
return Py::new_reference_to(list);
}
PyObject* VoronoiPy::getSegments(PyObject* args)
PyObject* VoronoiPy::getSegments(PyObject* args) const
{
double z = 0;
if (!PyArg_ParseTuple(args, "|d", &z)) {
@@ -331,7 +331,7 @@ PyObject* VoronoiPy::getSegments(PyObject* args)
return Py::new_reference_to(list);
}
PyObject* VoronoiPy::numPoints(PyObject* args)
PyObject* VoronoiPy::numPoints(PyObject* args) const
{
if (!PyArg_ParseTuple(args, "")) {
throw Py::RuntimeError("no arguments accepted");
@@ -339,7 +339,7 @@ PyObject* VoronoiPy::numPoints(PyObject* args)
return PyLong_FromLong(getVoronoiPtr()->vd->points.size());
}
PyObject* VoronoiPy::numSegments(PyObject* args)
PyObject* VoronoiPy::numSegments(PyObject* args) const
{
if (!PyArg_ParseTuple(args, "")) {
throw Py::RuntimeError("no arguments accepted");
@@ -347,9 +347,7 @@ PyObject* VoronoiPy::numSegments(PyObject* args)
return PyLong_FromLong(getVoronoiPtr()->vd->segments.size());
}
// custom attributes get/set
PyObject* VoronoiPy::getCustomAttributes(const char* /*attr*/) const
{
return nullptr;

View File

@@ -147,7 +147,7 @@ Py::Object VoronoiVertexPy::getIncidentEdge() const
return Py::asObject(new VoronoiEdgePy(new VoronoiEdge(v->dia, v->ptr->incident_edge())));
}
PyObject* VoronoiVertexPy::toPoint(PyObject* args)
PyObject* VoronoiVertexPy::toPoint(PyObject* args) const
{
double z = 0.0;
if (!PyArg_ParseTuple(args, "|d", &z)) {