Merge from gnus--devo--0

Revision: emacs@sv.gnu.org/emacs--devo--0--patch-1460

diff --git a/lisp/gnus/ChangeLog b/lisp/gnus/ChangeLog
index 66fafa90ad60c84d023bc59bd5794eb87bbf7152..21132ebb5680369414d4e82df9e8fcb618bc7622 100644 (file)
--- a/lisp/gnus/ChangeLog
+++ b/lisp/gnus/ChangeLog
@@ -1,3 +1,14 @@
+2008-10-31  Teodor Zlatanov  <tzz@lifelogs.com>
+
+       * ietf-drums.el (ietf-drums-remove-comments): Localize second
+       condition-case to only the forward-sexp call.
+
+2008-10-31  Teodor Zlatanov  <tzz@lifelogs.com>
+
+       * ietf-drums.el (ietf-drums-remove-comments): Fix bug with single
+       quotes contained.  Make it more robust regardless by an extra
+       condition-case wrapper.
+
 2008-10-03  Katsumi Yamaoka  <yamaoka@jpl.org>
 
        * nnml.el (nnml-request-expire-articles): Check if the function set to

diff --git a/lisp/gnus/ietf-drums.el b/lisp/gnus/ietf-drums.el
index 4f2e9afe56d6e49f242751af3e0c44abb1a3f7c2..78567f622df130aebd831b4496700ccff435978c 100644 (file)
--- a/lisp/gnus/ietf-drums.el
+++ b/lisp/gnus/ietf-drums.el
@@ -123,9 +123,18 @@ backslash and doublequote.")
        (setq c (char-after))
        (cond
         ((eq c ?\")
-         (forward-sexp 1))
+         (condition-case err
+             (forward-sexp 1)
+           (error (goto-char (point-max)))))
         ((eq c ?\()
-         (delete-region (point) (progn (forward-sexp 1) (point))))
+         (delete-region
+              (point)
+              (condition-case nil
+                  (with-syntax-table (copy-syntax-table ietf-drums-syntax-table)
+                    (modify-syntax-entry ?\" "w")
+                    (forward-sexp 1)
+                    (point))
+                (error (point-max)))))
         (t
          (forward-char 1))))
       (buffer-string))))