Date: Sat, 4 Aug 2007 02:18:27 +0200 From: Pieter de Goeje <pieter@degoeje.nl> To: "Sean C. Farley" <scf@freebsd.org> Cc: Xin LI <delphij@delphij.net>, freebsd-stable@freebsd.org, Ted Hatfield <ted@pat.io.com> Subject: Re: Bug in less version 406. Message-ID: <200708040218.28169.pieter@degoeje.nl> In-Reply-To: <20070803184340.L84415@thor.farley.org> References: <20070802114827.U26646@pat.io.com> <200708040129.58761.pieter@degoeje.nl> <20070803184340.L84415@thor.farley.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Saturday 04 August 2007, Sean C. Farley wrote: > On Sat, 4 Aug 2007, Pieter de Goeje wrote: > > *snip* > > > There's also another regression in less: it doesn't automatically > > repaint the screen anymore when you resize the terminal. > > I have already reported that regression to Mark Nudelman. He is looking > into an appropriate fix since this regression was introduced when fixing > another bug. > > Sean Hmm I wonder what that other bug might have been... If I look at signal.c I see two signal handlers for things related to window changes. One for SIGWINCH and one for SIGWIND. The "if(reading) intread();" statement was removed from the SIGWINCH handler. If removing that statement fixed the other bug why wasn't it removed from SIGWIND's handler? Does SIGWIND have different semantics? Anyway, re-adding "if(reading) intread();" to signal.c:96 makes it work again, but I wonder what I broke by doing that. Regards, Pieter de Goeje
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200708040218.28169.pieter>