[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
- lynx-dev lynx2.8.4dev.8, Thomas Dickey, 2000/08/24
- Re: lynx-dev lynx2.8.4dev.8, pg, 2000/08/25
- Re: lynx-dev lynx2.8.4dev.8, Thomas Dickey, 2000/08/25
- Re: lynx-dev lynx2.8.4dev.8,
pg <=
- Re: lynx-dev lynx2.8.4dev.8, Thomas E. Dickey, 2000/08/25
- Re: lynx-dev lynx2.8.4dev.8, pg, 2000/08/25
- Re: lynx-dev lynx2.8.4dev.8, Thomas E. Dickey, 2000/08/25
- Re: lynx-dev lynx2.8.4dev.8, pg, 2000/08/25
- Re: lynx-dev lynx2.8.4dev.8, Thomas E. Dickey, 2000/08/25
- Re: lynx-dev lynx2.8.4dev.8, pg, 2000/08/25
- Re: lynx-dev lynx2.8.4dev.8, Thomas E. Dickey, 2000/08/25