[education/rkward] rkward: Remove explicit warnings

Thomas Friedrichsmeier null at kde.org
Sun Sep 21 20:45:24 BST 2025


Git commit 3d491005877b9e7c0c8bc0c7b1dec08b6e94123f by Thomas Friedrichsmeier.
Committed on 21/09/2025 at 19:36.
Pushed by tfry into branch 'master'.

Remove explicit warnings

M  +1    -3    rkward/core/rcontainerobject.cpp
M  +1    -3    rkward/core/robject.cpp
M  +1    -3    rkward/dataeditor/rkeditordataframe.cpp
M  +1    -3    rkward/plugin/rkcomponentproperties.cpp
M  +1    -3    rkward/plugin/rktabpage.cpp
M  +1    -3    rkward/rbackend/rkstructuregetter.cpp
M  +1    -3    rkward/rbackend/rkwarddevice/rkgraphicsdevice_frontendtransmitter.cpp
M  +2    -3    rkward/windows/rkcommandeditorwindow.cpp

https://invent.kde.org/education/rkward/-/commit/3d491005877b9e7c0c8bc0c7b1dec08b6e94123f

diff --git a/rkward/core/rcontainerobject.cpp b/rkward/core/rcontainerobject.cpp
index 0dca24f18..4d9b82d5f 100644
--- a/rkward/core/rcontainerobject.cpp
+++ b/rkward/core/rcontainerobject.cpp
@@ -283,9 +283,7 @@ RObject *RContainerObject::createPendingChild(const QString &name, int position,
 	RK_TRACE(OBJECTS);
 	RK_ASSERT(isType(GlobalEnv) || isInGlobalEnv());
 
-#ifdef __GNUC__
-#	warning TODO validize name
-#endif
+	// TODO: validize name? or is this always called with valid name?
 	RObject *ret;
 	if (container) {
 		ret = new RContainerObject(this, name);
diff --git a/rkward/core/robject.cpp b/rkward/core/robject.cpp
index 4e76004ce..287081993 100644
--- a/rkward/core/robject.cpp
+++ b/rkward/core/robject.cpp
@@ -514,9 +514,7 @@ bool RObject::updateDimensions(RData *new_data) {
 				return (true);
 			}
 		} else {
-#ifdef __GNUC__
-#	warning TODO: ugly hack. Should be moved to RKVariable, somehow.
-#endif
+			// TODO / HACK: Should be move to RKVariable, instead
 			if (type & Variable) static_cast<RKVariable *>(this)->extendToLength(new_dimensions[0]);
 
 			dimensions = new_dimensions;
diff --git a/rkward/dataeditor/rkeditordataframe.cpp b/rkward/dataeditor/rkeditordataframe.cpp
index cce81d0f4..4b664786b 100644
--- a/rkward/dataeditor/rkeditordataframe.cpp
+++ b/rkward/dataeditor/rkeditordataframe.cpp
@@ -104,11 +104,9 @@ void RKEditorDataFrame::waitForLoad() {
 	    open_chain);
 }
 
+// TODO: remove here, use model API, directly
 void RKEditorDataFrame::restoreObject(RObject *object) {
 	RK_TRACE(EDITOR);
 
-#ifdef __GNUC__
-#	warning TODO: this interface should be moved to the model for good.
-#endif
 	datamodel->restoreObject(object, nullptr);
 }
diff --git a/rkward/plugin/rkcomponentproperties.cpp b/rkward/plugin/rkcomponentproperties.cpp
index 9b43013bd..e1e8202b1 100644
--- a/rkward/plugin/rkcomponentproperties.cpp
+++ b/rkward/plugin/rkcomponentproperties.cpp
@@ -132,9 +132,7 @@ void RKComponentPropertyBase::connectToGovernor(RKComponentPropertyBase *governo
 
 void RKComponentPropertyBase::governorValueChanged(RKComponentPropertyBase *property) {
 	RK_TRACE(PLUGIN);
-#ifdef __GNUC__
-#	warning TOOD: connected value should be stored as qvariant
-#endif
+	// TODO:  connected value should be stored as qvariant
 	setValue(property->value(governor_modifier).toString());
 }
 
diff --git a/rkward/plugin/rktabpage.cpp b/rkward/plugin/rktabpage.cpp
index 66a2ed7d5..d500b1903 100644
--- a/rkward/plugin/rktabpage.cpp
+++ b/rkward/plugin/rktabpage.cpp
@@ -42,9 +42,7 @@ void RKTabPage::visibleEnabledChanged(RKComponentPropertyBase *property) {
 	if (property == visibility_property) {
 		if (visibility_property->boolValue()) {
 			if (!inserted) {
-#ifdef __GNUC__
-#	warning this may not be reliable, if an earlier page is invisible as well
-#endif
+				// TODO / WARNING: this may not be reliable, if an earlier page is invisible as well
 				tabbook->insertTab(index, this, label);
 				inserted = true;
 			}
diff --git a/rkward/rbackend/rkstructuregetter.cpp b/rkward/rbackend/rkstructuregetter.cpp
index 531dafddb..c4ee07329 100644
--- a/rkward/rbackend/rkstructuregetter.cpp
+++ b/rkward/rbackend/rkstructuregetter.cpp
@@ -223,9 +223,7 @@ void RKStructureGetter::getStructureWorker(SEXP val, const QString &name, int ad
 
 	// basic classification
 	for (int i = classes.size() - 1; i >= 0; --i) {
-#ifdef __GNUC__
-#	warning : Using is.data.frame() may be more reliable (would need to be called only on List-objects, thus no major performance hit)
-#endif
+		// TODO: Using is.data.frame() may be more reliable (would need to be called only on List-objects, thus no major performance hit)
 		if (classes[i] == QLatin1String("data.frame")) type |= RObject::DataFrame;
 	}
 
diff --git a/rkward/rbackend/rkwarddevice/rkgraphicsdevice_frontendtransmitter.cpp b/rkward/rbackend/rkwarddevice/rkgraphicsdevice_frontendtransmitter.cpp
index 75c087734..b3e0d1e6a 100644
--- a/rkward/rbackend/rkwarddevice/rkgraphicsdevice_frontendtransmitter.cpp
+++ b/rkward/rbackend/rkwarddevice/rkgraphicsdevice_frontendtransmitter.cpp
@@ -195,9 +195,7 @@ static QFont readFont(QDataStream &instream) {
 	quint8 fontface;
 	QString fontfamily;
 	instream >> cex >> ps >> lineheight >> fontface >> fontfamily;
-#ifdef __GNUC__
-#	warning TODO deal with line-height
-#endif
+	// TODO: deal with lineheight!
 	QFont ret;
 	if (!(fontfamily.isEmpty() || fontfamily == QLatin1String("Symbol"))) ret.setFamily(fontfamily); // NOTE: QPainter won't paint with "Symbol", somehow
 	if (fontface == 2 || fontface == 4) ret.setWeight(QFont::Bold);
diff --git a/rkward/windows/rkcommandeditorwindow.cpp b/rkward/windows/rkcommandeditorwindow.cpp
index 2fb9ad71e..233d0fdc3 100644
--- a/rkward/windows/rkcommandeditorwindow.cpp
+++ b/rkward/windows/rkcommandeditorwindow.cpp
@@ -252,9 +252,8 @@ RKCommandEditorWindow::RKCommandEditorWindow(QWidget *parent, const QUrl &_url,
 
 	connect(m_doc, &KTextEditor::Document::documentUrlChanged, this, &RKCommandEditorWindow::urlChanged);
 	connect(m_doc, &KTextEditor::Document::modifiedChanged, this, &RKCommandEditorWindow::updateCaption); // of course most of the time this causes a redundant call to updateCaption. Not if a modification is undone, however.
-#ifdef __GNUC__
-#	warning remove this in favor of KTextEditor::Document::restore()
-#endif
+
+	// TODO: remove custom autosave mechanism in favor of KTextEditor::Document::restore()
 	connect(m_doc, &KTextEditor::Document::modifiedChanged, this, &RKCommandEditorWindow::autoSaveHandlerModifiedChanged);
 	connect(m_doc, &KTextEditor::Document::textChanged, this, &RKCommandEditorWindow::textChanged);
 	connect(m_view, &KTextEditor::View::selectionChanged, this, &RKCommandEditorWindow::selectionChanged);



More information about the rkward-tracker mailing list