changeset 110713:06aa6ebe38fa

* lisp/subr.el (make-local-hook): Remove function obsolete since 21.1. * lisp/progmodes/cc-mode.el (make-local-hook): Don't do cc-bytecomp stuff. (c-basic-common-init, c-font-lock-init): Only call make-local-hook on XEmacs. * lisp/progmodes/cc-styles.el (make-local-hook): Don't do cc-bytecomp stuff. (c-make-styles-buffer-local): Only call make-local-hook on XEmacs. * lisp/gnu/gnus-util.el (gnus-make-local-hook): Simplify. * etc/NEWS: Mention above change.
author Glenn Morris <rgm@gnu.org>
date Sat, 02 Oct 2010 18:53:04 -0700
parents 9f132a615f50
children 34826102748f
files etc/NEWS lisp/ChangeLog lisp/gnus/ChangeLog lisp/gnus/gnus-util.el lisp/progmodes/cc-mode.el lisp/progmodes/cc-styles.el lisp/subr.el
diffstat 7 files changed, 22 insertions(+), 44 deletions(-) [+]
line wrap: on
line diff
--- a/etc/NEWS	Sat Oct 02 18:35:07 2010 -0700
+++ b/etc/NEWS	Sat Oct 02 18:53:04 2010 -0700
@@ -551,7 +551,8 @@
 x-make-font-bold, x-make-font-demibold, x-make-font-unbold
 x-make-font-italic, x-make-font-oblique, x-make-font-unitalic
 x-make-font-bold-italic, mldrag-drag-mode-line, mldrag-drag-vertical-line,
-iswitchb-default-keybindings, char-bytes, isearch-return-char
+iswitchb-default-keybindings, char-bytes, isearch-return-char,
+make-local-hook
 
 ** The following variables and aliases, obsolete since at least Emacs 21.1,
 have been removed:
--- a/lisp/ChangeLog	Sat Oct 02 18:35:07 2010 -0700
+++ b/lisp/ChangeLog	Sat Oct 02 18:53:04 2010 -0700
@@ -1,5 +1,12 @@
 2010-10-03  Glenn Morris  <rgm@gnu.org>
 
+	* subr.el (make-local-hook): Remove function obsolete since 21.1.
+	* progmodes/cc-mode.el (make-local-hook): Don't do cc-bytecomp stuff.
+	(c-basic-common-init, c-font-lock-init): Only call make-local-hook on
+	XEmacs.
+	* progmodes/cc-styles.el (make-local-hook): Don't do cc-bytecomp stuff.
+	(c-make-styles-buffer-local): Only call make-local-hook on XEmacs.
+
 	* ps-def.el (leading-code-private-22, charset-bytes, charset-id)
 	(charset-width, find-charset-region, chars-in-region, forward-point)
 	(encode-coding-string, coding-system-p, ccl-execute-on-string)
--- a/lisp/gnus/ChangeLog	Sat Oct 02 18:35:07 2010 -0700
+++ b/lisp/gnus/ChangeLog	Sat Oct 02 18:53:04 2010 -0700
@@ -1,3 +1,7 @@
+2010-10-03  Glenn Morris  <rgm@gnu.org>
+
+	* gnus-util.el (gnus-make-local-hook): Simplify.
+
 2010-10-02  Julien Danjou  <julien@danjou.info>
 
 	* gnus-util.el (gnus-iswitchb-completing-read): New function.
--- a/lisp/gnus/gnus-util.el	Sat Oct 02 18:35:07 2010 -0700
+++ b/lisp/gnus/gnus-util.el	Sat Oct 02 18:53:04 2010 -0700
@@ -149,11 +149,9 @@
 ;; XEmacs.  In Emacs we don't need to call `make-local-hook' first.
 ;; It's harmless, though, so the main purpose of this alias is to shut
 ;; up the byte compiler.
-(defalias 'gnus-make-local-hook
-  (if (eq (get 'make-local-hook 'byte-compile)
-	  'byte-compile-obsolete)
-      'ignore				; Emacs
-    'make-local-hook))			; XEmacs
+(defalias 'gnus-make-local-hook (if (featurep 'xemacs)
+                                    'make-local-hook
+                                  'ignore)
 
 (defun gnus-delete-first (elt list)
   "Delete by side effect the first occurrence of ELT as a member of LIST."
--- a/lisp/progmodes/cc-mode.el	Sat Oct 02 18:35:07 2010 -0700
+++ b/lisp/progmodes/cc-mode.el	Sat Oct 02 18:53:04 2010 -0700
@@ -100,7 +100,6 @@
 (cc-bytecomp-defvar adaptive-fill-first-line-regexp) ; Emacs
 (cc-bytecomp-defun set-keymap-parents)	; XEmacs
 (cc-bytecomp-defun run-mode-hooks)	; Emacs 21.1
-(cc-bytecomp-obsolete-fun make-local-hook) ; Marked obsolete in Emacs 21.1.
 
 ;; We set these variables during mode init, yet we don't require
 ;; font-lock.
@@ -600,9 +599,10 @@
 
   ;; Install the functions that ensure that various internal caches
   ;; don't become invalid due to buffer changes.
-  (make-local-hook 'before-change-functions)
+  (when (featurep 'xemacs)
+    (make-local-hook 'before-change-functions)
+    (make-local-hook 'after-change-functions))
   (add-hook 'before-change-functions 'c-before-change nil t)
-  (make-local-hook 'after-change-functions)
   (add-hook 'after-change-functions 'c-after-change nil t)
   (set (make-local-variable 'font-lock-extend-after-change-region-function)
  	'c-extend-after-change-region))	; Currently (2009-05) used by all
@@ -1113,8 +1113,8 @@
 	  c-beginning-of-syntax
 	  (font-lock-mark-block-function
 	   . c-mark-function)))
-
-  (make-local-hook 'font-lock-mode-hook)
+  (if (featurep 'xemacs)
+      (make-local-hook 'font-lock-mode-hook))
   (add-hook 'font-lock-mode-hook 'c-after-font-lock-init nil t))
 
 (defun c-extend-after-change-region (beg end old-len)
--- a/lisp/progmodes/cc-styles.el	Sat Oct 02 18:35:07 2010 -0700
+++ b/lisp/progmodes/cc-styles.el	Sat Oct 02 18:53:04 2010 -0700
@@ -50,7 +50,6 @@
 
 ;; Silence the compiler.
 (cc-bytecomp-defvar adaptive-fill-first-line-regexp) ; Emacs
-(cc-bytecomp-obsolete-fun make-local-hook) ; Marked obsolete in Emacs 21.1.
 
 
 (defvar c-style-alist
@@ -649,7 +648,7 @@
     (mapc func varsyms)
     ;; Hooks must be handled specially
     (if this-buf-only-p
-	(make-local-hook 'c-special-indent-hook)
+	(if (featurep 'xemacs) (make-local-hook 'c-special-indent-hook))
       (with-no-warnings (make-variable-buffer-local 'c-special-indent-hook))
       (setq c-style-variables-are-local-p t))
     ))
--- a/lisp/subr.el	Sat Oct 02 18:35:07 2010 -0700
+++ b/lisp/subr.el	Sat Oct 02 18:53:04 2010 -0700
@@ -1163,37 +1163,6 @@
 
 ;;;; Hook manipulation functions.
 
-(defun make-local-hook (hook)
-  "Make the hook HOOK local to the current buffer.
-The return value is HOOK.
-
-You never need to call this function now that `add-hook' does it for you
-if its LOCAL argument is non-nil.
-
-When a hook is local, its local and global values
-work in concert: running the hook actually runs all the hook
-functions listed in *either* the local value *or* the global value
-of the hook variable.
-
-This function works by making t a member of the buffer-local value,
-which acts as a flag to run the hook functions in the default value as
-well.  This works for all normal hooks, but does not work for most
-non-normal hooks yet.  We will be changing the callers of non-normal
-hooks so that they can handle localness; this has to be done one by
-one.
-
-This function does nothing if HOOK is already local in the current
-buffer.
-
-Do not use `make-local-variable' to make a hook variable buffer-local."
-  (if (local-variable-p hook)
-      nil
-    (or (boundp hook) (set hook nil))
-    (make-local-variable hook)
-    (set hook (list t)))
-  hook)
-(make-obsolete 'make-local-hook "not necessary any more." "21.1")
-
 (defun add-hook (hook function &optional append local)
   "Add to the value of HOOK the function FUNCTION.
 FUNCTION is not added if already present.