On Wed, May 28, 2003 at 05:12:45PM +0100, 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
> > > 
> > > OK.  I'll fix it.
> > 
> > presumably not by simply rolling back the change...
> 
> Why not ?
> 
> 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));
> 
> in the broken version since TRACE(...) is a null macro, cpp removes 
> the call to SET_TTYSIZE.

then I'm not sure what part of the source you're looking at, since none
of it encloses the SET_TTYSIZE within a TRACE macro.  I quoted the lines
you cite as "Old" from the code I last modified in March.

Is there a branch you're looking at?

-- 
Thomas E. Dickey <[EMAIL PROTECTED]>
http://invisible-island.net
ftp://invisible-island.net
_______________________________________________
Devel mailing list
[EMAIL PROTECTED]
http://XFree86.Org/mailman/listinfo/devel

Reply via email to