Fix a couple of build warnings

bsmokeman · Oct 2, 2024 · ff975c5 · ff975c5
1 parent afd511f
commit ff975c5
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/src/tableModels/StepTableModelBase.h b/src/tableModels/StepTableModelBase.h
@@ -101,7 +101,7 @@ class StepTableModelBase : public CuriouslyRecurringTemplateBase<StepTableModelP
 
    //! \returns true if \c step is successfully found and removed.
    bool doRemoveStep(std::shared_ptr<StepClass> step) {
-      int ii {this->derived().rows.indexOf(step)};
+      int ii {static_cast<int>(this->derived().rows.indexOf(step))};
       if (ii >= 0) {
          qDebug() <<
             Q_FUNC_INFO << "Removing" << StepClass::staticMetaObject.className() << step->name() << "(#" <<

diff --git a/src/tableModels/TableModelBase.h b/src/tableModels/TableModelBase.h
@@ -737,7 +737,7 @@ class TableModelBase : public CuriouslyRecurringTemplateBase<TableModelTraits, D
          // with optional values.
          //
          // ItemDelegate::writeDataToModel should have just given us a raw string
-         Q_ASSERT(value.canConvert(QVariant::String));
+         Q_ASSERT(value.canConvert<QString>());
 
          //
          // For cases where we have an Amount and a drop-down chooser to select PhysicalQuantity (eg between Mass and

diff --git a/src/widgets/SmartLabel.h b/src/widgets/SmartLabel.h
@@ -183,14 +183,14 @@ class SmartLabel : public QLabel, public SmartBase<SmartLabel> {
     * \brief We override the \c QWidget event handlers \c enterEvent and \c leaveEvent to implement mouse-over effects
     *        on the label text - specifically to give the user a visual clue that the label text is (right)-clickable
     */
-   virtual void enterEvent(QEvent* event);
-   virtual void leaveEvent(QEvent* event);
+   virtual void enterEvent(QEvent* event) override;
+   virtual void leaveEvent(QEvent* event) override;
 
    /**
     * \brief We override the \c QWidget event handler \c mouseReleaseEvent to capture left mouse clicks on us.  (Right
     *        clicks get notified to us via the \c QWidget::customContextMenuRequested signal.)
     */
-   virtual void mouseReleaseEvent(QMouseEvent * event);
+   virtual void mouseReleaseEvent(QMouseEvent * event) override;
 
 
 private: