[app] Use unique pointer for query builder

elpaso · elpaso · commit 17ca25fb6958 · 2017-11-21T16:06:59.000+01:00
diff --git a/src/app/qgisapp.cpp b/src/app/qgisapp.cpp
@@ -8928,7 +8928,7 @@ void QgisApp::layerSubsetString()
   }
 
   // launch the query builder
-  QgsQueryBuilder *qb = new QgsQueryBuilder( vlayer, this );
+  std::unique_ptr<QgsQueryBuilder> qb( new QgsQueryBuilder( vlayer, this ) );
   QString subsetBefore = vlayer->subsetString();
 
   // Set the sql in the query builder to the same in the prop dialog
@@ -8947,8 +8947,6 @@ void QgisApp::layerSubsetString()
     }
   }
 
-  // delete the query builder object
-  delete qb;
 }
 
 void QgisApp::saveLastMousePosition( const QgsPointXY &p )

Original file line number	Diff line number	Diff line change
`@@ -8928,7 +8928,7 @@ void QgisApp::layerSubsetString()`
`8928`	`8928`	`}`
`8929`	`8929`
`8930`	`8930`	`// launch the query builder`
`8931`		`- QgsQueryBuilder *qb = new QgsQueryBuilder( vlayer, this );`
	`8931`	`+ std::unique_ptr<QgsQueryBuilder> qb( new QgsQueryBuilder( vlayer, this ) );`
`8932`	`8932`	`QString subsetBefore = vlayer->subsetString();`
`8933`	`8933`
`8934`	`8934`	`// Set the sql in the query builder to the same in the prop dialog`
`@@ -8947,8 +8947,6 @@ void QgisApp::layerSubsetString()`
`8947`	`8947`	`}`
`8948`	`8948`	`}`
`8949`	`8949`
`8950`		`- // delete the query builder object`
`8951`		`- delete qb;`
`8952`	`8950`	`}`
`8953`	`8951`
`8954`	`8952`	`void QgisApp::saveLastMousePosition( const QgsPointXY &p )`