[prev in list] [next in list] [prev in thread] [next in thread] 

List:       konsole-devel
Subject:    Bug#35294: [Konsole-devel] Bug#35294: Fix for joe. (backport)
From:       Waldo Bastian <bastian () kde ! org>
Date:       2001-11-27 1:16:32
[Download RAW message or body]

On Monday 26 November 2001 02:01 pm, Ivan E. Moore II wrote:
> The "Fix for joe. (backport)" commit is what causes this particular
> bug.  If I remove this patch it works properly.
>
> (In reference to Bug #35294)
>
> Ivan

Does this patch fixes it?

Cheers,
Waldo
["konsole.diff" (text/x-diff)]

Index: TEmuVt102.C
===================================================================
RCS file: /home/kde/kdebase/konsole/konsole/TEmuVt102.cpp,v
retrieving revision 1.67
retrieving revision 1.68
diff -u -r1.67 -r1.68
--- TEmuVt102.C	2001/10/25 05:26:33	1.67
+++ TEmuVt102.C	2001/11/01 02:50:59	1.68
@@ -573,6 +573,8 @@
 
     case TY_CSI_PR('h',   47) :          setMode      (MODE_AppScreen); break; \
                //VT100
     case TY_CSI_PR('l',   47) :        resetMode      (MODE_AppScreen); break; \
//VT100 +    case TY_CSI_PR('s',   47) :         saveMode      (MODE_AppScreen); \
break; //XTERM +    case TY_CSI_PR('r',   47) :      restoreMode      \
(MODE_AppScreen); break; //XTERM  
     // XTerm defines the following modes:
     // SET_VT200_MOUSE             1000
@@ -604,16 +606,20 @@
     case TY_CSI_PR('r', 1003) :      restoreMode      (MODE_Mouse1000); break; \
//XTERM  
     case TY_CSI_PR('h', 1047) :          setMode      (MODE_AppScreen); break; \
                //XTERM
-    case TY_CSI_PR('l', 1047) :        resetMode      (MODE_AppScreen); break; \
//XTERM +    case TY_CSI_PR('l', 1047) : screen[1]->clearEntireScreen(); \
resetMode(MODE_AppScreen); break; //XTERM +    case TY_CSI_PR('s', 1047) :         \
saveMode      (MODE_AppScreen); break; //XTERM +    case TY_CSI_PR('r', 1047) :      \
restoreMode      (MODE_AppScreen); break; //XTERM  
     //FIXME: Unitoken: save translations
     case TY_CSI_PR('h', 1048) :      saveCursor           (          ); break; \
                //XTERM
     case TY_CSI_PR('l', 1048) :      restoreCursor        (          ); break; \
//XTERM +    case TY_CSI_PR('s', 1048) :      saveCursor           (          ); \
break; //XTERM +    case TY_CSI_PR('r', 1048) :      restoreCursor        (          \
); break; //XTERM  
     //FIXME: every once new sequences like this pop up in xterm.
     //       Here's a guess of what they could mean.
-    case TY_CSI_PR('h', 1049) :          setMode      (MODE_AppScreen); break; \
                //XTERM
-    case TY_CSI_PR('l', 1049) :        resetMode      (MODE_AppScreen); break; \
//XTERM +    case TY_CSI_PR('h', 1049) : saveCursor(); \
screen[1]->clearEntireScreen(); setMode(MODE_AppScreen); break; //XTERM +    case \
TY_CSI_PR('l', 1049) : resetMode(MODE_AppScreen); restoreCursor(); break; //XTERM  
     //FIXME: when changing between vt52 and ansi mode evtl do some resetting.
     case TY_VT52__('A'      ) : scr->cursorUp             (         1); break; \
//VT52 @@ -1001,10 +1013,10 @@
   {
     case MODE_Mouse1000 : gui->setMouseMarks(FALSE);
     break;
+
     case MODE_AppScreen : screen[1]->clearSelection();
-                          screen[1]->clearEntireScreen();
                           setScreen(1);
-	  break;
+    break;
   }
   if (m < MODES_SCREEN || m == MODE_NewLine)
   {


_______________________________________________
konsole-devel mailing list
konsole-devel@mail.kde.org
http://mail.kde.org/mailman/listinfo/konsole-devel


[prev in list] [next in list] [prev in thread] [next in thread]