Re: lib/51819: curses vs. news/tin display problems

[Brett Lymn <blymn%internode.on.net@localhost>]

The following reply was made to PR lib/51819; it has been noted by GNATS.

From: Brett Lymn <blymn%internode.on.net@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: lib-bug-people%netbsd.org@localhost, gnats-admin%netbsd.org@localhost, netbsd-bugs%netbsd.org@localhost,
        uwe%NetBSD.org@localhost
Subject: Re: lib/51819: curses vs. news/tin display problems
Date: Thu, 12 Jan 2017 07:51:22 +1030

 --vtzGhvizbBRQ85DL
 Content-Type: text/plain; charset=us-ascii
 Content-Disposition: inline
 
 On Wed, Jan 11, 2017 at 03:30:01PM +0000, Valery Ushakov wrote:
 >  
 >    The interaction of these functions with setscrreg() is currently
 >    unspecified.
 >  
 
 Try the attached patch, I have removed the check where the cursor is and
 just scroll the requested window.
 
 -- 
 Brett Lymn
 Let go, or be dragged - Zen proverb.
 
 --vtzGhvizbBRQ85DL
 Content-Type: text/plain; charset=us-ascii
 Content-Disposition: attachment; filename="scroll.diff"
 
 Index: scroll.c
 ===================================================================
 RCS file: /cvsroot/src/lib/libcurses/scroll.c,v
 retrieving revision 1.23
 diff -u -r1.23 scroll.c
 --- scroll.c	6 Jan 2017 13:53:18 -0000	1.23
 +++ scroll.c	11 Jan 2017 21:20:03 -0000
 @@ -100,12 +100,7 @@
  #ifdef DEBUG
  	__CTRACE(__CTRACE_WINDOW, "wscrl: y=%d\n", oy);
  #endif
 -	if (oy < win->scr_t || oy > win->scr_b)
 -		/* Outside scrolling region */
 -		wmove(win, 0, 0);
 -	else
 -		/* Inside scrolling region */
 -		wmove(win, win->scr_t, 0);
 +	wmove(win, win->scr_t, 0);
  	winsdelln(win, 0 - nlines);
  	wmove(win, oy, ox);
  
 
 --vtzGhvizbBRQ85DL--