[Aptitude-svn-commit] r4171 - in branches/aptitude-0.3/aptitude: . src/vscreen

[Daniel Burrows]

Author: dburrows
Date: Thu Sep 22 16:42:49 2005
New Revision: 4171

Modified:
   branches/aptitude-0.3/aptitude/ChangeLog
   branches/aptitude-0.3/aptitude/src/vscreen/vscreen.cc
Log:
Fix cursor placement

Modified: branches/aptitude-0.3/aptitude/ChangeLog
==============================================================================
--- branches/aptitude-0.3/aptitude/ChangeLog	(original)
+++ branches/aptitude-0.3/aptitude/ChangeLog	Thu Sep 22 16:42:49 2005
@@ -1,5 +1,9 @@
 2005-09-22  Daniel Burrows  <dburrows at debian.org>
 
+	* src/vscreen/vscreen.cc:
+
+	  (hopefully) fix the cursor placement problems that I was having.
+
 	* src/vscreen/curses++.h:
 
 	  Fix mvaddnstr for std::wstrings.

Modified: branches/aptitude-0.3/aptitude/src/vscreen/vscreen.cc
==============================================================================
--- branches/aptitude-0.3/aptitude/src/vscreen/vscreen.cc	(original)
+++ branches/aptitude-0.3/aptitude/src/vscreen/vscreen.cc	Thu Sep 22 16:42:49 2005
@@ -825,8 +825,9 @@
   if(toplevel->get_cursorvisible())
     {
       point p=toplevel->get_cursorloc();
-      toplevel->win.move(p.y, p.x);
       toplevel->win.leaveok(false);
+      toplevel->win.move(p.y, p.x);
+      toplevel->win.noutrefresh();
     }
   else
     toplevel->win.leaveok(true);
@@ -848,10 +849,6 @@
     {
       toplevel->display(get_style("Default"));
       toplevel->sync();
-
-      vscreen_updatecursornow();
-
-      doupdate();
     }
 }
 
@@ -883,11 +880,10 @@
   if(needs.update)
     vscreen_updatenow();
 
-  if(needs.cursorupdate)
-    {
-      vscreen_updatecursornow();
-      refresh();
-    }
+  if(needs.update || needs.cursorupdate)
+    vscreen_updatecursornow();
+
+  doupdate();
 
   // \todo This appears to just paper over sloppiness -- screen update
   // routines shouldn't be queuing more updates!