Use "static" for helper functions

These functions act only on the input parameters, so let's make them
static to more clearly indicate that they do not act upon any object.

diff --git a/vncviewer/GestureHandler.h b/vncviewer/GestureHandler.h
index df30ebc2b30bf5993119a7f1f9aa57633947f2c7..2b31703a9e64680e79c837bc8e00317ff897bc6d 100644 (file)
--- a/vncviewer/GestureHandler.h
+++ b/vncviewer/GestureHandler.h
@@ -47,7 +47,7 @@ class GestureHandler : public rfb::Timer::Callback {
     void twoTouchTimeout();
 
     void pushEvent(GestureEventType t);
-    GestureEventGesture stateToGesture(unsigned char state);
+    static GestureEventGesture stateToGesture(unsigned char state);
 
     void getPosition(double *firstX, double *firstY,
                      double *lastX, double *lastY);

diff --git a/vncviewer/MonitorIndicesParameter.h b/vncviewer/MonitorIndicesParameter.h
index 80512c328a69e5b6fad187c923d135c9fe567529..e71439a8264e6400c69cb2f88841b8f48c891126 100644 (file)
--- a/vncviewer/MonitorIndicesParameter.h
+++ b/vncviewer/MonitorIndicesParameter.h
@@ -36,8 +36,8 @@ private:
         int fltkIndex;
     } Monitor;
 
-    bool parseIndices(const char* value, std::set<int> *indices,
-                      bool complain=false);
+    static bool parseIndices(const char* value, std::set<int> *indices,
+                             bool complain=false);
     std::vector<MonitorIndicesParameter::Monitor> fetchMonitors();
     static int compare(const void*, const void*);
 };