[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

address@hidden: diff-find-file-name and /dev/null]

From: Richard Stallman
Subject: address@hidden: diff-find-file-name and /dev/null]
Date: Sat, 18 Aug 2007 20:44:48 -0400

Would someone please install this if it's correct, and then ack?

------- Start of forwarded message -------
X-Spam-Status: No, score=0.0 required=5.0 tests=UNPARSEABLE_RELAY 
        autolearn=failed version=3.1.0
To: address@hidden
From: Nikolaj Schumacher <address@hidden>
Date: Sat, 18 Aug 2007 00:53:23 +0200
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="=-=-="
Subject: diff-find-file-name and /dev/null

- --=-=-=


using diff files (from git) that create new files like this:

- --- /dev/null
+++ b/foo/bar.c

I noticed that `diff-goto-source' opens /dev/null, which is usually not
desirable.  I use the attached small change to get the actual file.

Nikolaj Schumacher

- --=-=-=
Content-Type: text/x-patch
Content-Disposition: inline; filename=emacs-diff-find-file-name.patch
Content-Description: patch

- --- lisp/diff-mode.el 2007-08-17 06:03:27.000000000 +0200
+++ lisp/diff-mode.el   2007-08-18 00:48:25.000000000 +0200
@@ -667,6 +667,7 @@
       (or (ignore-errors (diff-beginning-of-file))
          (re-search-forward diff-file-header-re nil t)))
     (let ((fs (diff-hunk-file-names old)))
+      (setq fs (delete "/dev/null" fs))
       (if prefix (setq fs (mapcar (lambda (f) (concat prefix f)) fs)))
        ;; use any previously used preference

- --=-=-=
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

bug-gnu-emacs mailing list

- --=-=-=--
------- End of forwarded message -------

reply via email to

[Prev in Thread] Current Thread [Next in Thread]