Skip to content

Commit f6ecc8c

Browse files
committedOct 4, 2018
[mssql] Fix debug noise when reading features with null geometry
1 parent af0ddb6 commit f6ecc8c

File tree

1 file changed

+8
-13
lines changed

1 file changed

+8
-13
lines changed
 

‎src/providers/mssql/qgsmssqlfeatureiterator.cpp

Lines changed: 8 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -330,24 +330,19 @@ bool QgsMssqlFeatureIterator::fetchFeature( QgsFeature &feature )
330330

331331
feature.setId( mQuery->record().value( mSource->mFidColName ).toLongLong() );
332332

333+
feature.clearGeometry();
333334
if ( mSource->isSpatial() )
334335
{
335336
QByteArray ar = mQuery->record().value( mSource->mGeometryColName ).toByteArray();
336-
unsigned char *wkb = mParser.ParseSqlGeometry( ( unsigned char * )ar.data(), ar.size() );
337-
if ( wkb )
337+
if ( !ar.isEmpty() )
338338
{
339-
QgsGeometry g;
340-
g.fromWkb( wkb, mParser.GetWkbLen() );
341-
feature.setGeometry( g );
339+
if ( unsigned char *wkb = mParser.ParseSqlGeometry( ( unsigned char * )ar.data(), ar.size() ) )
340+
{
341+
QgsGeometry g;
342+
g.fromWkb( wkb, mParser.GetWkbLen() );
343+
feature.setGeometry( g );
344+
}
342345
}
343-
else
344-
{
345-
feature.clearGeometry();
346-
}
347-
}
348-
else
349-
{
350-
feature.clearGeometry();
351346
}
352347

353348
feature.setValid( true );

0 commit comments

Comments
 (0)
Please sign in to comment.