GNU bug report logs - #72731
Wrong source comment in src/w32term.c

Previous Next

Package: emacs;

Reported by: Cecilio Pardo <cpardo <at> imayhem.com>

Date: Tue, 20 Aug 2024 11:45:01 UTC

Severity: wishlist

To reply to this bug, email your comments to 72731 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 bug-gnu-emacs <at> gnu.org:
bug#72731; Package emacs. (Tue, 20 Aug 2024 11:45:01 GMT) Full text and rfc822 format available.

Acknowledgement sent to Cecilio Pardo <cpardo <at> imayhem.com>:
New bug report received and forwarded. Copy sent to bug-gnu-emacs <at> gnu.org. (Tue, 20 Aug 2024 11:45:01 GMT) Full text and rfc822 format available.

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

From: Cecilio Pardo <cpardo <at> imayhem.com>
To: bug-gnu-emacs <at> gnu.org
Subject: Wrong source comment in src/w32term.c
Date: Tue, 20 Aug 2024 13:43:14 +0200
I think this comment in w32term.c is wrong:

    ...
    case WM_RBUTTONUP:
    case WM_XBUTTONDOWN:
    case WM_XBUTTONUP:
      {
            /* If we decide we want to generate an event to be seen
               by the rest of Emacs, we put it here.  */
        Lisp_Object tab_bar_arg = Qnil;
        bool tab_bar_p = 0;
    ...

It was added on

https://git.savannah.gnu.org/cgit/emacs.git/commit/?id=791f420fea18d7ef5ddfc386509de2c84d4f39c3 


+ /* If we decide we want to generate an event to be seen
+ by the rest of Emacs, we put it here. */
+ struct input_event emacs_event;
+ int tool_bar_p = 0;
int button;
int up;


The comment is attached to the emacs_event variable?

The variable was then removed on:

https://git.savannah.gnu.org/cgit/emacs.git/commit/?id=004b1d3853dbb7ef5c5b2e3b5d011718e602c33a 



/* If we decide we want to generate an event to be seen
by the rest of Emacs, we put it here. */
- struct input_event emacs_event;
int tool_bar_p = 0;
int button;
int up;


But the comment stayed there for years to come.





Severity set to 'wishlist' from 'normal' Request was from Stefan Kangas <stefankangas <at> gmail.com> to control <at> debbugs.gnu.org. (Mon, 30 Sep 2024 01:42:01 GMT) Full text and rfc822 format available.

This bug report was last modified 257 days ago.

Previous Next


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