Fix Qt::UniqueConnection with lambda

qgis · Nov 5, 2019 · 9346dba · 9346dba
1 parent 8f52da7
commit 9346dba
Show file tree

Hide file tree

Showing 5 changed files with 27 additions and 14 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -761,7 +761,7 @@ IF (WITH_CORE)
   MARK_AS_ADVANCED (ADD_CLAZY_CHECKS)
   IF (ADD_CLAZY_CHECKS)
   SET(CMAKE_CXX_BASE_FLAGS "${CMAKE_CXX_FLAGS}")
-  SET(CLAZY_BASE_CHECKS "connect-3arg-lambda")
+  SET(CLAZY_BASE_CHECKS "connect-3arg-lambda,lambda-unique-connection")
   SET(CMAKE_CXX_FLAGS "${CMAKE_CXX_BASE_FLAGS} -Xclang -plugin-arg-clazy -Xclang ${CLAZY_BASE_CHECKS}")
   ENDIF (ADD_CLAZY_CHECKS)
 ENDIF (WITH_CORE)

diff --git a/src/gui/editorwidgets/qgshtmlwidgetwrapper.cpp b/src/gui/editorwidgets/qgshtmlwidgetwrapper.cpp
@@ -49,12 +49,8 @@ void QgsHtmlWidgetWrapper::initWidget( QWidget *editor )
   const int horizontalDpi = qApp->desktop()->screen()->logicalDpiX();
   mWidget->setZoomFactor( horizontalDpi / 96.0 );
 
-  auto page = mWidget->page();
-  connect( page, &QWebPage::contentsChanged, this, [ = ]
-  {
-    auto docHeight { page->mainFrame()->contentsSize().height() };
-    mWidget->setFixedHeight( docHeight );
-  }, Qt::ConnectionType::UniqueConnection );
+  QWebPage *page = mWidget->page();
+  connect( page, &QWebPage::contentsChanged, this, &QgsHtmlWidgetWrapper::fixHeight, Qt::ConnectionType::UniqueConnection );
 #endif
 
 }
@@ -95,6 +91,15 @@ void QgsHtmlWidgetWrapper::setHtmlContext( )
   mWidget->setHtml( mHtmlCode );
 }
 
+#ifdef WITH_QTWEBKIT
+void QgsHtmlWidgetWrapper::fixHeight()
+{
+  QWebPage *page = mWidget->page();
+  int docHeight { page->mainFrame()->contentsSize().height() };
+  mWidget->setFixedHeight( docHeight );
+}
+#endif
+
 void QgsHtmlWidgetWrapper::setFeature( const QgsFeature &feature )
 {
   if ( !mWidget )

diff --git a/src/gui/editorwidgets/qgshtmlwidgetwrapper.h b/src/gui/editorwidgets/qgshtmlwidgetwrapper.h
@@ -59,6 +59,9 @@ class GUI_EXPORT QgsHtmlWidgetWrapper : public QgsWidgetWrapper
   private slots:
     //! sets the html context with the current values
     void setHtmlContext( );
+#ifdef WITH_QTWEBKIT
+    void fixHeight();
+#endif
 
   private:
     QString mHtmlCode;

diff --git a/src/gui/editorwidgets/qgsvaluerelationwidgetwrapper.cpp b/src/gui/editorwidgets/qgsvaluerelationwidgetwrapper.cpp
@@ -172,13 +172,7 @@ void QgsValueRelationWidgetWrapper::initWidget( QWidget *editor )
   }
   else if ( mLineEdit )
   {
-    connect( mLineEdit, &QLineEdit::textChanged, this, [ = ]( const QString & value )
-    {
-      Q_NOWARN_DEPRECATED_PUSH
-      emit valueChanged( value );
-      Q_NOWARN_DEPRECATED_POP
-      emit valuesChanged( value );
-    }, Qt::UniqueConnection );
+    connect( mLineEdit, &QLineEdit::textChanged, this, &QgsValueRelationWidgetWrapper::emitValueChanged, Qt::UniqueConnection );
   }
 }
 
@@ -472,3 +466,11 @@ QList<QgsVectorLayerRef> QgsValueRelationWidgetWrapper::layerDependencies() cons
   }
   return result;
 }
+
+void QgsValueRelationWidgetWrapper::emitValueChanged( const QString &value )
+{
+  Q_NOWARN_DEPRECATED_PUSH
+  emit valueChanged( value );
+  Q_NOWARN_DEPRECATED_POP
+  emit valuesChanged( value );
+}
diff --git a/src/gui/editorwidgets/qgsvaluerelationwidgetwrapper.h b/src/gui/editorwidgets/qgsvaluerelationwidgetwrapper.h
@@ -104,6 +104,9 @@ class GUI_EXPORT QgsValueRelationWidgetWrapper : public QgsEditorWidgetWrapper
 
     QList<QgsVectorLayerRef> layerDependencies() const override;
 
+  private slots:
+    void emitValueChanged( const QString &value );
+
   private:
     void updateValues( const QVariant &value, const QVariantList & = QVariantList() ) override;