GNU bug report logs - #27427
26.0.50; Native line numbers lead to display error in company-mode popup

Previous Next

Package: emacs;

Reported by: Alexander Miller <alexanderm <at> web.de>

Date: Mon, 19 Jun 2017 16:51:02 UTC

Severity: normal

Found in version 26.0.50

Done: Dmitry Gutov <dgutov <at> yandex.ru>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: martin rudalics <rudalics <at> gmx.at>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: alexanderm <at> web.de, 27427 <at> debbugs.gnu.org, dgutov <at> yandex.ru
Subject: bug#27427: 26.0.50; Native line numbers lead to display error in company-mode popup
Date: Sun, 02 Jul 2017 09:54:12 +0200
> Yes, but I presumed it will do it the same way company-mode reacts to
> such changes today.  Currently. company-mode recomputes and moves its
> overlay; it will instead recompute the text and its position.

To my knowledge ‘company-mode’ and all packages trying to solve similar
display problems are based on idle timers.  The overlay is moved, if
necessry, when the timeout expires.  I'm not sure whether it makes sense
to use a timer-less approach.  If it does, we should allow moving the
overlay right after recalculating the frame's glyph matrix on TTYs.
Whether removing timers makes sense for GUIs is yet another question.

In this context the ‘window-text-change-functions’ I mentioned earlier
seems completely useless.  Here doing for example

(add-hook 'window-text-change-functions 'ignore)

just gets me into a loop and I have to kill Emacs.  Also, it's not clear
to me why this hook is considered abnormal and how to identify from
Elisp the window that is redisplayed at the time the hook is run.  The
fact that the window's buffer is current at that time is hardly useful.
So I think we should either remove that hook or completely redesign it.

martin





This bug report was last modified 7 years and 314 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.