[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[nongnu] elpa/reformatter b2963f5100 23/81: Narrow to region when replac
From: |
ELPA Syncer |
Subject: |
[nongnu] elpa/reformatter b2963f5100 23/81: Narrow to region when replacing contents with reformatter output |
Date: |
Tue, 5 Sep 2023 04:03:33 -0400 (EDT) |
branch: elpa/reformatter
commit b2963f51009948d5e4885237a148695008d4ccbc
Author: Steve Purcell <steve@sanityinc.com>
Commit: Steve Purcell <steve@sanityinc.com>
Narrow to region when replacing contents with reformatter output
---
reformatter.el | 13 +++++++++----
1 file changed, 9 insertions(+), 4 deletions(-)
diff --git a/reformatter.el b/reformatter.el
index a481fe850a..34baedefdb 100644
--- a/reformatter.el
+++ b/reformatter.el
@@ -180,10 +180,15 @@ DISPLAY-ERRORS, shows a buffer if the formatting fails."
(special-mode))
(if (eq retcode 0)
(progn
- (insert-file-contents out-file nil nil nil t)
- ;; In future this might be made optional, or a
user-provided
- ;; ":after" form could be inserted for execution
- (whitespace-cleanup))
+ (save-restriction
+ ;; This replacement method minimises
+ ;; disruption to marker positions and the
+ ;; undo list
+ (narrow-to-region beg end)
+ (insert-file-contents out-file nil nil nil t)
+ ;; In future this might be made optional, or a
user-provided
+ ;; ":after" form could be inserted for execution
+ (whitespace-cleanup)))
(if display-errors
(display-buffer error-buffer)
(message ,(concat (symbol-name name) " failed: see %s")
(buffer-name error-buffer)))))
- [nongnu] elpa/reformatter 664fa96fd0 24/81: Add/update comments, (continued)
- [nongnu] elpa/reformatter 664fa96fd0 24/81: Add/update comments, ELPA Syncer, 2023/09/05
- [nongnu] elpa/reformatter d0d58c4eef 01/81: Initial commit, ELPA Syncer, 2023/09/05
- [nongnu] elpa/reformatter 9c6c239371 04/81: Generate a separate custom variable for the mode lighter, ELPA Syncer, 2023/09/05
- [nongnu] elpa/reformatter 9478d6ca2d 19/81: Fix -on-save-mode docstring, ELPA Syncer, 2023/09/05
- [nongnu] elpa/reformatter 466740b40a 17/81: Link to example of using reformatter in a user config, ELPA Syncer, 2023/09/05
- [nongnu] elpa/reformatter c684c0b30f 10/81: Add links to usages in the wild, ELPA Syncer, 2023/09/05
- [nongnu] elpa/reformatter 2c85cd76e5 03/81: Always output :keymap and :lighter minor mode arguments, ELPA Syncer, 2023/09/05
- [nongnu] elpa/reformatter 6484d45a87 07/81: Only support string mode lighters, ELPA Syncer, 2023/09/05
- [nongnu] elpa/reformatter 123d192fb0 16/81: Use https for sanityinc.com link, ELPA Syncer, 2023/09/05
- [nongnu] elpa/reformatter 2b2785557a 15/81: Clearer documentation, ELPA Syncer, 2023/09/05
- [nongnu] elpa/reformatter b2963f5100 23/81: Narrow to region when replacing contents with reformatter output,
ELPA Syncer <=
- [nongnu] elpa/reformatter 78eb8bc98a 09/81: Doc clarifications, ELPA Syncer, 2023/09/05
- [nongnu] elpa/reformatter 0da7241ca2 34/81: Actually check for broken replace-buffer-contents, ELPA Syncer, 2023/09/05
- [nongnu] elpa/reformatter 2fd158102f 54/81: Merge pull request #27 from purcell/files-in-place, ELPA Syncer, 2023/09/05
- [nongnu] elpa/reformatter 6c5e7f64c5 41/81: Never use `replace-buffer-contents', ELPA Syncer, 2023/09/05
- [nongnu] elpa/reformatter 7219a0804c 48/81: Support for formatters which succeed with non-zero exit codes, ELPA Syncer, 2023/09/05
- [nongnu] elpa/reformatter 00413b21ec 28/81: Add reminder to use a `defgroup` form, ELPA Syncer, 2023/09/05
- [nongnu] elpa/reformatter dc6278a6b1 49/81: Merge pull request #23 from purcell/non-zero-exit, ELPA Syncer, 2023/09/05
- [nongnu] elpa/reformatter c9450a39c3 32/81: Merge pull request #8 from wbolster/patch-1, ELPA Syncer, 2023/09/05
- [nongnu] elpa/reformatter 48605c92a7 42/81: Add CI, ELPA Syncer, 2023/09/05
- [nongnu] elpa/reformatter e15598a0cc 40/81: Prefer `delete-trailing-whitespace` to the aggressive `whitespace-cleanup`, ELPA Syncer, 2023/09/05