Simply screen refresh handler

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

1 files changed, 4 insertions, 9 deletions

diff --git a/vncviewer/MonitorArrangement.cxx b/vncviewer/MonitorArrangement.cxx
index 4c8e8db6..93227c59 100644
--- 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;
 }