From: Manuel Klimek Date: Thu, 8 Jan 2015 15:29:03 +0000 (+0000) Subject: Improve clang-format.el. X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=687528e67ab58234af9c95d819d237aa56ed5e66;p=clang Improve clang-format.el. - includes header/footer as required by MELPA - correctly handles buffers that are not associated with a file - displays stderr and exit code of clang-format process - customizable via the emacs customization interface and file-/directory- local variables Patch by Johann Klähn. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@225447 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/tools/clang-format/clang-format.el b/tools/clang-format/clang-format.el index 4f6ff3b3f0..e6c0c3c9a4 100644 --- a/tools/clang-format/clang-format.el +++ b/tools/clang-format/clang-format.el @@ -1,58 +1,127 @@ -;;; Clang-format emacs integration for use with C/Objective-C/C++. +;;; clang-format.el --- Format code using clang-format -;; This defines a function clang-format-region that you can bind to a key. -;; A minimal .emacs would contain: +;; Keywords: tools, c +;; Package-Requires: ((json "1.3")) + +;;; Commentary: + +;; This package allows to filter code through clang-format to fix its formatting. +;; clang-format is a tool that formats C/C++/Obj-C code according to a set of +;; style options, see . +;; Note that clang-format 3.4 or newer is required. + +;; clang-format.el is available via MELPA and can be installed via ;; -;; (load "/tools/clang-format/clang-format.el") -;; (global-set-key [C-M-tab] 'clang-format-region) +;; M-x package-install clang-format ;; -;; Depending on your configuration and coding style, you might need to modify -;; 'style' in clang-format, below. +;; when ("melpa" . "http://melpa.org/packages/") is included in +;; `package-archives'. Alternatively, ensure the directory of this +;; file is in your `load-path' and add +;; +;; (require 'clang-format) +;; +;; to your .emacs configuration. + +;; You may also want to bind `clang-format-region' to a key: +;; +;; (global-set-key [C-M-tab] 'clang-format-region) + +;;; Code: (require 'json) -;; *Location of the clang-format binary. If it is on your PATH, a full path name -;; need not be specified. -(defvar clang-format-binary "clang-format") +(defgroup clang-format nil + "Format code using clang-format." + :group 'tools) + +(defcustom clang-format-executable + (or (executable-find "clang-format") + "clang-format") + "Location of the clang-format executable. + +A string containing the name or the full path of the executable." + :group 'clang-format + :type 'string + :risky t) + +(defcustom clang-format-style "file" + "Style argument to pass to clang-format. + +By default clang-format will load the style configuration from +a file named .clang-format located in one of the parent directories +of the buffer." + :group 'clang-format + :type 'string + :safe #'stringp) +(make-variable-buffer-local 'clang-format-style) + +;;;###autoload +(defun clang-format-region (start end &optional style) + "Use clang-format to format the code between START and END according to STYLE. +If called interactively uses the region or the current statement if there +is no active region. If no style is given uses `clang-format-style'." + (interactive + (if (use-region-p) + (list (region-beginning) (region-end)) + (list (point) (point)))) + + (unless style + (setq style clang-format-style)) + + (let* ((temp-file (make-temp-file "clang-format")) + (keep-stderr (list t temp-file)) + (window-starts + (mapcar (lambda (w) (list w (window-start w))) + (get-buffer-window-list))) + (status) + (stderr) + (json)) -(defun clang-format-region () - "Use clang-format to format the currently active region." - (interactive) - (let ((beg (if mark-active - (region-beginning) - (min (line-beginning-position) (1- (point-max))))) - (end (if mark-active - (region-end) - (line-end-position)))) - (clang-format beg end))) - -(defun clang-format-buffer () - "Use clang-format to format the current buffer." - (interactive) - (clang-format (point-min) (point-max))) - -(defun clang-format (begin end) - "Use clang-format to format the code between BEGIN and END." - (let* ((orig-windows (get-buffer-window-list (current-buffer))) - (orig-window-starts (mapcar #'window-start orig-windows)) - (orig-point (point)) - (style "file")) (unwind-protect - (call-process-region (point-min) (point-max) clang-format-binary - t (list t nil) nil - "-offset" (number-to-string (1- begin)) - "-length" (number-to-string (- end begin)) - "-cursor" (number-to-string (1- (point))) - "-assume-filename" (buffer-file-name) - "-style" style) - (goto-char (point-min)) - (let ((json-output (json-read-from-string - (buffer-substring-no-properties - (point-min) (line-beginning-position 2))))) - (delete-region (point-min) (line-beginning-position 2)) - (goto-char (1+ (cdr (assoc 'Cursor json-output)))) - (dotimes (index (length orig-windows)) - (set-window-start (nth index orig-windows) - (nth index orig-window-starts))))))) + (setq status + (call-process-region + (point-min) (point-max) clang-format-executable + 'delete keep-stderr nil + + "-assume-filename" (or (buffer-file-name) "") + "-style" style + "-offset" (number-to-string (1- start)) + "-length" (number-to-string (- end start)) + "-cursor" (number-to-string (1- (point)))) + stderr + (with-temp-buffer + (insert-file-contents temp-file) + (when (> (point-max) (point-min)) + (insert ": ")) + (buffer-substring-no-properties + (point-min) (line-end-position)))) + (delete-file temp-file)) + + (cond + ((stringp status) + (error "(clang-format killed by signal %s%s)" status stderr)) + ((not (equal 0 status)) + (error "(clang-format failed with code %d%s)" status stderr)) + (t (message "(clang-format succeeded%s)" stderr))) + + (goto-char (point-min)) + (setq json (json-read-from-string + (buffer-substring-no-properties + (point-min) (line-end-position)))) + + (delete-region (point-min) (line-beginning-position 2)) + (mapc (lambda (w) (apply #'set-window-start w)) + window-starts) + (goto-char (1+ (cdr (assoc 'Cursor json)))))) + +;;;###autoload +(defun clang-format-buffer (&optional style) + "Use clang-format to format the current buffer according to STYLE." + (interactive) + (clang-format-region (point-min) (point-max) style)) + +;;;###autoload +(defalias 'clang-format 'clang-format-region) (provide 'clang-format) +;;; clang-format.el ends here