lynx-dev
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: lynx-dev lynx2.8.4dev.8


From: pg
Subject: Re: lynx-dev lynx2.8.4dev.8
Date: Fri, 25 Aug 2000 06:46:49 -0600 (MDT)

In a recent note, Thomas Dickey said:

> Date: Fri, 25 Aug 2000 05:31:09 -0400
> On Fri, Aug 25, 2000 at 12:11:02AM -0600, address@hidden wrote:
> > Thanks for catching this.  I notice that this suppresses not only the call 
> > to
> > delscreen that I added, but also the call to endwin.  endwin had previously
> > been in the path for ncurses.  Was suppressing it intentional?
> 
> perhaps not - I'll look at it again.  I made that change while I was looking
> for the other piece where I reset the recent_sizechange flag -- for some
> reason it resized ok before your change in the Linux/libc5 configuration,
> and didn't after.  It worked after I made the changes (but there may be
> other/better choices).
>  
I was able to cause considerable misbehavior in Solaris shell escapes by
forcing your #ifdef to go the other way.  Attached is my possible
revision to your revision.

Hastily composed; poorly tested; integrate only if needed.

Hastily indeed.  I just noticed a redundant term in my #ifdef  :-)

-- gil
-- 
StorageTek
INFORMATION made POWERFUL
########################################################################
%%% Created Fri Aug 25 06:36:05 MDT 2000 by target lynx.patch. %%%
diff -bru orig/lynx2-8-4/src/LYCurses.c lynx2-8-4/src/LYCurses.c
--- orig/lynx2-8-4/src/LYCurses.c       Thu Aug 24 19:30:11 2000
+++ lynx2-8-4/src/LYCurses.c    Fri Aug 25 06:25:39 2000
@@ -657,16 +657,18 @@
 #endif
 }
 
-#if       !defined(VMS) && !defined(USE_SLANG)
+#if       !defined(VMS) && !defined(USE_SLANG) && \
+    !defined(NCURSES) && !defined(USE_SLANG)
 /*
  * If newterm is not defined, assume a curses subset which
  * supports only initscr.  --gil
  */
 #ifdef    HAVE_NEWTERM
 static SCREEN *LYscreen = NULL;
-#define LYDELSCR(scr) { \
-    delscreen(scr);     \
-    scr = NULL; }
+#define LYDELSCR() { \
+if (recent_sizechange) { \
+    delscreen(LYscreen); \
+    LYscreen = NULL; } }
 /*
  * Surrogates for newterm annd delscreen
  */
@@ -1051,12 +1053,10 @@
      */
     if(LYCursesON == TRUE)     {
        lynx_enable_mouse (0);
-#if !defined(NCURSES) && !defined(VMS) && !defined(USE_SLANG) && 
(!defined(WIN_EX) || defined(__CYGWIN__))     /* @@@ */
+#if (!defined(WIN_EX) || defined(__CYGWIN__))  /* @@@ */
        if(LYscreen) {
            endwin();   /* stop curses */
-           if (recent_sizechange) {
-               LYDELSCR(LYscreen);
-           }
+           LYDELSCR();
         }
 #endif
     }

; To UNSUBSCRIBE: Send "unsubscribe lynx-dev" to address@hidden

reply via email to

[Prev in Thread] Current Thread [Next in Thread]