From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Iris =?UTF-8?Q?Garc=C3=ADa?= Newsgroups: gmane.emacs.bugs Subject: bug#48674: Frames and minibuffer bug Date: Tue, 1 Jun 2021 13:29:25 +0200 Message-ID: References: <1911d1b0-ed9f-7359-b28c-fbaef27df8f3@gmx.at> <1e21b121-91c1-cbe9-d9ae-24915f163ae5@gmx.at> Mime-Version: 1.0 Content-Type: multipart/alternative; boundary="0000000000003215ea05c3b2a82d" Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="12586"; mail-complaints-to="usenet@ciao.gmane.io" Cc: 48674-done@debbugs.gnu.org To: Alan Mackenzie Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Tue Jun 01 13:30:40 2021 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1lo2bY-00034L-2e for geb-bug-gnu-emacs@m.gmane-mx.org; Tue, 01 Jun 2021 13:30:40 +0200 Original-Received: from localhost ([::1]:42850 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lo2bW-0002qy-Km for geb-bug-gnu-emacs@m.gmane-mx.org; Tue, 01 Jun 2021 07:30:38 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:46496) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lo2aw-0001jT-N6 for bug-gnu-emacs@gnu.org; Tue, 01 Jun 2021 07:30:02 -0400 Original-Received: from debbugs.gnu.org ([209.51.188.43]:52540) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1lo2aw-0002fU-B6 for bug-gnu-emacs@gnu.org; Tue, 01 Jun 2021 07:30:02 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1lo2aw-0002mO-7W for bug-gnu-emacs@gnu.org; Tue, 01 Jun 2021 07:30:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Iris =?UTF-8?Q?Garc=C3=ADa?= Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Tue, 01 Jun 2021 11:30:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 48674 X-GNU-PR-Package: emacs Original-Received: via spool by 48674-done@debbugs.gnu.org id=D48674.162254698510636 (code D ref 48674); Tue, 01 Jun 2021 11:30:02 +0000 Original-Received: (at 48674-done) by debbugs.gnu.org; 1 Jun 2021 11:29:45 +0000 Original-Received: from localhost ([127.0.0.1]:35853 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1lo2af-0002lU-4B for submit@debbugs.gnu.org; Tue, 01 Jun 2021 07:29:45 -0400 Original-Received: from mail-ej1-f42.google.com ([209.85.218.42]:42970) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1lo2ac-0002lE-Sb for 48674-done@debbugs.gnu.org; Tue, 01 Jun 2021 07:29:43 -0400 Original-Received: by mail-ej1-f42.google.com with SMTP id qq22so12880476ejb.9 for <48674-done@debbugs.gnu.org>; Tue, 01 Jun 2021 04:29:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=3Cz16JuDN8CTXpewADsDWKadRd0qpmxJZftDDA1rIOw=; b=pIV7dwZnP90tEr7oyy4yKqzAH3q2G87bAXw4s+VaddinaWi1PdJ+TE4tzHi0lwvVmM FydaXSSgbjCm6EDkRMpv1PrnnEQHbCI1UQvw2ZOKDPBzLuCQv99kuHXYozTLeIkO8x5u /IxIaxJVp0ei+ze1AhLC9Em3FtG8Bl27FrGBZTWIZ43AtxoxpK7pEuwCfXRmAVQSndhd 9BrAgRlpEbsv8eYLjp42bp6p1dpsLPNmkF6d5TFouXfXF439dGQqXUWCD6WVIFdJaQ8B nXCJcApFJs32Gi6CnFPYB16extzThW6gTb/2IX35m2iseGomh1WIi/IEmf4qYlpVY12m r0Vg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=3Cz16JuDN8CTXpewADsDWKadRd0qpmxJZftDDA1rIOw=; b=NHnFHE+W7fQ+vgmCLlfEDtyvUU/ONPVnS+l2I1x1vsEQLxEbKhGk4z7mfxzGhuTJ8X QnL6XAwvTN5FkHquR2ktIYHlZAbwPfRTGxQDw4kmjdjvMHEzl7XR+BCLScZUcKUJL6mt iYlCEEa6lvsb1YR+lDTqz4kZw8WZan5OucnPh8/nwLoISPnSdlYRw0IDv5DR2nAXVkBn 9guA1DxCkIUQCTjkjwz8YbKSstjUg8GTZ9J67L3uYy5soXG7NdcGMNG7UIb1JTyHVYY4 Xz3YfTCLV8ewLCJKZMk+W8y6JlZtjETN/VKl0r4erIbEhVguoqN3K0E3HkOqLOLhwx4X 400w== X-Gm-Message-State: AOAM531YJrqYhrA/VJR9L+CfAkyw3PyPk2E4gQcE5k8hC7ugjD9cks6d MSk/UQmBZLO37gwpwTsauFVwjxADo2gZEAnwix0= X-Google-Smtp-Source: ABdhPJxz80wq0fAzRec8npCr57sFIZd/j885+8k3J9kI/31nNKAMkgzOSfDrTKlUTZ0ldwkyaaxiWBem6N126aMk0jU= X-Received: by 2002:a17:906:eb88:: with SMTP id mh8mr11116193ejb.540.1622546977001; Tue, 01 Jun 2021 04:29:37 -0700 (PDT) In-Reply-To: X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Original-Sender: "bug-gnu-emacs" Xref: news.gmane.io gmane.emacs.bugs:207768 Archived-At: --0000000000003215ea05c3b2a82d Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Hi Alan, No worries! I did follow the thread and tried your 3 patches, the last one is indeed working as expected as far as I can tell. Thank you very much for the quick response and fix. Regards, Iris. On Mon, 31 May 2021 at 18:36, Alan Mackenzie wrote: > Hello, Iris. > > Firstly, forgive me for not answering you sooner. I didn't want want to > waste any more of your time with any more unusable patches. This was a > tricky bug to solve, and indeed only the third patch attempt was > satisfactory. > > I have now committed this third patch, and would ask you to remove the > patch I sent you a few days ago, and update your Emacs to the current > master version. > > I am closing the bug with this post, but if you find any more trouble > with it, would you please let us know, so that we can open it again. > Thanks! > > -- > Alan Mackenzie (Nuremberg, Germany). > > > On Thu, May 27, 2021 at 19:56:03 +0000, Iris Garc=C3=ADa wrote: > > Hi Martin, > > > I forgot to include you in my last mail where I said: > > > I think I have found the new issue (it is related to the former one), m= y > > > code this time was the following: > > > > (defvar box-cursor t) > > > > > > (defun test/set-cursor() > > > "Set cursor in all frames depending on the active state." > > > (interactive) > > > (dolist (frame (frame-list)) > > > (with-selected-frame frame > > > (if box-cursor > > > (progn > > > (modify-frame-parameters > > > frame (list (cons 'cursor-type 'box))) > > > (modify-frame-parameters > > > frame (list (cons 'cursor-color "#00A9FE")))) > > > (progn > > > (modify-frame-parameters > > > frame (list (cons 'cursor-type 'hbar))) > > > (modify-frame-parameters > > > frame (list (cons 'cursor-color "green"))) > > > ))))) > > > > > > (defun test/enter-minibuffer() > > > (setq box-cursor nil) > > > (test/set-cursor)) > > > > > > (defun test/exit-minibuffer() > > > (setq box-cursor t) > > > (test/set-cursor)) > > > > > > > > > (add-hook 'window-state-change-hook #'test/enter-minibuffer) > > > (add-hook 'window-state-change-hook #'test/exit-minibuffer) > > > > > > (server-start) > > > (make-frame > > > > The only difference is the add-hook, this time using > > > window-state-change-hook instead of minibuffer-... > > > This leads to the same bug. > > > > Regards, > > > > Iris. > --0000000000003215ea05c3b2a82d Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Hi Alan,

No worries! I did f= ollow the thread and tried your 3 patches, the last one is indeed working a= s expected as far as I can tell.

Thank you very mu= ch for the quick response and fix.

Regards,
<= div>
Iris.

On Mon, 31 May 2021 at 18:36, Alan Macken= zie <acm@muc.de> wrote:
Hello, Iris.

Firstly, forgive me for not answering you sooner.=C2=A0 I didn't want w= ant to
waste any more of your time with any more unusable patches.=C2=A0 This was = a
tricky bug to solve, and indeed only the third patch attempt was
satisfactory.

I have now committed this third patch, and would ask you to remove the
patch I sent you a few days ago, and update your Emacs to the current
master version.

I am closing the bug with this post, but if you find any more trouble
with it, would you please let us know, so that we can open it again.
Thanks!

--
Alan Mackenzie (Nuremberg, Germany).


On Thu, May 27, 2021 at 19:56:03 +0000, Iris Garc=C3=ADa wrote:
> Hi Martin,

> I forgot to include you in my last mail where I said:

> I think I have found the new issue (it is related to the former one), = my
> > code this time was the following:

> > (defvar box-cursor t)
> >
> > (defun test/set-cursor()
> >=C2=A0 =C2=A0"Set cursor in all frames depending on the activ= e state."
> >=C2=A0 =C2=A0(interactive)
> >=C2=A0 =C2=A0(dolist (frame (frame-list))
> >=C2=A0 =C2=A0 =C2=A0(with-selected-frame frame
> >=C2=A0 =C2=A0 =C2=A0 =C2=A0(if box-cursor
> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0(progn
> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0(modify-frame-para= meters
> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 frame (list (cons= 'cursor-type 'box)))
> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0(modify-frame-para= meters
> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 frame (list (cons= 'cursor-color "#00A9FE"))))
> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0(progn
> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0(modify-frame-parameters<= br> > >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 frame (list (cons 'c= ursor-type 'hbar)))
> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0(modify-frame-parameters<= br> > >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 frame (list (cons 'c= ursor-color "green")))
> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0)))))
> >
> > (defun test/enter-minibuffer()
> >=C2=A0 =C2=A0(setq box-cursor nil)
> >=C2=A0 =C2=A0(test/set-cursor))
> >
> > (defun test/exit-minibuffer()
> >=C2=A0 =C2=A0(setq box-cursor t)
> >=C2=A0 =C2=A0(test/set-cursor))
> >
> >
> > (add-hook 'window-state-change-hook #'test/enter-minibuff= er)
> > (add-hook 'window-state-change-hook #'test/exit-minibuffe= r)
> >
> > (server-start)
> > (make-frame

> > The only difference is the add-hook, this time using
> > window-state-change-hook instead of minibuffer-...
> > This leads to the same bug.

> > Regards,

> > Iris.
--0000000000003215ea05c3b2a82d--