From: "Óscar Fuentes" <ofv@wanadoo.es>
To: <emacs-devel@gnu.org>
Subject: New function: diff-delete-new-trailing-whitespace
Date: Fri, 25 Jul 2008 05:00:04 +0200 [thread overview]
Message-ID: <d4l2zn0r.fsf@telefonica.net> (raw)
This was discussed past April. Vinicius Jose Latorre suggested good
stylistic improvements for this function, but those are not contemplated
here.
2008-07-25 Óscar Fuentes <ofv@wanadoo.es>
* diff-mode.el (diff-delete-new-trailing-whitespace):
New function.
Index: diff-mode.el
===================================================================
RCS file: /sources/emacs/emacs/lisp/diff-mode.el,v
retrieving revision 1.147
diff -u -u -r1.147 diff-mode.el
--- diff-mode.el 22 Jul 2008 20:55:30 -0000 1.147
+++ diff-mode.el 25 Jul 2008 02:46:45 -0000
@@ -1881,6 +1881,46 @@
;; When there's no more hunks, diff-hunk-next signals an error.
(error nil)))))
+(defun diff-delete-new-trailing-whitespace ()
+ "When on a buffer that contains a diff, inspects the
+differences and removes trailing whitespace (spaces, tabs) from
+the lines modified or introduced by this diff. Shows a message
+with the name of the altered buffers, which are unsaved. If a
+file referenced on the diff has no buffer and needs to be fixed,
+a buffer visiting that file is created."
+ (interactive)
+ (goto-char (point-min))
+ ;; We assume that the diff header has no trailing whitespace.
+ (setq modified-buffers nil)
+ (setq white-positions nil)
+ (while (re-search-forward "^[+!>]" (point-max) t)
+ (save-excursion
+ (destructuring-bind (buf line-offset pos src dst &optional switched)
+ (diff-find-source-location t t)
+ (when line-offset
+ (set-buffer buf)
+ (save-excursion
+ (goto-char (+ (car pos) (cdr src)))
+ (beginning-of-line)
+ (when (re-search-forward "\\([ \t]+\\)$" (line-end-position) t)
+ (when (not (member buf modified-buffers))
+ (push buf modified-buffers))
+ (goto-char (match-end 0))
+ (push (point-marker) white-positions)
+ (goto-char (match-beginning 0))
+ (push (point-marker) white-positions)
+ (push buf white-positions)))))))
+ (while white-positions
+ (save-excursion
+ (set-buffer (pop white-positions))
+ (delete-region (pop white-positions) (pop white-positions))))
+ (if modified-buffers
+ (let ((msg "Deleted new trailing whitespace from:"))
+ (dolist (f modified-buffers)
+ (setq msg (concat msg " `" (buffer-name f) "'")))
+ (message "%s" msg))
+ (message "No fixes needed.")))
+
;; provide the package
(provide 'diff-mode)
next reply other threads:[~2008-07-25 3:00 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-07-25 3:00 Óscar Fuentes [this message]
2008-07-25 3:46 ` New function: diff-delete-new-trailing-whitespace Óscar Fuentes
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=d4l2zn0r.fsf@telefonica.net \
--to=ofv@wanadoo.es \
--cc=emacs-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).