* display bug with compctl -X
@ 1996-10-18 14:12 Peter Stephenson
1996-10-18 16:45 ` Geoff Wing
0 siblings, 1 reply; 2+ messages in thread
From: Peter Stephenson @ 1996-10-18 14:12 UTC (permalink / raw)
To: Zsh hackers list
The following is guaranteed repeatable on my system with 3.0.1-test3.
The terminal is an xterm.
% zsh -f
% compctl -D -X 'Default completion' -f
% setopt alwayslastprompt
% ls xxx^D # there's no such file
Default completion xx
# ^ cursor is here, one line too low.
By the way, should alwayslastprompt be on by default? There was some
discussion about this years ago when Sven wrote it and I was rather
surprised to find it wasn't when I tried the above. It's not clear to
me why it shouldn't be.
--
Peter Stephenson <pws@ifh.de> Tel: +49 33762 77366
WWW: http://www.ifh.de/~pws/ Fax: +49 33762 77413
Deutches Electronen-Synchrotron --- Institut fuer Hochenergiephysik Zeuthen
DESY-IfH, 15735 Zeuthen, Germany.
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: display bug with compctl -X
1996-10-18 14:12 display bug with compctl -X Peter Stephenson
@ 1996-10-18 16:45 ` Geoff Wing
0 siblings, 0 replies; 2+ messages in thread
From: Geoff Wing @ 1996-10-18 16:45 UTC (permalink / raw)
To: Peter Stephenson; +Cc: zsh-workers
Peter Stephenson wrote:
:The following is guaranteed repeatable on my system with 3.0.1-test3.
:The terminal is an xterm.
:% zsh -f
:% compctl -D -X 'Default completion' -f
:% setopt alwayslastprompt
:% ls xxx^D # there's no such file
:Default completion xx
:# ^ cursor is here, one line too low.
:
:By the way, should alwayslastprompt be on by default? There was some
:discussion about this years ago when Sven wrote it and I was rather
:surprised to find it wasn't when I tried the above. It's not clear to
:me why it shouldn't be.
Well obviously the baseline shouldn't have it by default because I don't :-)
listmatches() is going through with no possible matches and is calculating
the number of lines to go up incorrectly.
This whole section of code could use some more comments in it.
Someone who knows this bit of code might want to check that this is the
best solution.
I've just looked at this patch, and the previous bit of code may or may not
also need a similar check - your terminal would need to be set to 3 or 4
columns (depending on other factors).
Setting at under, say, ten columns still causes serious problems / core-dumps
in several bits of the code.
*** zle_tricky.c.orig Sat Oct 19 01:32:41 1996
--- zle_tricky.c Sat Oct 19 02:14:59 1996
***************
*** 3558,3564 ****
(!(haswhat & HAS_MISC) ? nfpl + nfsl : nlpl + nlsl))) / columns;
} else {
colsz = (ct + fct - 1) / fct;
! up = colsz + nlnct - clearflag;
}
/* Print the explanation string, if any. */
--- 3558,3564 ----
(!(haswhat & HAS_MISC) ? nfpl + nfsl : nlpl + nlsl))) / columns;
} else {
colsz = (ct + fct - 1) / fct;
! up = colsz + nlnct - clearflag + (ct == 0);
}
/* Print the explanation string, if any. */
--
Geoff Wing [gwing@primenet.com.au] PrimeNet - Internet Consultancy
Web: http://www.primenet.com.au/ Facsimile: +61-3-9819 3788
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~1996-10-18 16:53 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
1996-10-18 14:12 display bug with compctl -X Peter Stephenson
1996-10-18 16:45 ` Geoff Wing
Code repositories for project(s) associated with this public inbox
https://git.vuxu.org/mirror/zsh/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).