Remove redundant mutex locks

nyalldawson · nyalldawson · commit 312c6881514e · 2018-10-19T10:15:59.000+10:00
diff --git a/src/auth/basic/qgsauthbasicmethod.cpp b/src/auth/basic/qgsauthbasicmethod.cpp
@@ -67,7 +67,6 @@ bool QgsAuthBasicMethod::updateNetworkRequest( QNetworkRequest &request, const Q
     const QString &dataprovider )
 {
   Q_UNUSED( dataprovider )
-  QMutexLocker locker( &mMutex );
   QgsAuthMethodConfig mconfig = getMethodConfig( authcfg );
   if ( !mconfig.isValid() )
   {
diff --git a/src/auth/esritoken/qgsauthesritokenmethod.cpp b/src/auth/esritoken/qgsauthesritokenmethod.cpp
@@ -59,8 +59,7 @@ QString QgsAuthEsriTokenMethod::displayDescription() const
 bool QgsAuthEsriTokenMethod::updateNetworkRequest( QNetworkRequest &request, const QString &authcfg,
     const QString &dataprovider )
 {
-  Q_UNUSED( dataprovider )
-  QMutexLocker locker( &mMutex );
+  Q_UNUSED( dataprovider );
   QgsAuthMethodConfig mconfig = getMethodConfig( authcfg );
   if ( !mconfig.isValid() )
   {

Original file line number	Diff line number	Diff line change
`@@ -67,7 +67,6 @@ bool QgsAuthBasicMethod::updateNetworkRequest( QNetworkRequest &request, const Q`
`67`	`67`	`const QString &dataprovider )`
`68`	`68`	`{`
`69`	`69`	`Q_UNUSED( dataprovider )`
`70`		`- QMutexLocker locker( &mMutex );`
`71`	`70`	`QgsAuthMethodConfig mconfig = getMethodConfig( authcfg );`
`72`	`71`	`if ( !mconfig.isValid() )`
`73`	`72`	`{`
Original file line number	Diff line number	Diff line change
`@@ -59,8 +59,7 @@ QString QgsAuthEsriTokenMethod::displayDescription() const`
`59`	`59`	`bool QgsAuthEsriTokenMethod::updateNetworkRequest( QNetworkRequest &request, const QString &authcfg,`
`60`	`60`	`const QString &dataprovider )`
`61`	`61`	`{`
`62`		`- Q_UNUSED( dataprovider )`
`63`		`- QMutexLocker locker( &mMutex );`
	`62`	`+ Q_UNUSED( dataprovider );`
`64`	`63`	`QgsAuthMethodConfig mconfig = getMethodConfig( authcfg );`
`65`	`64`	`if ( !mconfig.isValid() )`
`66`	`65`	`{`