GNU bug report logs - #20072
Incorrect popup showing

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 20072 in the body.
You can then email your comments to 20072 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Nikolay Puzanov <punzik <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: Incorrect popup showing
Date: Tue, 10 Mar 2015 10:48:38 +0000

[Message part 1 (text/plain, inline)]

Hi,

I found bug on display a popup window from auto-complete-mode. I attach two
screenshot and init.el. Bug appear when global-linum-mode is on.

Commit e589765 bring this bug. I changed the line that seemed most
appropriate. I not understand how it work, but this patch fix it:

--- a/src/xdisp.c
+++ b/src/xdisp.c
@@ -3434,6 +3434,7 @@ handle_stop (struct it *it)
   it->dpvec = NULL;
   it->current.dpvec_index = -1;
   handle_overlay_change_p = !it->ignore_overlay_strings_at_pos_p;
+  it->ignore_overlay_strings_at_pos_p = false;
   it->ellipsis_p = 0;

   /* Use face of preceding text for ellipsis (if invisible) */

Note, this patch do not break fix by e589765 (
http://lists.gnu.org/archive/html/bug-gnu-emacs/2015-02/msg00075.html).

Screenshots also uploaded to the external server:
http://www.embddr.com/emacs-popup-good.png
http://www.embddr.com/emacs-popup-bad.png

Init.el:
(require 'package)
(add-to-list 'package-archives '("melpa" . "
http://melpa.milkbox.net/packages/"))
(package-initialize)

(require 'auto-complete-config)
(auto-complete-mode t)
(ac-config-default)

(global-linum-mode t)


Nikolay Puzanov.

[Message part 2 (text/html, inline)]

[emacs-popup-bad.png (image/png, attachment)]

[emacs-popup-good.png (image/png, attachment)]

[init.el (text/x-emacs-lisp, attachment)]

Message #8 received at 20072 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Nikolay Puzanov <punzik <at> gmail.com>
Cc: 20072 <at> debbugs.gnu.org
Subject: Re: bug#20072: Incorrect popup showing
Date: Tue, 10 Mar 2015 19:03:41 +0200

> From: Nikolay Puzanov <punzik <at> gmail.com>
> Date: Tue, 10 Mar 2015 10:48:38 +0000
> 
> I found bug on display a popup window from auto-complete-mode. I attach two
> screenshot and init.el. Bug appear when global-linum-mode is on.

Damn that linum-mode!

> Commit e589765 bring this bug. I changed the line that seemed most appropriate.
> I not understand how it work, but this patch fix it:
> 
> --- a/src/xdisp.c
> +++ b/src/xdisp.c
> @@ -3434,6 +3434,7 @@ handle_stop (struct it *it)
>    it->dpvec = NULL;
>    it->current.dpvec_index = -1;
>    handle_overlay_change_p = !it->ignore_overlay_strings_at_pos_p;
> +  it->ignore_overlay_strings_at_pos_p = false;
>    it->ellipsis_p = 0;
>    
>    /* Use face of preceding text for ellipsis (if invisible) */

Thanks, but I don't think this is the right fix.  It is fundamentally
wrong to unconditionally reset this flag at that spot, because there's
no way of knowing there that we have processed all the overlay
strings.

> Note, this patch do not break fix by e589765
> (http://lists.gnu.org/archive/html/bug-gnu-emacs/2015-02/msg00075.html).

The problem fixed by e589765 is much more fundamental than just what
bug 19307 reported.  When I worked on the fix, I tested a lot more
situations than originally reported, and some of them will be broken
by your suggested change.

So we need to look for another solution.

> Screenshots also uploaded to the external server:
> http://www.embddr.com/emacs-popup-good.png
> http://www.embddr.com/emacs-popup-bad.png

Thanks, I will look into this when I have time.

> Init.el:
> (require 'package)
> (add-to-list 'package-archives '("melpa" .
> "http://melpa.milkbox.net/packages/"))
> (package-initialize)
> 
> (require 'auto-complete-config)
> (auto-complete-mode t)
> (ac-config-default)
> 
> (global-linum-mode t)

Is it enough to do just this instead:

  (require 'auto-complete-config)
  (auto-complete-mode 1)
  (ac-config-default)
  (global-linum-mode 1)

IOW, can I skip the steps that set up package.el, and instead download
auto-complete manually?

Message #11 received at 20072 <at> debbugs.gnu.org (full text, mbox):

From: Nikolay Puzanov <punzik <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 20072 <at> debbugs.gnu.org
Subject: Re: bug#20072: Incorrect popup showing
Date: Tue, 10 Mar 2015 17:30:31 +0000

[Message part 1 (text/plain, inline)]

>
> So we need to look for another solution.
>
>
I would be happy to help you, but I think my knowledge is not enough.


> > Init.el:
> > (require 'package)
> > (add-to-list 'package-archives '("melpa" .
> > "http://melpa.milkbox.net/packages/"))
> > (package-initialize)
> >
> > (require 'auto-complete-config)
> > (auto-complete-mode t)
> > (ac-config-default)
> >
> > (global-linum-mode t)
>
> Is it enough to do just this instead:
>
>   (require 'auto-complete-config)
>   (auto-complete-mode 1)
>   (ac-config-default)
>   (global-linum-mode 1)
>
> IOW, can I skip the steps that set up package.el, and instead download
> auto-complete manually?
>

Yes, you can. That I did:

$ mkdir -p ~/.emacs.d/packages
$ cd ~/.emacs.d/packages
$ git clone https://github.com/auto-complete/auto-complete.git
$ git clone https://github.com/auto-complete/popup-el.git

And init.el:

(add-to-list 'load-path "~/.emacs.d/packages/auto-complete")
(add-to-list 'load-path "~/.emacs.d/packages/popup-el")
(require 'auto-complete-config)
(auto-complete-mode t)
(ac-config-default)
(global-linum-mode t)

[Message part 2 (text/html, inline)]

Message #14 received at 20072 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Nikolay Puzanov <punzik <at> gmail.com>
Cc: 20072 <at> debbugs.gnu.org
Subject: Re: bug#20072: Incorrect popup showing
Date: Fri, 13 Mar 2015 17:50:36 +0200

> From: Nikolay Puzanov <punzik <at> gmail.com>
> Date: Tue, 10 Mar 2015 17:30:31 +0000
> Cc: 20072 <at> debbugs.gnu.org
> 
> $ mkdir -p ~/.emacs.d/packages
> $ cd ~/.emacs.d/packages
> $ git clone https://github.com/auto-complete/auto-complete.git
> $ git clone https://github.com/auto-complete/popup-el.git
> 
> And init.el:
> 
> (add-to-list 'load-path "~/.emacs.d/packages/auto-complete")
> (add-to-list 'load-path "~/.emacs.d/packages/popup-el")
> (require 'auto-complete-config)
> (auto-complete-mode t)
> (ac-config-default)
> (global-linum-mode t)

And what should I do after this to reproduce the broken display you
posted?

Message #17 received at 20072 <at> debbugs.gnu.org (full text, mbox):

From: Nikolay Puzanov <punzik <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 20072 <at> debbugs.gnu.org
Subject: Re: bug#20072: Incorrect popup showing
Date: Fri, 13 Mar 2015 17:17:39 +0000

[Message part 1 (text/plain, inline)]

Open init.el, put cursor to the beginning of buffer, type "(lo", wait for
popup.

пт, 13 марта 2015 г. в 18:50, Eli Zaretskii <eliz <at> gnu.org>:

> > From: Nikolay Puzanov <punzik <at> gmail.com>
> > Date: Tue, 10 Mar 2015 17:30:31 +0000
> > Cc: 20072 <at> debbugs.gnu.org
> >
> > $ mkdir -p ~/.emacs.d/packages
> > $ cd ~/.emacs.d/packages
> > $ git clone https://github.com/auto-complete/auto-complete.git
> > $ git clone https://github.com/auto-complete/popup-el.git
> >
> > And init.el:
> >
> > (add-to-list 'load-path "~/.emacs.d/packages/auto-complete")
> > (add-to-list 'load-path "~/.emacs.d/packages/popup-el")
> > (require 'auto-complete-config)
> > (auto-complete-mode t)
> > (ac-config-default)
> > (global-linum-mode t)
>
> And what should I do after this to reproduce the broken display you
> posted?
>

[Message part 2 (text/html, inline)]

Message #20 received at 20072 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Nikolay Puzanov <punzik <at> gmail.com>
Cc: 20072 <at> debbugs.gnu.org
Subject: Re: bug#20072: Incorrect popup showing
Date: Fri, 13 Mar 2015 19:46:20 +0200

> From: Nikolay Puzanov <punzik <at> gmail.com>
> Date: Fri, 13 Mar 2015 17:17:39 +0000
> Cc: 20072 <at> debbugs.gnu.org
> 
> Open init.el, put cursor to the beginning of buffer, type "(lo", wait for
> popup.

I don't have init.el, I evaluated all of that stuff in *scratch*.  I
did try typing "(lo", but nothing happened.

What am I missing?

Message #23 received at 20072 <at> debbugs.gnu.org (full text, mbox):

From: Nikolay Puzanov <punzik <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 20072 <at> debbugs.gnu.org
Subject: Re: bug#20072: Incorrect popup showing
Date: Fri, 13 Mar 2015 17:53:48 +0000

[Message part 1 (text/plain, inline)]

Switch to emacs-lisp-mode.
Below the cursor should be a few lines of text.


пт, 13 марта 2015 г. в 20:46, Eli Zaretskii <eliz <at> gnu.org>:

> > From: Nikolay Puzanov <punzik <at> gmail.com>
> > Date: Fri, 13 Mar 2015 17:17:39 +0000
> > Cc: 20072 <at> debbugs.gnu.org
> >
> > Open init.el, put cursor to the beginning of buffer, type "(lo", wait for
> > popup.
>
> I don't have init.el, I evaluated all of that stuff in *scratch*.  I
> did try typing "(lo", but nothing happened.
>
> What am I missing?
>

[Message part 2 (text/html, inline)]

Message #26 received at 20072 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Nikolay Puzanov <punzik <at> gmail.com>, m2ym.pub <at> gmail.com, tomo <at> cx4a.org
Cc: 20072 <at> debbugs.gnu.org
Subject: Re: bug#20072: Incorrect popup showing
Date: Sat, 14 Mar 2015 20:05:34 +0200

> From: Nikolay Puzanov <punzik <at> gmail.com>
> Date: Fri, 13 Mar 2015 17:53:48 +0000
> Cc: 20072 <at> debbugs.gnu.org
> 
> Switch to emacs-lisp-mode.
> Below the cursor should be a few lines of text.

OK, I see the problem now (some corruption happens even without
linum-mode, btw).  But I have trouble debugging it: the problematic
display disappears the moment I type anything, even "M-x".  This makes
debugging next to impossible.

So I need a simpler example, one that will produce the problematic
display and not disappear.  I tried to prepare such an example, but
failed, which isn't surprising, as I know almost nothing about the
packages involved.  Can you guys help me with that?

Message #29 received at 20072 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: punzik <at> gmail.com, m2ym.pub <at> gmail.com, tomo <at> cx4a.org
Cc: 20072 <at> debbugs.gnu.org
Subject: Re: bug#20072: Incorrect popup showing
Date: Mon, 16 Mar 2015 22:20:44 +0200

> Date: Sat, 14 Mar 2015 20:05:34 +0200
> From: Eli Zaretskii <eliz <at> gnu.org>
> Cc: 20072 <at> debbugs.gnu.org
> 
> > From: Nikolay Puzanov <punzik <at> gmail.com>
> > Date: Fri, 13 Mar 2015 17:53:48 +0000
> > Cc: 20072 <at> debbugs.gnu.org
> > 
> > Switch to emacs-lisp-mode.
> > Below the cursor should be a few lines of text.
> 
> OK, I see the problem now (some corruption happens even without
> linum-mode, btw).  But I have trouble debugging it: the problematic
> display disappears the moment I type anything, even "M-x".  This makes
> debugging next to impossible.
> 
> So I need a simpler example, one that will produce the problematic
> display and not disappear.  I tried to prepare such an example, but
> failed, which isn't surprising, as I know almost nothing about the
> packages involved.  Can you guys help me with that?

Ping!  Anyone out there interested enough in this bug to help me solve
it?  Please?

Message #34 received at 20072 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Nikolay Puzanov <punzik <at> gmail.com>
Cc: 20072 <at> debbugs.gnu.org
Subject: Re: bug#20072: Incorrect popup showing
Date: Tue, 08 Feb 2022 10:14:31 +0100

Nikolay Puzanov <punzik <at> gmail.com> writes:

> I found bug on display a popup window from auto-complete-mode. I attach two
> screenshot and init.el. Bug appear when global-linum-mode is on.
>
> Commit e589765 bring this bug. I changed the line that seemed most appropriate.
> I not understand how it work, but this patch fix it:

(I'm going through old bug reports that unfortunately weren't resolved
at the time.)

Are you still seeing this issue in recent Emacs versions?  I can't
recall seeing any similar reports, and some of this display code has
changed since this was originally reported.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

Message #39 received at 20072 <at> debbugs.gnu.org (full text, mbox):

From: Николай Пузанов <punzik <at> gmail.com>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 20072 <at> debbugs.gnu.org
Subject: Re: bug#20072: Incorrect popup showing
Date: Tue, 8 Feb 2022 13:12:44 +0300

Hi!
This bug did not show up again. Now I checked on the latest version of Emacs,
everything is fine. To be honest, I even forgot that I sent a bug
report about it.
I think this issue can be closed.

On Tue, Feb 8, 2022 at 12:14 PM Lars Ingebrigtsen <larsi <at> gnus.org> wrote:
>
> Nikolay Puzanov <punzik <at> gmail.com> writes:
>
> > I found bug on display a popup window from auto-complete-mode. I attach two
> > screenshot and init.el. Bug appear when global-linum-mode is on.
> >
> > Commit e589765 bring this bug. I changed the line that seemed most appropriate.
> > I not understand how it work, but this patch fix it:
>
> (I'm going through old bug reports that unfortunately weren't resolved
> at the time.)
>
> Are you still seeing this issue in recent Emacs versions?  I can't
> recall seeing any similar reports, and some of this display code has
> changed since this was originally reported.
>
> --
> (domestic pets only, the antidote for overdose, milk.)
>    bloggy blog: http://lars.ingebrigtsen.no

Message #42 received at 20072 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Николай Пузанов <punzik <at> gmail.com>
Cc: 20072 <at> debbugs.gnu.org
Subject: Re: bug#20072: Incorrect popup showing
Date: Tue, 08 Feb 2022 11:13:34 +0100

Николай Пузанов <punzik <at> gmail.com> writes:

> This bug did not show up again. Now I checked on the latest version of Emacs,
> everything is fine. To be honest, I even forgot that I sent a bug
> report about it.
> I think this issue can be closed.

Thanks for checking; I'm closing this bug report, then.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

Previous Next

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