unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* local-after-save-hook?
@ 2002-06-21 20:06 Sam Steingold
  2002-06-21 20:20 ` local-after-save-hook? Stefan Monnier
  0 siblings, 1 reply; 4+ messages in thread
From: Sam Steingold @ 2002-06-21 20:06 UTC (permalink / raw)


There is a buffer-local hook `local-write-file-hooks' (which is called
before writing the file) but no `local-after-save-hook' (called after
the file is written).  Why?

I can emulate this behavior:

(defun my-after-save-hook-function ()
  do something useful
  (remove-hook 'after-save-hook 'my-after-save-hook-function))

(add-hook 'local-write-file-hooks
  (lambda () (add-hook 'after-save-hook 'my-after-save-hook-function)))

but I would prefer to have `local-after-save-hook'

do you mind if I add it?

-- 
Sam Steingold (http://www.podval.org/~sds) running RedHat7.2 GNU/Linux
<http://www.camera.org> <http://www.iris.org.il> <http://www.memri.org/>
<http://www.mideasttruth.com/> <http://www.palestine-central.com/links.html>
When we write programs that "learn", it turns out we do and they don't.

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

* Re: local-after-save-hook?
  2002-06-21 20:06 local-after-save-hook? Sam Steingold
@ 2002-06-21 20:20 ` Stefan Monnier
  2002-06-22 22:51   ` local-after-save-hook? Richard Stallman
  0 siblings, 1 reply; 4+ messages in thread
From: Stefan Monnier @ 2002-06-21 20:20 UTC (permalink / raw)
  Cc: emacs-devel

> There is a buffer-local hook `local-write-file-hooks' (which is called
> before writing the file) but no `local-after-save-hook' (called after
> the file is written).  Why?

Because local-write-file-hooks should disappear.
Just use the LOCAL argument to `add-hook' instead.


	Stefan

PS: I have a patch to rename those *-file-*-hooks to *-file-*-hook or
    *-file-*-function to better follow the naming convention.  Now that
    we have defvaralias, we can do this switch with 100% backward
    compatibility.  Any interest ?

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

* Re: local-after-save-hook?
  2002-06-21 20:20 ` local-after-save-hook? Stefan Monnier
@ 2002-06-22 22:51   ` Richard Stallman
  2002-06-23 22:23     ` local-after-save-hook? Stefan Monnier
  0 siblings, 1 reply; 4+ messages in thread
From: Richard Stallman @ 2002-06-22 22:51 UTC (permalink / raw)
  Cc: sds, emacs-devel

    PS: I have a patch to rename those *-file-*-hooks to *-file-*-hook or
	*-file-*-function to better follow the naming convention.  Now that
	we have defvaralias, we can do this switch with 100% backward
	compatibility.  Any interest ?

That seems like a good idea.  Please do it.

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

* Re: local-after-save-hook?
  2002-06-22 22:51   ` local-after-save-hook? Richard Stallman
@ 2002-06-23 22:23     ` Stefan Monnier
  0 siblings, 0 replies; 4+ messages in thread
From: Stefan Monnier @ 2002-06-23 22:23 UTC (permalink / raw)
  Cc: monnier+gnu/emacs, sds, emacs-devel

>     PS: I have a patch to rename those *-file-*-hooks to *-file-*-hook or
> 	*-file-*-function to better follow the naming convention.  Now that
> 	we have defvaralias, we can do this switch with 100% backward
> 	compatibility.  Any interest ?
> 
> That seems like a good idea.  Please do it.

Done.


	Stefan

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

end of thread, other threads:[~2002-06-23 22:23 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2002-06-21 20:06 local-after-save-hook? Sam Steingold
2002-06-21 20:20 ` local-after-save-hook? Stefan Monnier
2002-06-22 22:51   ` local-after-save-hook? Richard Stallman
2002-06-23 22:23     ` local-after-save-hook? Stefan Monnier

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).