Skip to content
Permalink
Browse files

Remove cryptic assert (was failing regularly)

  • Loading branch information
nyalldawson committed Apr 21, 2016
1 parent eacca44 commit b1b439f5cc4f709db9839d0ab8209346defe4bb7
Showing with 0 additions and 2 deletions.
  1. +0 −2 src/core/qgsrulebasedlabeling.cpp
@@ -266,8 +266,6 @@ QgsRuleBasedLabeling::Rule::RegisterResult QgsRuleBasedLabeling::Rule::registerF

bool registered = false;

Q_ASSERT( !mSettings == subProviders.contains( this ) );

// do we have active subprovider for the rule?
if ( subProviders.contains( this ) && mIsActive )
{

2 comments on commit b1b439f

@nyalldawson

This comment has been minimized.

Copy link
Collaborator Author

@nyalldawson nyalldawson replied Apr 21, 2016

@jef-n Can you explain what the intention with this assert was? (we only allow either mSettings or having the subProvider containing this but not both?) It consistently trips up the labeling unit tests.

@lcoandrade

This comment has been minimized.

Copy link

@lcoandrade lcoandrade replied Aug 10, 2016

When will this be available for QGIS 2.14 LTR? Thanks for fixing this!!!

Please sign in to comment.
You can’t perform that action at this time.