Skip to content

Commit

Permalink
Fix limited random color ramp always returns 1 less color than set
Browse files Browse the repository at this point in the history
(cherry-picked from 1d98b10)
  • Loading branch information
nyalldawson committed Aug 29, 2016
1 parent f23b778 commit e304c4d
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion src/core/symbology-ng/qgsvectorcolorrampv2.cpp
Expand Up @@ -296,8 +296,11 @@ double QgsVectorRandomColorRampV2::value( int index ) const

QColor QgsVectorRandomColorRampV2::color( double value ) const
{
if ( value < 0 || value > 1 )
return QColor();

int colorCnt = mColors.count();
int colorIdx = static_cast< int >( value * ( colorCnt - 1 ) );
int colorIdx = qMin( static_cast< int >( value * colorCnt ), colorCnt - 1 );

if ( colorIdx >= 0 && colorIdx < colorCnt )
return mColors.at( colorIdx );
Expand Down

0 comments on commit e304c4d

Please sign in to comment.