GNU bug report logs - #80098
30.2; [PATCH] Typo in -resetCursorRects makes condition always false

Previous Next

Package: emacs;

Reported by: Yavor Doganov <yavor <at> gnu.org>

Date: Tue, 30 Dec 2025 05:45:02 UTC

Severity: normal

Tags: patch

Found in version 30.2

To reply to this bug, email your comments to 80098 AT debbugs.gnu.org.

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

View this report as an mbox folder, status mbox, maintainer mbox

Report forwarded to yavor <at> gnu.org, bug-gnu-emacs <at> gnu.org:
bug#80098; Package emacs. (Tue, 30 Dec 2025 05:45:02 GMT) Full text and rfc822 format available.
Acknowledgement sent to Yavor Doganov <yavor <at> gnu.org>:
New bug report received and forwarded. Copy sent to yavor <at> gnu.org, bug-gnu-emacs <at> gnu.org. (Tue, 30 Dec 2025 05:45:02 GMT) Full text and rfc822 format available.

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

From: Yavor Doganov <yavor <at> gnu.org>
To: bug-gnu-emacs <at> gnu.org
Subject: 30.2; [PATCH] Typo in -resetCursorRects makes condition always false
Date: Tue, 30 Dec 2025 07:44:32 +0200

[Message part 1 (text/plain, inline)]
There's an innocent typo in both method implementations which makes
the condition always false so the setOnMouseEntered: call is always
skipped.  Also, respondsToSelector: will always return TRUE on GNUstep
(because GNUstep never removes implementations of methods that are
deprecated by Apple) so this expensive runtime check is entirely
redundant there.

I haven't noticed any change in behavior after this change; I just
spotted it while investigating mysterious UI freezes on GNUstep.  The
bug was introduced in commit 95e7195 (December 2017).

[0001-NS-Fix-typo-in-resetCursorRects-implementations.patch (text/plain, attachment)]
This bug report was last modified today.

Previous Next

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