Move selection labels to renderer from render items

Creating a new label on demand is fast enough, so we don't need to
store old labels. Storing labels unnecessarily leads to unchecked
use of graphics memory.

Optimized bar slices a bit by having separate class of render items
for slice items.

Change-Id: I5afdecfb3949393341b6c3eaefbc667c65da32e3
Reviewed-by: Tomi Korpipää <tomi.korpipaa@digia.com>

1 files changed, 1 insertions, 5 deletions

diff --git a/src/datavisualization/data/scatterrenderitem_p.h b/src/datavisualization/data/scatterrenderitem_p.h
index 8b560c89..eb070682 100644
--- a/src/datavisualization/data/scatterrenderitem_p.h
+++ b/src/datavisualization/data/scatterrenderitem_p.h
@@ -45,12 +45,8 @@ public:
     inline const QVector3D &position() const { return m_position; }
     inline void setPosition(const QVector3D &pos)
     {
-        if (m_position != pos) {
+        if (m_position != pos)
             m_position = pos;
-            // Force reformatting on next access by setting label string to null string
-            if (!m_selectionLabel.isNull())
-                setSelectionLabel(QString());
-        }
     }
 
     inline bool isVisible() const { return m_visible; }