GNU bug report logs - #63415
30.0.50; More Build Warnings

Previous Next

Package: emacs;

Reported by: Jacob Faibussowitsch <jacob.fai <at> gmail.com>

Date: Wed, 10 May 2023 11:26:01 UTC

Severity: normal

Found in version 30.0.50

Done: Andrea Corallo <acorallo <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Andrea Corallo <acorallo <at> gnu.org>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#63415: closed (30.0.50; More Build Warnings)
Date: Thu, 21 Mar 2024 11:10:02 +0000
[Message part 1 (text/plain, inline)]
Your message dated Thu, 21 Mar 2024 07:08:31 -0400
with message-id <yp1jzlvsuz4.fsf <at> fencepost.gnu.org>
and subject line Re: bug#63415: 30.0.50; More Build Warnings
has caused the debbugs.gnu.org bug report #63415,
regarding 30.0.50; More Build Warnings
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
63415: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=63415
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
From: Jacob Faibussowitsch <jacob.fai <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 30.0.50; More Build Warnings
Date: Wed, 10 May 2023 07:25:34 -0400
[Message part 3 (text/plain, inline)]
Hello,

dispnew.c:3650:12: warning: variable 'changed_p' set but not used [-Wunused-but-set-variable]
      bool changed_p = 0, mouse_face_overwritten_p = 0;
           ^
1 warning generated.

In native--compile-async:
comp.el:4240:14: Warning: value from call to ‘cl-substitute’ is unused

In end of data:
dynamic-setting.el:58:12: Warning: the function ‘reconsider-frame-fonts’ is not known to be defined.

In end of data:
prog-mode.el:164:38: Warning: the function ‘treesit-node-at’ might not be defined at runtime.

In haiku-dnd-wheel-modifier-type:
haiku-win.el:520:25: Warning: reference to free variable ‘mouse-wheel-scroll-amount’

In haiku-handle-drag-wheel:
haiku-win.el:543:21: Warning: reference to free variable ‘mwheel-scroll-up-function’
haiku-win.el:545:21: Warning: reference to free variable ‘mwheel-scroll-down-function’
haiku-win.el:546:34: Warning: reference to free variable ‘mouse-wheel-flip-direction’
haiku-win.el:547:34: Warning: reference to free variable ‘mwheel-scroll-right-function’
haiku-win.el:548:32: Warning: reference to free variable ‘mwheel-scroll-left-function’
haiku-win.el:571:39: Warning: reference to free variable ‘mouse-wheel-progressive-speed’

In x-dnd-wheel-modifier-type:
x-dnd.el:757:25: Warning: reference to free variable ‘mouse-wheel-scroll-amount’

In x-dnd-mwheel-scroll:
x-dnd.el:794:29: Warning: reference to free variable ‘mouse-wheel-progressive-speed’
x-dnd.el:796:23: Warning: reference to free variable ‘mouse-wheel-tilt-scroll’
x-dnd.el:804:34: Warning: reference to free variable ‘mwheel-scroll-right-function’
x-dnd.el:805:32: Warning: reference to free variable ‘mwheel-scroll-down-function’
x-dnd.el:808:34: Warning: reference to free variable ‘mwheel-scroll-left-function’
x-dnd.el:809:32: Warning: reference to free variable ‘mwheel-scroll-up-function’
x-dnd.el:811:34: Warning: reference to free variable ‘mouse-wheel-flip-direction’

Best regards,

Jacob Faibussowitsch
(Jacob Fai - booss - oh - vitch)

[config.log (application/octet-stream, attachment)]
[Message part 5 (text/plain, inline)]

[Message part 6 (message/rfc822, inline)]
From: Andrea Corallo <acorallo <at> gnu.org>
To: Andrea Corallo <akrl <at> sdf.org>
Cc: Eli Zaretskii <eliz <at> gnu.org>, 63415-done <at> debbugs.gnu.org,
 jacob.fai <at> gmail.com
Subject: Re: bug#63415: 30.0.50; More Build Warnings
Date: Thu, 21 Mar 2024 07:08:31 -0400
Andrea Corallo <akrl <at> sdf.org> writes:

> Eli Zaretskii <eliz <at> gnu.org> writes:
>
>>> From: Andrea Corallo <akrl <at> sdf.org>
>>> Cc: jacob.fai <at> gmail.com,  63415 <at> debbugs.gnu.org
>>> Date: Wed, 17 May 2023 14:11:44 +0000
>>> 
>>> Eli Zaretskii <eliz <at> gnu.org> writes:
>>> 
>>> > This is just a cleanup, and for a warning that is new on master.  So
>>> > please install on master.
>>> 
>>> Unfortunatelly this is a bugfix.
>>> 
>>> The case is when a manual async compilation has been asked for a file,
>>> and before it's initiated the jit machinery ask as well for a
>>> compilation.  I think with the current code at worst we'll miss to load
>>> automatically the .eln for the jit case and everything will keep on
>>> working with the bytecoded version.
>>> 
>>> I don't think the whole mechanism is trivial to be verified, so given
>>> the unlikeliness of the condition and the non severity of the bug I
>>> prefered to push the fix only to master for now with 8a92d00470b.  Happy
>>> to change my mind if we feel is important.
>>
>> Thanks.  Let's try to remember revisiting this when Emacs 29.2 will
>> come close to release.  By that time I hope we will know whether this
>> fix is safe or not, and will be able to backport it if it is.
>
> SGTM

My bad I think we missed the 29.2 train, anyway I installed this into
emacs-29 with 51848e4731f.

Closing this.

Thanks

  Andrea


This bug report was last modified 1 year and 158 days ago.

Previous Next


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