Id like to use a diff on my repository to quickly navigate changes, without keeping the buffer open.
Is it possible to make a function that runs 'Jump to source', and deletes the diff buffer?
(Am looking to make a key-binding, but expect this will involve writing a function)
Here are 2 functions I came up with that open a fullscreen diff and close it when jumping to the line.
;; Override pop-up-windows
(defun diff-goto-source-and-close ()
"Go to the source and close the current diff buffer."
(interactive)
(let ((buf (current-buffer))
(pop-up-windows nil))
(diff-goto-source)
(kill-buffer buf)))
(defun vc-root-diff-fullscreen ()
"Open a diff of the repository in the current frame."
(interactive)
(let ((pop-up-windows nil))
(call-interactively 'vc-root-diff)))
Notes:
pop-up-windows needed to be set so the window isn't opened in a split, for both opening the diff and switching to the buffer.vc-root-diff is used to get a diff on the entire project (not just the current file).
(delete-window)or(kill-buffer)? Both of those functions take arguments so that it is not necessary to be in the target while closing the window or killing the buffer. Look at the tail end of your Jump to source function and see if there is a hook, if so, consider using it. [You can useM-x find-functionto locate the source code.] If not, consider creating a new function incorporating either or both of the above-mentioned functions.diff-goto-sourceyou will see(pop-to-buffer buf), which is the source of your pain. Consider changing that toswitch-to-bufferorset-window-buffer. See the doc-strings for each function to see what arguments are applicable.my-diff-goto-sourceand rebind the keyboard shortcut to your new function. Or, you could do something like(require 'diff-mode) (defun diff-goto-source (&optional other-file event) ....)and redefine it in your init.el. If you want to save a split second loading Emacs, consider usingeval-after-loadto redefine the function. If you want to get really fancy, perhaps you could play with advice. Or, you could usecl-letfto redefinepop-to-buffertemporarily. Or, you could modifying the behavior ofpop-to-buffer.