Re: ncurses-5.4-20040731.patch.gz

[Folkert van Heusden]

> > Are you sure that:
> > +#ifdef TRACE
> > +       if (_nc_tracing & TRACE_UPDATE) {
> > +           LINES = ToLines - was_stolen;
> > +           COLS = ToCols;
> > +           show_window_sizes("after");
> > +       }
> > +#endif
> > is correct? It is now only adjusting LINES and COLS when tracing is enabled.
> > Shouldn't it always adjust these?
> look down a few lines - the same assignments appear there.  I copied them
> into the TRACE ifdef so I could get the right numbers in the first line
> written by show_window_sizes().  That made the trace file easier to
> understand.
> I know it looks odd, but didn't want to add a flag variable so I could
> move that chunk of code below the place where I copied the code from.

Ah, ok. I only looked at the patch itself.
Thanks.
I've also mailed the debian ncurses maintainer to urge him to update the
package.


Folkert van Heusden

+------------------------------------------------------------------+
|UNIX admin? Then give MultiTail (http://vanheusden.com/multitail/)|
|a try, it brings monitoring logfiles to a different level! See    |
|http://vanheusden.com/multitail/features.html for a feature list. |
+------------------------------------------= www.unixsoftware.nl =-+