Skip to content

Commit

Permalink
QgsSymbolV2: only pop expression scope if it was really added
Browse files Browse the repository at this point in the history
  • Loading branch information
m-kuhn committed Apr 13, 2016
1 parent cbe919d commit ef77e19
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/core/symbology-ng/qgssymbolv2.cpp
Expand Up @@ -978,7 +978,8 @@ void QgsSymbolV2::renderFeature( const QgsFeature& feature, QgsRenderContext& co
delete segmentizedGeometry;
}

context.expressionContext().popScope();
if ( mSymbolRenderContext->expressionContextScope() )
context.expressionContext().popScope();
}

QgsSymbolV2RenderContext* QgsSymbolV2::symbolRenderContext()
Expand Down

1 comment on commit ef77e19

@m-kuhn
Copy link
Member Author

@m-kuhn m-kuhn commented on ef77e19 Apr 13, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since this should only be called after startRender() there should always be a scope I think. But better safe than sorry.

Please sign in to comment.