Re: master d8a00879309: Cease preloading touch-screen.el outside X and Android

Re: master d8a00879309: Cease preloading touch-screen.el outside X and Android

[Arash Esbati]

Po Lu via Mailing list for Emacs changes <emacs-diffs@gnu.org> writes:

> branch: master
> commit d8a00879309a3bf62f6ffcae103aa3bdba776ee9
> Author: Po Lu <luangruo@yahoo.com>
> Commit: Po Lu <luangruo@yahoo.com>
>
>     Cease preloading touch-screen.el outside X and Android
>     
>     * lisp/calc/calc.el (touch-screen-display-keyboard):
>     
>     * lisp/minibuffer.el (clear-minibuffer-message):
>     
>     * lisp/term.el (touch-screen-display-keyboard): Declare
>     touch-screen-display-keyboard before binding or setting it.
>     
>     * lisp/loadup.el: Don't autoload touch-screen.el outside X and
>     Android.
>     
>     * lisp/touch-screen.el: Autoload functions called from commands
>     responding to touch screen events.

I think this change produces these warnings:

  In end of data:
  button.el:498:26: Warning: the function ‘touch-screen-track-tap’ is not
  known to be defined.

  In clear-minibuffer-message:
  minibuffer.el:979:23: Warning: reference to free variable
  ‘touch-screen-current-tool’

  In tab-bar-touchscreen-begin:
  tab-bar.el:440:55: Warning: reference to free variable
  ‘touch-screen-delay’

  In end of data:
  dired.el:5185:8: Warning: the function ‘touch-screen-hold’ is not known
  to be defined.

  In tab-line-track-tap:
  tab-line.el:978:50: Warning: reference to free variable
  ‘touch-screen-delay’

This is with Emacs from master (d8a0087930) on macOS, built with:

git clean -fdx
./autogen
./configure ...
make

Best, Arash

Re: master d8a00879309: Cease preloading touch-screen.el outside X and Android

[Po Lu]

Arash Esbati <arash@gnu.org> writes:

> I think this change produces these warnings:
>
>   In end of data:
>   button.el:498:26: Warning: the function ‘touch-screen-track-tap’ is not
>   known to be defined.
>
>   In clear-minibuffer-message:
>   minibuffer.el:979:23: Warning: reference to free variable
>   ‘touch-screen-current-tool’
>
>   In tab-bar-touchscreen-begin:
>   tab-bar.el:440:55: Warning: reference to free variable
>   ‘touch-screen-delay’
>
>   In end of data:
>   dired.el:5185:8: Warning: the function ‘touch-screen-hold’ is not known
>   to be defined.
>
>   In tab-line-track-tap:
>   tab-line.el:978:50: Warning: reference to free variable
>   ‘touch-screen-delay’
>
> This is with Emacs from master (d8a0087930) on macOS, built with:
>
> git clean -fdx
> ./autogen
> ./configure ...
> make
>
> Best, Arash

I fixed three of these, and the remainder should resolve themselves once
autoloads are generated.  Thanks.

Re: master d8a00879309: Cease preloading touch-screen.el outside X and Android

[Arash Esbati]

Po Lu <luangruo@yahoo.com> writes:

> I fixed three of these, and the remainder should resolve themselves once
> autoloads are generated.  Thanks.

Thanks.  Updating to e4e1e268c8 and building again, I get:

  In clear-minibuffer-message:
  minibuffer.el:979:23: Warning: reference to free variable
  ‘touch-screen-current-tool’

Not sure if I missed that with my last message.

Best, Arash

Re: master d8a00879309: Cease preloading touch-screen.el outside X and Android

[Eli Zaretskii]

> From: Arash Esbati <arash@gnu.org>
> Cc: Po Lu <luangruo@yahoo.com>
> Date: Wed, 06 Dec 2023 23:12:21 +0100
> 
> Po Lu via Mailing list for Emacs changes <emacs-diffs@gnu.org> writes:
> 
> > branch: master
> > commit d8a00879309a3bf62f6ffcae103aa3bdba776ee9
> > Author: Po Lu <luangruo@yahoo.com>
> > Commit: Po Lu <luangruo@yahoo.com>
> >
> >     Cease preloading touch-screen.el outside X and Android
> >     
> >     * lisp/calc/calc.el (touch-screen-display-keyboard):
> >     
> >     * lisp/minibuffer.el (clear-minibuffer-message):
> >     
> >     * lisp/term.el (touch-screen-display-keyboard): Declare
> >     touch-screen-display-keyboard before binding or setting it.
> >     
> >     * lisp/loadup.el: Don't autoload touch-screen.el outside X and
> >     Android.
> >     
> >     * lisp/touch-screen.el: Autoload functions called from commands
> >     responding to touch screen events.
> 
> I think this change produces these warnings:
> 
>   In end of data:
>   button.el:498:26: Warning: the function ‘touch-screen-track-tap’ is not
>   known to be defined.
> 
>   In clear-minibuffer-message:
>   minibuffer.el:979:23: Warning: reference to free variable
>   ‘touch-screen-current-tool’
> 
>   In tab-bar-touchscreen-begin:
>   tab-bar.el:440:55: Warning: reference to free variable
>   ‘touch-screen-delay’
> 
>   In end of data:
>   dired.el:5185:8: Warning: the function ‘touch-screen-hold’ is not known
>   to be defined.
> 
>   In tab-line-track-tap:
>   tab-line.el:978:50: Warning: reference to free variable
>   ‘touch-screen-delay’
> 
> This is with Emacs from master (d8a0087930) on macOS, built with:
> 
> git clean -fdx
> ./autogen
> ./configure ...
> make

The two functions are autoloaded, so I'm guessing that the warnings go
away once loaddefs.el is generated.

The variables indeed need a defvar, I think.