diff options
| author | Ulf Hermann <ulf.hermann@qt.io> | 2016-11-08 18:11:36 +0100 |
|---|---|---|
| committer | Ulf Hermann <ulf.hermann@qt.io> | 2016-11-09 11:55:02 +0000 |
| commit | faf77fd0e55bbb7a6ef0b36b82023eed587c3b2b (patch) | |
| tree | 0742c06475a35ebfb294fc5429535fd7e4d97348 /src/plugins/qmlprofiler/pixmapcachemodel.cpp | |
| parent | aaca50f705ac8f41f44991478fe11f9eb37fe0bd (diff) | |
Timeline: Use QRgb and a lookup table for colors
It makes no sense to return a QColor as the only things we are using
are the red, green, and blue components. Furthermore, colorFromHue()
can only generate 360 different colors which we can easily cache
instead of recalculating them on each request.
This significantly reduces the time it takes to update the timeline
render nodes.
Change-Id: I7961014364a1bec5b089285148b2e6c141a6dc7d
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
Diffstat (limited to 'src/plugins/qmlprofiler/pixmapcachemodel.cpp')
| -rw-r--r-- | src/plugins/qmlprofiler/pixmapcachemodel.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/plugins/qmlprofiler/pixmapcachemodel.cpp b/src/plugins/qmlprofiler/pixmapcachemodel.cpp index b38256c9a8..01770c579b 100644 --- a/src/plugins/qmlprofiler/pixmapcachemodel.cpp +++ b/src/plugins/qmlprofiler/pixmapcachemodel.cpp @@ -60,7 +60,7 @@ int PixmapCacheModel::typeId(int index) const return m_data[index].typeId; } -QColor PixmapCacheModel::color(int index) const +QRgb PixmapCacheModel::color(int index) const { if (m_data[index].pixmapEventType == PixmapCacheCountChanged) return colorByHue(s_pixmapCacheCountHue); |