On Wed, May 28, 2003 at 01:28:20PM -0400, David Dawes wrote:
> On Wed, May 28, 2003 at 12:54:57PM -0400, Thomas Dickey wrote:
> >On Wed, May 28, 2003 at 05:20:15PM +0100, Dr Andrew C Aitchison wrote:
> >> On Wed, 28 May 2003, Dr Andrew C Aitchison wrote:
> >>
> >> > On Wed, 28 May 2003, Thomas Dickey wrote:
> >> >
> >> > > On Wed, May 28, 2003 at 07:16:08AM -0600, Marc Aurele La France wrote:
> >> > > > On Wed, 28 May 2003, Dr Andrew C Aitchison wrote:
> >> > > >
> >> > > > > Patch 3.62 to xc/programs/xterm/screen.c breaks xterm resizing.
> >> > > > > The call to SET_TTYSIZE no longer happens when TRACE isn't enabled.
> >> > > > > Fix is to revert to version 3.61
...
> >> > Old, working code:
> >> > - code = SET_TTYSIZE(screen->respond, ts);
> >> > - TRACE(("return %d from SET_TTYSIZE %dx%d\n", code, rows, cols));
> >> >
> >> > New, broken code:
> >> > + TRACE(("return %d from SET_TTYSIZE %dx%d\n",
> >> > + SET_TTYSIZE(screen->respond, ts), rows, cols));
> Judging from the log messages, the original code (what Andrew quoted
> as "old, working code") was functionally fine, but generated a
> compiler warning, probably because 'code' was assigned a value that
> was never used. The '(void)code;' line should just silence that warning.
what he quoted as "old, working code" is the current code afaik.
(what he's showing as "new, broken code" would be a coding error)
--
Thomas E. Dickey <[EMAIL PROTECTED]>
http://invisible-island.net
ftp://invisible-island.net
_______________________________________________
Devel mailing list
[EMAIL PROTECTED]
http://XFree86.Org/mailman/listinfo/devel
