fix #4819

vmora · m-kuhn · commit ed1e19f984a2 · 2013-07-18T16:20:00.000+02:00
improves rendering speed of complex rubberbands
diff --git a/src/gui/qgsrubberband.cpp b/src/gui/qgsrubberband.cpp
@@ -412,7 +412,9 @@ void QgsRubberBand::paint( QPainter* p )
       QList<QgsPoint>::const_iterator it = mPoints.at( i ).constBegin();
       for ( ; it != mPoints.at( i ).constEnd(); ++it )
       {
-        pts.append( toCanvasCoordinates( QgsPoint( it->x() + mTranslationOffsetX, it->y() + mTranslationOffsetY ) ) - pos() );
+        const QPointF cur = toCanvasCoordinates( QgsPoint( it->x() + mTranslationOffsetX, it->y() + mTranslationOffsetY ) ) - pos();
+        if ( pts.empty() || std::abs( pts.back().x() - cur.x() ) > 1 ||  std::abs( pts.back().y() - cur.y() ) > 1 )
+          pts.append( cur );
       }
 
       switch ( mGeometryType )

Original file line number	Diff line number	Diff line change
`@@ -412,7 +412,9 @@ void QgsRubberBand::paint( QPainter* p )`
`412`	`412`	`QList<QgsPoint>::const_iterator it = mPoints.at( i ).constBegin();`
`413`	`413`	`for ( ; it != mPoints.at( i ).constEnd(); ++it )`
`414`	`414`	`{`
`415`		`- pts.append( toCanvasCoordinates( QgsPoint( it->x() + mTranslationOffsetX, it->y() + mTranslationOffsetY ) ) - pos() );`
	`415`	`+ const QPointF cur = toCanvasCoordinates( QgsPoint( it->x() + mTranslationOffsetX, it->y() + mTranslationOffsetY ) ) - pos();`
	`416`	`+ if ( pts.empty() \|\| std::abs( pts.back().x() - cur.x() ) > 1 \|\| std::abs( pts.back().y() - cur.y() ) > 1 )`
	`417`	`+ pts.append( cur );`
`416`	`418`	`}`
`417`	`419`
`418`	`420`	`switch ( mGeometryType )`