Monkey patch for the cc-mode that comes with emacs-23.x; that version
authorTom Yu <tlyu@mit.edu>
Tue, 3 Nov 2009 03:14:41 +0000 (03:14 +0000)
committerTom Yu <tlyu@mit.edu>
Tue, 3 Nov 2009 03:14:41 +0000 (03:14 +0000)
of cc-mode has a bug that causes incorrect indentation of case labels
containing character constants.  Already fixed upstream in unreleased
cc-mode sources.

git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@23122 dc483132-0cff-0310-8789-dd5450dbe970

src/util/krb5-c-style.el
src/util/krb5-hack-cc-mode-caselabel.el [new file with mode: 0644]

index 2aa7dfce64bda38c57f76ce1a2ef5beeaf9fcb6b..b060e8803a73306c83a1c01cf27f61e5e52a3ccb 100644 (file)
 ;; Use hack-local-variables-hook because the c-mode hooks run before
 ;; hack-local-variables runs.
 (add-hook 'hack-local-variables-hook 'krb5-c-mode-hook)
+
+;; emacs-23.x has a buggy cc-mode that incorrectly deals with case
+;; labels with character constants.
+(if (and (string-match "^23\." emacs-version)
+         (require 'cc-defs)
+         (string-match "5.31.[0-7]" c-version))
+    (let ((load-path (cons (file-name-directory load-file-name) load-path)))
+      (load "krb5-hack-cc-mode-caselabel")))
diff --git a/src/util/krb5-hack-cc-mode-caselabel.el b/src/util/krb5-hack-cc-mode-caselabel.el
new file mode 100644 (file)
index 0000000..4857c53
--- /dev/null
@@ -0,0 +1,44 @@
+;;; -*- mode: emacs-lisp; indent-tabs-mode: nil -*-
+
+;; emacs-23.x has a bug in cc-mode that that incorrectly deals with
+;; case labels with character constants.
+
+(require 'cl)
+(require 'cc-defs)
+(require 'cc-vars)
+(require 'cc-langs)
+
+;; Hack load-in-progress to silence the c-lang-defconst error.  For
+;; some reason, load-in-progress is nil at some times when it
+;; shouldn't be, at least on released emacs-23.1.1.
+(let ((load-in-progress t))
+
+  ;; Updated c-nonlabel-token-key based on cc-langs.el 5.267.2.22, to
+  ;; allow character constants in case labels.
+  (c-lang-defconst c-nonlabel-token-key
+    "Regexp matching things that can't occur in generic colon labels,
+neither in a statement nor in a declaration context.  The regexp is
+tested at the beginning of every sexp in a suspected label,
+i.e. before \":\".  Only used if `c-recognize-colon-labels' is set."
+    t (concat
+       ;; Don't allow string literals.
+       "\"\\|"
+       ;; All keywords except `c-label-kwds' and `c-protection-kwds'.
+       (c-make-keywords-re t
+         (set-difference (c-lang-const c-keywords)
+                         (append (c-lang-const c-label-kwds)
+                                 (c-lang-const c-protection-kwds))
+                         :test 'string-equal)))
+    ;; Also check for open parens in C++, to catch member init lists in
+    ;; constructors.  We normally allow it so that macros with arguments
+    ;; work in labels.
+    c++ (concat "\\s\(\\|" (c-lang-const c-nonlabel-token-key)))
+  (c-lang-defvar c-nonlabel-token-key (c-lang-const c-nonlabel-token-key))
+
+  ;; Monkey-patch by way of c-mode-common-hook, as the byte-compiled
+  ;; version of c-init-language-vars will have the old value.  This
+  ;; avoids finding some way to re-evaluate the defun for
+  ;; c-init-language-vars.
+  (defun krb5-c-monkey-patch-caselabel ()
+    (setq c-nonlabel-token-key (c-lang-const c-nonlabel-token-key)))
+  (add-hook 'c-mode-common-hook 'krb5-c-monkey-patch-caselabel))