gui: Don't use QOverload since older versions of Qt do not have it.

pololu · Apr 20, 2018 · 00c7306 · 00c7306
1 parent f931685
commit 00c7306
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/gui/qt/current_spin_box.cpp b/gui/qt/current_spin_box.cpp
@@ -27,10 +27,12 @@ current_spin_box::current_spin_box(QWidget * parent)
   // We rely on Qt's documented behavior that if multiple slots are connected to
   // one signal, they are called in the order they were connected.
   //
+  // Note: We don't use QOverload so we can support Qt 5.5 in Ubuntu 16.04.
+  //
   // Warning: Don't use a QSignalBlocker with this class unless you really know
   // what you're doing, since you'll block this signal.
-  connect(this, QOverload<int>::of(&QSpinBox::valueChanged),
-    this, &current_spin_box::set_code_from_value);
+  connect(this, SIGNAL(valueChanged(int)),
+    this, SLOT(set_code_from_value()));
 
   connect(this, &QSpinBox::editingFinished,
     this, &current_spin_box::editing_finished);

diff --git a/gui/qt/current_spin_box.h b/gui/qt/current_spin_box.h
@@ -16,10 +16,10 @@ class current_spin_box : public QSpinBox
 
 private slots:
   void editing_finished();
+  void set_code_from_value();
 
 private:
   void fix_code_if_not_allowed();
-  void set_code_from_value();
   void set_value_from_code();
 
   QMap<int, int> mapping;