Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[mssql] Fix more clazy/clang warnings
  • Loading branch information
nyalldawson committed Oct 4, 2018
1 parent c5ecc64 commit 0729653
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 8 deletions.
2 changes: 0 additions & 2 deletions src/providers/mssql/qgsmssqldataitems.cpp
Expand Up @@ -136,8 +136,6 @@ QVector<QgsDataItem *> QgsMssqlConnectionItem::createChildren()
return children;
}

QString connectionName = db.connectionName();

// build sql statement
QString query( QStringLiteral( "select " ) );
if ( mUseGeometryColumns )
Expand Down
10 changes: 5 additions & 5 deletions src/providers/mssql/qgsmssqlfeatureiterator.cpp
Expand Up @@ -229,7 +229,7 @@ void QgsMssqlFeatureIterator::BuildStatement( const QgsFeatureRequest &request )
{
QString part;
part = compiler.result();
part += clause.ascending() ? " ASC" : " DESC";
part += clause.ascending() ? QStringLiteral( " ASC" ) : QStringLiteral( " DESC" );
orderByParts << part;
}
else
Expand Down Expand Up @@ -292,7 +292,7 @@ bool QgsMssqlFeatureIterator::fetchFeature( QgsFeature &feature )

if ( !mDatabase.open() )
{
QgsDebugMsg( "Failed to open database" );
QgsDebugMsg( QStringLiteral( "Failed to open database" ) );
QgsDebugMsg( mDatabase.lastError().text() );
return false;
}
Expand All @@ -310,7 +310,7 @@ bool QgsMssqlFeatureIterator::fetchFeature( QgsFeature &feature )

if ( !mQuery->isActive() )
{
QgsDebugMsg( "Read attempt on inactive query" );
QgsDebugMsg( QStringLiteral( "Read attempt on inactive query" ) );
return false;
}

Expand All @@ -336,7 +336,7 @@ bool QgsMssqlFeatureIterator::fetchFeature( QgsFeature &feature )
QByteArray ar = mQuery->record().value( mSource->mGeometryColName ).toByteArray();
if ( !ar.isEmpty() )
{
if ( unsigned char *wkb = mParser.ParseSqlGeometry( ( unsigned char * )ar.data(), ar.size() ) )
if ( unsigned char *wkb = mParser.ParseSqlGeometry( reinterpret_cast< unsigned char * >( ar.data() ), ar.size() ) )
{
QgsGeometry g;
g.fromWkb( wkb, mParser.GetWkbLen() );
Expand Down Expand Up @@ -374,7 +374,7 @@ bool QgsMssqlFeatureIterator::rewind()

if ( mStatement.isEmpty() )
{
QgsDebugMsg( "QgsMssqlFeatureIterator::rewind on empty statement" );
QgsDebugMsg( QStringLiteral( "QgsMssqlFeatureIterator::rewind on empty statement" ) );
return false;
}

Expand Down
2 changes: 1 addition & 1 deletion src/providers/mssql/qgsmssqlfeatureiterator.h
Expand Up @@ -106,7 +106,7 @@ class QgsMssqlFeatureIterator : public QgsAbstractFeatureIteratorFromSource<QgsM
QString mFallbackStatement;

// Field index of FID column
long mFidCol = -1;
int mFidCol = -1;

// List of attribute indices to fetch with nextFeature calls
QgsAttributeList mAttributesToFetch;
Expand Down

0 comments on commit 0729653

Please sign in to comment.