The function was originally designed to return the color that
correspond to a given PID. However we now have 3 different color
maps (for PID, CPU Id and Stream Id) and the same method can be
used with each of the 3 maps. We also want to prevent the method
from modifying the map itself.
Signed-off-by: Yordan Karadzhov (VMware) <y.karadz@xxxxxxxxx>
---
src/KsPlotTools.cpp | 10 +++++-----
src/KsPlotTools.hpp | 2 +-
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/src/KsPlotTools.cpp b/src/KsPlotTools.cpp
index 7a0450b..52c3cbb 100644
--- a/src/KsPlotTools.cpp
+++ b/src/KsPlotTools.cpp
@@ -216,17 +216,17 @@ ColorTable getStreamColorTable()
}
/**
- * @brief Search the Hash table of Rainbow colors for a particular key (pid).
+ * @brief Search the Hash table of Rainbow colors for a particular key (Id).
*
* @param colors: Input location for the ColorTable instance.
- * @param pid: the Process Id to search for.
+ * @param id: The Id to search for.
*
- * @returns The Rainbow color of the key "pid". If "pid" does not exist, the
+ * @returns The Rainbow color of the key "id". If "id" does not exist, the
* returned color is Black.
*/
-Color getColor(ColorTable *colors, int pid)
+Color getColor(const ColorTable *colors, int id)
{
- auto item = colors->find(pid);
+ auto item = colors->find(id);
if (item != colors->end())
return item->second;
diff --git a/src/KsPlotTools.hpp b/src/KsPlotTools.hpp
index ab28f43..dbb15bf 100644
--- a/src/KsPlotTools.hpp
+++ b/src/KsPlotTools.hpp
@@ -80,7 +80,7 @@ ColorTable getCPUColorTable();
ColorTable getStreamColorTable();
-Color getColor(ColorTable *colors, int pid);
+Color getColor(const ColorTable *colors, int id);
/** Represents an abstract graphical element. */
class PlotObject {
--
2.25.1
|