Fix random color ramps returning invalid QColors for value of 1.0

nyalldawson · nyalldawson · commit 1874c371ae2d · 2015-08-04T21:22:23.000+10:00
(cherry-picked from 1d2b4cb)
diff --git a/src/core/symbology-ng/qgsvectorcolorrampv2.cpp b/src/core/symbology-ng/qgsvectorcolorrampv2.cpp
@@ -295,7 +295,7 @@ double QgsVectorRandomColorRampV2::value( int index ) const
 QColor QgsVectorRandomColorRampV2::color( double value ) const
 {
   int colorCnt = mColors.count();
-  int colorIdx = ( int )( value * colorCnt );
+  int colorIdx = ( int )( value * ( colorCnt - 1 ) );
 
   if ( colorIdx >= 0 && colorIdx < colorCnt )
     return mColors.at( colorIdx );

Original file line number	Diff line number	Diff line change
`@@ -295,7 +295,7 @@ double QgsVectorRandomColorRampV2::value( int index ) const`
`295`	`295`	`QColor QgsVectorRandomColorRampV2::color( double value ) const`
`296`	`296`	`{`
`297`	`297`	`int colorCnt = mColors.count();`
`298`		`- int colorIdx = ( int )( value * colorCnt );`
	`298`	`+ int colorIdx = ( int )( value * ( colorCnt - 1 ) );`
`299`	`299`
`300`	`300`	`if ( colorIdx >= 0 && colorIdx < colorCnt )`
`301`	`301`	`return mColors.at( colorIdx );`