unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Re: master 0dc75daf11: Filter out NS non-key events from `where-is-internal'
       [not found] ` <20220618120636.5EF8CC0169A@vcs2.savannah.gnu.org>
@ 2022-06-18 12:35   ` Po Lu
  2022-06-18 12:37     ` Lars Ingebrigtsen
  0 siblings, 1 reply; 6+ messages in thread
From: Po Lu @ 2022-06-18 12:35 UTC (permalink / raw)
  To: emacs-devel; +Cc: Lars Ingebrigtsen

Lars Ingebrigtsen <larsi@gnus.org> writes:

> +	       (cons  1 (make-non-key-event 'ns-power-off))
> +;;;	       (cons  9 (make-non-key-event 'ns-insert-working-text))
> +;;;	       (cons 10 (make-non-key-event 'ns-delete-working-text))

These are actually keys and probably should not be filtered out.

> +	       (cons 14 (make-non-key-event 'ns-show-prefs))))))

This is reportedly sent in response to a key press (s-,) but I couldn't
verify that.



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: master 0dc75daf11: Filter out NS non-key events from `where-is-internal'
  2022-06-18 12:35   ` master 0dc75daf11: Filter out NS non-key events from `where-is-internal' Po Lu
@ 2022-06-18 12:37     ` Lars Ingebrigtsen
  2022-06-18 13:04       ` Po Lu
  0 siblings, 1 reply; 6+ messages in thread
From: Lars Ingebrigtsen @ 2022-06-18 12:37 UTC (permalink / raw)
  To: Po Lu; +Cc: emacs-devel

Po Lu <luangruo@yahoo.com> writes:

> Lars Ingebrigtsen <larsi@gnus.org> writes:
>
>> +	       (cons  1 (make-non-key-event 'ns-power-off))
>> +;;;	       (cons  9 (make-non-key-event 'ns-insert-working-text))
>> +;;;	       (cons 10 (make-non-key-event 'ns-delete-working-text))
>
> These are actually keys and probably should not be filtered out.

power-off or all three?  (The other two are commented out.)

>> +	       (cons 14 (make-non-key-event 'ns-show-prefs))))))
>
> This is reportedly sent in response to a key press (s-,) but I couldn't
> verify that.

s-, is still bound on Macos.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: master 0dc75daf11: Filter out NS non-key events from `where-is-internal'
  2022-06-18 12:37     ` Lars Ingebrigtsen
@ 2022-06-18 13:04       ` Po Lu
  2022-06-18 13:13         ` Stefan Kangas
  2022-06-18 13:17         ` Lars Ingebrigtsen
  0 siblings, 2 replies; 6+ messages in thread
From: Po Lu @ 2022-06-18 13:04 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: emacs-devel

Lars Ingebrigtsen <larsi@gnus.org> writes:

> power-off or all three?  (The other two are commented out.)

All three.  power-off is sent when pressing the power button.



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: master 0dc75daf11: Filter out NS non-key events from `where-is-internal'
  2022-06-18 13:04       ` Po Lu
@ 2022-06-18 13:13         ` Stefan Kangas
  2022-06-18 13:17         ` Lars Ingebrigtsen
  1 sibling, 0 replies; 6+ messages in thread
From: Stefan Kangas @ 2022-06-18 13:13 UTC (permalink / raw)
  To: Po Lu; +Cc: Lars Ingebrigtsen, Emacs developers

Po Lu <luangruo@yahoo.com> writes:

> > power-off or all three?  (The other two are commented out.)
>
> All three.  power-off is sent when pressing the power button.

When I press the power button on my macOS machine, emacs -Q doesn't
pick it up (tested with C-h k).  (The power button generally has to be
pressed for a few seconds before it has any effect, on my 2015 MacBook
Pro.)



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: master 0dc75daf11: Filter out NS non-key events from `where-is-internal'
  2022-06-18 13:04       ` Po Lu
  2022-06-18 13:13         ` Stefan Kangas
@ 2022-06-18 13:17         ` Lars Ingebrigtsen
  2022-06-19  0:45           ` Po Lu
  1 sibling, 1 reply; 6+ messages in thread
From: Lars Ingebrigtsen @ 2022-06-18 13:17 UTC (permalink / raw)
  To: Po Lu; +Cc: emacs-devel

Po Lu <luangruo@yahoo.com> writes:

> Lars Ingebrigtsen <larsi@gnus.org> writes:
>
>> power-off or all three?  (The other two are commented out.)
>
> All three.  power-off is sent when pressing the power button.

That event isn't about picking up the user pressing the power button,
but rather:

@item ns-power-off
@cindex ns-power-off event
This event occurs when the user logs out and Emacs is still running, or when
``Quit Emacs'' is chosen from the application menu.
The default behavior is to save all file-visiting buffers.


-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: master 0dc75daf11: Filter out NS non-key events from `where-is-internal'
  2022-06-18 13:17         ` Lars Ingebrigtsen
@ 2022-06-19  0:45           ` Po Lu
  0 siblings, 0 replies; 6+ messages in thread
From: Po Lu @ 2022-06-19  0:45 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: emacs-devel

Lars Ingebrigtsen <larsi@gnus.org> writes:

> That event isn't about picking up the user pressing the power button,
> but rather:

It's sent when the power button is pressed for me, at least.
I guess the default action is to log out and power off.



^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2022-06-19  0:45 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <165555399600.11245.10010229138126288065@vcs2.savannah.gnu.org>
     [not found] ` <20220618120636.5EF8CC0169A@vcs2.savannah.gnu.org>
2022-06-18 12:35   ` master 0dc75daf11: Filter out NS non-key events from `where-is-internal' Po Lu
2022-06-18 12:37     ` Lars Ingebrigtsen
2022-06-18 13:04       ` Po Lu
2022-06-18 13:13         ` Stefan Kangas
2022-06-18 13:17         ` Lars Ingebrigtsen
2022-06-19  0:45           ` Po Lu

Code repositories for project(s) associated with this inbox:

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).