diff options
| author | Dmitry Gutov | 2013-12-02 05:03:40 +0200 |
|---|---|---|
| committer | Dmitry Gutov | 2013-12-02 05:03:40 +0200 |
| commit | 3adc9c6dca2c52aa41165f112a4e875267c6315e (patch) | |
| tree | 7bfbc3c54d89c1602dc55838e7c61c95f36a084f | |
| parent | d9dfe8ca078e22a94952cbbe4095b8209ed4f6c4 (diff) | |
| download | emacs-3adc9c6dca2c52aa41165f112a4e875267c6315e.tar.gz emacs-3adc9c6dca2c52aa41165f112a4e875267c6315e.zip | |
Fix minor log-edit-show-files annoyances
* lisp/vc/log-edit.el (log-edit-kill-buffer): Move the use of
`save-selected-window' to `log-edit-hide-buf'. This makes
`log-edit-show-files' idempotent.
(log-edit-show-files): Mark the new window as dedicated.
| -rw-r--r-- | lisp/ChangeLog | 7 | ||||
| -rw-r--r-- | lisp/vc/log-edit.el | 11 |
2 files changed, 13 insertions, 5 deletions
diff --git a/lisp/ChangeLog b/lisp/ChangeLog index 6acc2c7a2d5..b943b3c5fa8 100644 --- a/lisp/ChangeLog +++ b/lisp/ChangeLog | |||
| @@ -1,5 +1,12 @@ | |||
| 1 | 2013-12-02 Dmitry Gutov <dgutov@yandex.ru> | 1 | 2013-12-02 Dmitry Gutov <dgutov@yandex.ru> |
| 2 | 2 | ||
| 3 | * vc/log-edit.el (log-edit-kill-buffer): Move the use of | ||
| 4 | `save-selected-window' to `log-edit-hide-buf'. This makes | ||
| 5 | `log-edit-show-files' idempotent. | ||
| 6 | (log-edit-show-files): Mark the new window as dedicated. | ||
| 7 | |||
| 8 | 2013-12-02 Dmitry Gutov <dgutov@yandex.ru> | ||
| 9 | |||
| 3 | * vc/log-edit.el (log-edit-mode-map): Add binding for | 10 | * vc/log-edit.el (log-edit-mode-map): Add binding for |
| 4 | `log-edit-kill-biffer'. | 11 | `log-edit-kill-biffer'. |
| 5 | (log-edit-hide-buf): Add a FIXME comment. | 12 | (log-edit-hide-buf): Add a FIXME comment. |
diff --git a/lisp/vc/log-edit.el b/lisp/vc/log-edit.el index 3c15d9147b6..0845284d6cc 100644 --- a/lisp/vc/log-edit.el +++ b/lisp/vc/log-edit.el | |||
| @@ -483,9 +483,10 @@ commands (under C-x v for VC, for example). | |||
| 483 | ;; FIXME: Should use something like `quit-windows-on' here, but | 483 | ;; FIXME: Should use something like `quit-windows-on' here, but |
| 484 | ;; that function never deletes this buffer's window because it | 484 | ;; that function never deletes this buffer's window because it |
| 485 | ;; was created using `cvs-pop-to-buffer-same-frame'. | 485 | ;; was created using `cvs-pop-to-buffer-same-frame'. |
| 486 | (let ((win (get-buffer-window buf where))) | 486 | (save-selected-window |
| 487 | (if win (ignore-errors (delete-window win)))) | 487 | (let ((win (get-buffer-window buf where))) |
| 488 | (bury-buffer buf))) | 488 | (if win (ignore-errors (delete-window win)))) |
| 489 | (bury-buffer buf)))) | ||
| 489 | 490 | ||
| 490 | (defun log-edit-add-new-comment (comment) | 491 | (defun log-edit-add-new-comment (comment) |
| 491 | (when (or (ring-empty-p log-edit-comment-ring) | 492 | (when (or (ring-empty-p log-edit-comment-ring) |
| @@ -545,8 +546,7 @@ Also saves its contents in the comment history and hides | |||
| 545 | `log-edit-files-buf'." | 546 | `log-edit-files-buf'." |
| 546 | (interactive) | 547 | (interactive) |
| 547 | (log-edit-add-new-comment (buffer-string)) | 548 | (log-edit-add-new-comment (buffer-string)) |
| 548 | (save-selected-window | 549 | (log-edit-hide-buf) |
| 549 | (log-edit-hide-buf)) | ||
| 550 | (let ((buf (current-buffer))) | 550 | (let ((buf (current-buffer))) |
| 551 | (quit-windows-on buf) | 551 | (quit-windows-on buf) |
| 552 | (kill-buffer buf))) | 552 | (kill-buffer buf))) |
| @@ -604,6 +604,7 @@ Also saves its contents in the comment history and hides | |||
| 604 | (save-selected-window | 604 | (save-selected-window |
| 605 | (cvs-pop-to-buffer-same-frame buf) | 605 | (cvs-pop-to-buffer-same-frame buf) |
| 606 | (shrink-window-if-larger-than-buffer) | 606 | (shrink-window-if-larger-than-buffer) |
| 607 | (set-window-dedicated-p (selected-window) t) | ||
| 607 | (selected-window))))) | 608 | (selected-window))))) |
| 608 | 609 | ||
| 609 | (defun log-edit-beginning-of-line (&optional n) | 610 | (defun log-edit-beginning-of-line (&optional n) |