cleanup of some files

3 files changed, 11 insertions, 11 deletions

diff --git a/src/FbTk/AutoReloadHelper.cc b/src/FbTk/AutoReloadHelper.cc
index e3c1d59..17124d4 100644
--- a/src/FbTk/AutoReloadHelper.cc
+++ b/src/FbTk/AutoReloadHelper.cc
@@ -40,19 +40,19 @@ void AutoReloadHelper::checkReload() {
     }
 }
 
-void AutoReloadHelper::setMainFile(std::string file) {
-    file = StringUtil::expandFilename(file);
-    if (file == m_main_file)
+void AutoReloadHelper::setMainFile(const std::string& file) {
+    std::string expanded_file = StringUtil::expandFilename(file);
+    if (expanded_file == m_main_file)
         return;
-    m_main_file = file;
+    m_main_file = expanded_file;
     reload();
 }
 
-void AutoReloadHelper::addFile(std::string file) {
+void AutoReloadHelper::addFile(const std::string& file) {
     if (file.empty())
         return;
-    file = StringUtil::expandFilename(file);
-    m_timestamps[file] = FileUtil::getLastStatusChangeTimestamp(file.c_str());
+    std::string expanded_file = StringUtil::expandFilename(file);
+    m_timestamps[expanded_file] = FileUtil::getLastStatusChangeTimestamp(expanded_file.c_str());
 }
 
 void AutoReloadHelper::reload() {
diff --git a/src/FbTk/AutoReloadHelper.hh b/src/FbTk/AutoReloadHelper.hh
index 36be8f1..b337a4a 100644
--- a/src/FbTk/AutoReloadHelper.hh
+++ b/src/FbTk/AutoReloadHelper.hh
@@ -34,8 +34,8 @@ namespace FbTk {
 class AutoReloadHelper {
 public:
 
-    void setMainFile(std::string filename);
-    void addFile(std::string filename);
+    void setMainFile(const std::string& filename);
+    void addFile(const std::string& filename);
     void setReloadCmd(RefCount<Command<void> > cmd) { m_reload_cmd = cmd; }
 
     void checkReload();
diff --git a/src/FbTk/ImageControl.cc b/src/FbTk/ImageControl.cc
index b9c3685..38114c8 100644
--- a/src/FbTk/ImageControl.cc
+++ b/src/FbTk/ImageControl.cc
@@ -495,7 +495,7 @@ void ImageControl::createColorTable() {
                 for (int b = 0; b < m_colors_per_channel; b++, i++) {
                     m_colors[i].red = (r * 0xffff) / (m_colors_per_channel - 1);
                     m_colors[i].green = (g * 0xffff) / (m_colors_per_channel - 1);
-                    m_colors[i].blue = (b * 0xffff) / (m_colors_per_channel - 1);;
+                    m_colors[i].blue = (b * 0xffff) / (m_colors_per_channel - 1);
                     m_colors[i].flags = DoRed|DoGreen|DoBlue;
                 }
             }
@@ -587,7 +587,7 @@ void ImageControl::createColorTable() {
             for (unsigned int i = 0; i < m_num_colors; i++) {
                 m_colors[i].red = (i * 0xffff) / (m_colors_per_channel - 1);
                 m_colors[i].green = (i * 0xffff) / (m_colors_per_channel - 1);
-                m_colors[i].blue = (i * 0xffff) / (m_colors_per_channel - 1);;
+                m_colors[i].blue = (i * 0xffff) / (m_colors_per_channel - 1);
                 m_colors[i].flags = DoRed|DoGreen|DoBlue;
 
                 if (! XAllocColor(disp, m_colormap,