Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Fix QObject::connect lambda without capturing this warnings on msvc
  • Loading branch information
nyalldawson committed Nov 4, 2021
1 parent 3a97d92 commit f85d6c6
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
3 changes: 2 additions & 1 deletion src/app/browser/qgsinbuiltdataitemproviders.cpp
Expand Up @@ -1414,8 +1414,9 @@ void QgsDatabaseItemGuiProvider::populateContextMenu( QgsDataItem *item, QMenu *
{
QAction *sqlAction = new QAction( QObject::tr( "Execute SQL…" ), menu );

QObject::connect( sqlAction, &QAction::triggered, item, [ item, context ]
QObject::connect( sqlAction, &QAction::triggered, item, [ item, context, this ]
{
( void )this;
std::unique_ptr<QgsAbstractDatabaseProviderConnection> conn2( item->databaseConnection() );
// This should never happen but let's play safe
if ( ! conn2 )
Expand Down
5 changes: 3 additions & 2 deletions src/app/qgsapplayertreeviewmenuprovider.cpp
Expand Up @@ -226,7 +226,7 @@ QMenu *QgsAppLayerTreeViewMenuProvider::createContextMenu()
{
const std::unique_ptr< QgsAbstractDatabaseProviderConnection> conn { QgsMapLayerUtils::databaseConnection( layer ) };
if ( conn )
menu->addAction( QgsApplication::getThemeIcon( QStringLiteral( "/dbmanager.svg" ) ), tr( "Update SQL Layer…" ), menu, [ layer ]
menu->addAction( QgsApplication::getThemeIcon( QStringLiteral( "/dbmanager.svg" ) ), tr( "Update SQL Layer…" ), menu, [ layer, this ]
{
std::unique_ptr< QgsAbstractDatabaseProviderConnection> conn2 { QgsMapLayerUtils::databaseConnection( layer ) };
if ( conn2 )
Expand All @@ -244,8 +244,9 @@ QMenu *QgsAppLayerTreeViewMenuProvider::createContextMenu()
queryResultWidget->layout()->setMargin( 0 );
dialog.layout()->addWidget( queryResultWidget );

connect( queryResultWidget, &QgsQueryResultWidget::createSqlVectorLayer, queryResultWidget, [queryResultWidget, layer ]( const QString &, const QString &, const QgsAbstractDatabaseProviderConnection::SqlVectorLayerOptions & options )
connect( queryResultWidget, &QgsQueryResultWidget::createSqlVectorLayer, queryResultWidget, [queryResultWidget, layer, this ]( const QString &, const QString &, const QgsAbstractDatabaseProviderConnection::SqlVectorLayerOptions & options )
{
( void )this;
std::unique_ptr< QgsAbstractDatabaseProviderConnection> conn3 { QgsMapLayerUtils::databaseConnection( layer ) };
if ( conn3 )
{
Expand Down

0 comments on commit f85d6c6

Please sign in to comment.