Simply screen refresh handler

We only need to check the event type once, and preferably early, and
avoid doing more work than necessary.

diff --git a/vncviewer/MonitorArrangement.cxx b/vncviewer/MonitorArrangement.cxx
index 4c8e8db6d5be6a795c31a0c077ec4ca07c845770..93227c5945a16c707bd76524a52cea5a1090774c 100644 (file)
--- a/vncviewer/MonitorArrangement.cxx
+++ b/vncviewer/MonitorArrangement.cxx
@@ -448,18 +448,13 @@ int MonitorArrangement::get_monitor_name(int m, char name[], size_t name_len)
 
 int MonitorArrangement::fltk_event_handler(int event)
 {
-  MonitorArrangement *self;
   std::set<MonitorArrangement *>::iterator it;
 
-  for (it = instances.begin(); it != instances.end(); it++) {
-    self = *it;
+  if (event != FL_SCREEN_CONFIGURATION_CHANGED)
+    return 0;
 
-    switch (event) {
-    case FL_SCREEN_CONFIGURATION_CHANGED:
-      self->refresh();
-      break;
-    }
-  }
+  for (it = instances.begin(); it != instances.end(); it++)
+    (*it)->refresh();
 
   return 0;
 }