mm-view.el (mm-display-inline-fontify): Make the font-lock variables buffer-local...

diff --git a/lisp/gnus/ChangeLog b/lisp/gnus/ChangeLog
index 27693ab771b3ecfae1f112f0dd84fbacb1ce0786..718983717ae944f51a79aedad0641d1395298c67 100644 (file)
--- a/lisp/gnus/ChangeLog
+++ b/lisp/gnus/ChangeLog
@@ -1,3 +1,10 @@
+2011-11-21  Lars Magne Ingebrigtsen  <larsi@gnus.org>
+
+       * mm-view.el (mm-display-inline-fontify): Make the font-lock variables
+       buffer-local instead of binding them to avoid warnings.  Also demote
+       errors (bug#10063).
+       (font-lock-mode-hook): Shut up byte compiler.
+
 2011-11-20  Juanma Barranquero  <lekktu@gmail.com>
 
        * mm-util.el (mm-charset-after): Fix typo.

diff --git a/lisp/gnus/mm-view.el b/lisp/gnus/mm-view.el
index 6358e34176f3b46dd1aa0cd35ad6637de2c06b95..854ca3497dafa86ec892e7c33d090355c73825d1 100644 (file)
--- a/lisp/gnus/mm-view.el
+++ b/lisp/gnus/mm-view.el
@@ -566,6 +566,8 @@
                     (face-property 'default prop) (current-buffer))))
              (delete-region ,(point-min-marker) ,(point-max-marker)))))))))
 
+;; Shut up byte-compiler.
+(defvar font-lock-mode-hook)
 (defun mm-display-inline-fontify (handle &optional mode)
   "Insert HANDLE inline fontifying with MODE.
 If MODE is not set, try to find mode automatically."
@@ -597,25 +599,25 @@ If MODE is not set, try to find mode automatically."
                    (t
                     text)))
       (require 'font-lock)
-      (let ((font-lock-maximum-size nil)
-           ;; Disable support modes, e.g., jit-lock, lazy-lock, etc.
-           (font-lock-mode-hook nil)
-           (font-lock-support-mode nil)
-           ;; I find font-lock a bit too verbose.
-           (font-lock-verbose nil))
+      ;; I find font-lock a bit too verbose.
+      (let ((font-lock-verbose nil))
+       ;; Disable support modes, e.g., jit-lock, lazy-lock, etc.
+       (set (make-local-variable 'font-lock-mode-hook) nil)
+       (set (make-local-variable 'font-lock-support-mode) nil)
         (setq buffer-file-name (mm-handle-filename handle))
         (set (make-local-variable 'enable-local-variables) nil)
-        (if mode
-            (funcall mode)
-         (let ((auto-mode-alist
-                (delq (rassq 'doc-view-mode-maybe auto-mode-alist)
-                      (copy-sequence auto-mode-alist))))
-           (set-auto-mode)))
-       ;; The mode function might have already turned on font-lock.
-        ;; Do not fontify if the guess mode is fundamental.
-       (unless (or (symbol-value 'font-lock-mode)
-                    (eq major-mode 'fundamental-mode))
-         (font-lock-fontify-buffer)))
+       (with-demoted-errors
+         (if mode
+             (funcall mode)
+           (let ((auto-mode-alist
+                  (delq (rassq 'doc-view-mode-maybe auto-mode-alist)
+                        (copy-sequence auto-mode-alist))))
+             (set-auto-mode)))
+         ;; The mode function might have already turned on font-lock.
+         ;; Do not fontify if the guess mode is fundamental.
+         (unless (or (symbol-value 'font-lock-mode)
+                     (eq major-mode 'fundamental-mode))
+           (font-lock-fontify-buffer))))
       ;; By default, XEmacs font-lock uses non-duplicable text
       ;; properties.  This code forces all the text properties
       ;; to be copied along with the text.