unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Re: master 9d54b12: Expand register-yank-media-handler doc string
       [not found] ` <20211106232430.4DEB920A22@vcs0.savannah.gnu.org>
@ 2021-11-06 23:46   ` Stefan Monnier
  2021-11-06 23:48     ` Lars Ingebrigtsen
  0 siblings, 1 reply; 7+ messages in thread
From: Stefan Monnier @ 2021-11-06 23:46 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: emacs-devel

>  (defun register-yank-media-handler (types handler)

AFAICT this doesn't have much to do with the other `register-`
functions, so could we put it in another, more relevant, namespace
(maybe `yank-media-`)?


        Stefan




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

* Re: master 9d54b12: Expand register-yank-media-handler doc string
  2021-11-06 23:46   ` master 9d54b12: Expand register-yank-media-handler doc string Stefan Monnier
@ 2021-11-06 23:48     ` Lars Ingebrigtsen
  2021-11-07  1:14       ` Lars Ingebrigtsen
  0 siblings, 1 reply; 7+ messages in thread
From: Lars Ingebrigtsen @ 2021-11-06 23:48 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel

Stefan Monnier <monnier@iro.umontreal.ca> writes:

>>  (defun register-yank-media-handler (types handler)
>
> AFAICT this doesn't have much to do with the other `register-`
> functions, so could we put it in another, more relevant, namespace
> (maybe `yank-media-`)?

Oh, I didn't think of registers...  Don't we have some naming convention
for something that...  registers itself with a service?  I'm blanking at
the moment.

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



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

* Re: master 9d54b12: Expand register-yank-media-handler doc string
  2021-11-06 23:48     ` Lars Ingebrigtsen
@ 2021-11-07  1:14       ` Lars Ingebrigtsen
  2021-11-07  3:06         ` Stefan Monnier
  0 siblings, 1 reply; 7+ messages in thread
From: Lars Ingebrigtsen @ 2021-11-07  1:14 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel

Lars Ingebrigtsen <larsi@gnus.org> writes:

> Oh, I didn't think of registers...  Don't we have some naming convention
> for something that...  registers itself with a service?  I'm blanking at
> the moment.

There's four functions called register-* already (that don't have
anything to do with register.el):

register-ccl-program
register-code-conversion-map
register-definition-prefixes
register-input-method

OK, that CCL thing is hopefully going away, but...  it's a very
descriptive word, so perhaps we should just continue to use it for stuff
that registers itself.

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



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

* Re: master 9d54b12: Expand register-yank-media-handler doc string
  2021-11-07  1:14       ` Lars Ingebrigtsen
@ 2021-11-07  3:06         ` Stefan Monnier
  2021-11-07  3:12           ` Lars Ingebrigtsen
  0 siblings, 1 reply; 7+ messages in thread
From: Stefan Monnier @ 2021-11-07  3:06 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: emacs-devel

> OK, that CCL thing is hopefully going away, but...  it's a very
> descriptive word, so perhaps we should just continue to use it for stuff
> that registers itself.

Most "registering" is done via `cl-defmethod`, `add-hook`, and
`add-to-list`, AFAIK.
[ Any chance yank-media handlers can be registered with one of
  those methods?  ]

In any case I don't see any benefit in not using the traditional
namespace prefix (and move the `register` key word *after* the prefix).


        Stefan




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

* Re: master 9d54b12: Expand register-yank-media-handler doc string
  2021-11-07  3:06         ` Stefan Monnier
@ 2021-11-07  3:12           ` Lars Ingebrigtsen
  2021-11-07  3:37             ` Stefan Monnier
  0 siblings, 1 reply; 7+ messages in thread
From: Lars Ingebrigtsen @ 2021-11-07  3:12 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel

Stefan Monnier <monnier@iro.umontreal.ca> writes:

> Most "registering" is done via `cl-defmethod`, `add-hook`, and
> `add-to-list`, AFAIK.
> [ Any chance yank-media handlers can be registered with one of
>   those methods?  ]

Probably not...

> In any case I don't see any benefit in not using the traditional
> namespace prefix (and move the `register` key word *after* the prefix).

`yank-media-handler' perhaps?  Or `define-yank-media-handler'...

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



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

* Re: master 9d54b12: Expand register-yank-media-handler doc string
  2021-11-07  3:12           ` Lars Ingebrigtsen
@ 2021-11-07  3:37             ` Stefan Monnier
  2021-11-07 13:08               ` Lars Ingebrigtsen
  0 siblings, 1 reply; 7+ messages in thread
From: Stefan Monnier @ 2021-11-07  3:37 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: emacs-devel

>> In any case I don't see any benefit in not using the traditional
>> namespace prefix (and move the `register` key word *after* the prefix).
>
> `yank-media-handler' perhaps?  Or `define-yank-media-handler'...

`yank-media-register-handler`?


        Stefan




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

* Re: master 9d54b12: Expand register-yank-media-handler doc string
  2021-11-07  3:37             ` Stefan Monnier
@ 2021-11-07 13:08               ` Lars Ingebrigtsen
  0 siblings, 0 replies; 7+ messages in thread
From: Lars Ingebrigtsen @ 2021-11-07 13:08 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel

Stefan Monnier <monnier@iro.umontreal.ca> writes:

>> `yank-media-handler' perhaps?  Or `define-yank-media-handler'...
>
> `yank-media-register-handler`?

That's really...  awkward...  I think.

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



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

end of thread, other threads:[~2021-11-07 13:08 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20211106232428.21519.78380@vcs0.savannah.gnu.org>
     [not found] ` <20211106232430.4DEB920A22@vcs0.savannah.gnu.org>
2021-11-06 23:46   ` master 9d54b12: Expand register-yank-media-handler doc string Stefan Monnier
2021-11-06 23:48     ` Lars Ingebrigtsen
2021-11-07  1:14       ` Lars Ingebrigtsen
2021-11-07  3:06         ` Stefan Monnier
2021-11-07  3:12           ` Lars Ingebrigtsen
2021-11-07  3:37             ` Stefan Monnier
2021-11-07 13:08               ` Lars Ingebrigtsen

Code repositories for project(s) associated with this public 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).