Code cleanups. The most important change is that we now always use UpdateTracker's getUpdateInfo() and never call get_changed(), get_copied() and get_delta() member functions.

git-svn-id: svn://svn.code.sf.net/p/tigervnc/code/trunk@2325 3789f03b-4d11-0410-bbf8-ca57d06f2519
diff --git a/common/rfb/VNCServerST.cxx b/common/rfb/VNCServerST.cxx
index cc18faa..c02c2eb 100644
--- a/common/rfb/VNCServerST.cxx
+++ b/common/rfb/VNCServerST.cxx
@@ -451,12 +451,15 @@
 
 void VNCServerST::checkUpdate()
 {
+  UpdateInfo ui;
+  comparer->getUpdateInfo(&ui, pb->getRect());
+
   bool renderCursor = needRenderedCursor();
 
-  if (comparer->is_empty() && !(renderCursor && renderedCursorInvalid))
+  if (ui.is_empty() && !(renderCursor && renderedCursorInvalid))
     return;
 
-  Region toCheck = comparer->get_changed().union_(comparer->get_copied());
+  Region toCheck = ui.changed.union_(ui.copied);
 
   if (renderCursor) {
     Rect clippedCursorRect
@@ -490,8 +493,8 @@
   std::list<VNCSConnectionST*>::iterator ci, ci_next;
   for (ci = clients.begin(); ci != clients.end(); ci = ci_next) {
     ci_next = ci; ci_next++;
-    (*ci)->add_copied(comparer->get_copied(), comparer->get_delta());
-    (*ci)->add_changed(comparer->get_changed());
+    (*ci)->add_copied(ui.copied, ui.copy_delta);
+    (*ci)->add_changed(ui.changed);
   }
 
   comparer->clear();