unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
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)




             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).