[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[nongnu] elpa/subed bbd239a 042/389: subed-srt--regenerate-ids: Wrap fun
From: |
ELPA Syncer |
Subject: |
[nongnu] elpa/subed bbd239a 042/389: subed-srt--regenerate-ids: Wrap function body in atomic-change-group |
Date: |
Fri, 3 Dec 2021 10:59:53 -0500 (EST) |
branch: elpa/subed
commit bbd239a0ee26bb9ef003084fe669c04a7fa419c0
Author: Random User <rndusr@posteo.de>
Commit: Random User <rndusr@posteo.de>
subed-srt--regenerate-ids: Wrap function body in atomic-change-group
---
subed/subed-srt.el | 21 ++++++++++++---------
1 file changed, 12 insertions(+), 9 deletions(-)
diff --git a/subed/subed-srt.el b/subed/subed-srt.el
index 01adbcf..6e711e0 100644
--- a/subed/subed-srt.el
+++ b/subed/subed-srt.el
@@ -445,15 +445,18 @@ Return point or nil if there is no previous subtitle."
(defun subed-srt--regenerate-ids ()
"Ensure subtitle IDs start at 1 and are incremented by 1 for
each subtitle."
- (save-match-data
- (save-excursion
- (goto-char (point-min))
- (let ((id 1))
- (while (looking-at "^[0-9]+$")
- (kill-word 1)
- (insert (format "%d" id))
- (setq id (1+ id))
- (subed-srt-forward-subtitle-id))))))
+ (interactive)
+ (atomic-change-group
+ (save-match-data
+ (save-excursion
+ (goto-char (point-min))
+ (kill-word 1)
+ (insert "1")
+ (let ((id 2))
+ (while (subed-srt-forward-subtitle-id)
+ (kill-word 1)
+ (insert (format "%d" id))
+ (setq id (1+ id))))))))
(defun subed-srt-sanitize ()
"Remove surplus newlines and whitespace"
- [nongnu] elpa/subed c895b91 012/389: Add test for sanitizing on empty subtitle texts, (continued)
- [nongnu] elpa/subed c895b91 012/389: Add test for sanitizing on empty subtitle texts, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 6797d10 028/389: Use lexical scoping, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 7939e0a 022/389: Move function to have same order as tests, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 31f8255 016/389: README: Turns out spaces are not the issue, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 27c8141 014/389: Some work on README, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 6b97e24 023/389: Fix test's description, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 7e04e16 013/389: Use C-u ... when inserting subtitles, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 005c0cc 027/389: Remove TODO for C-u N M-k, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed fd737b7 035/389: Use save-excursion in subed--save-excursion, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 0815e10 047/389: Simplify subed-srt--regexp-duration, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed bbd239a 042/389: subed-srt--regenerate-ids: Wrap function body in atomic-change-group,
ELPA Syncer <=
- [nongnu] elpa/subed 8790abf 048/389: Add subed-srt-validate, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 7d76a35 049/389: Simplify subed-srt-move-to-subtitle-text, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 821b259 045/389: subed-srt--regenerate-ids: Make this a public function, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed d3d2013 050/389: Look for stop time only on the relevant line, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 042941c 051/389: Fix docstrings, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 4e50796 052/389: Fix subed-srt-backward-subtitle-id, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 86914b4 053/389: Add more tests for moving to next/previous subtitle ID, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 6391996 056/389: Kill debugging buffer+window in subed-disable-debugging, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 0fe0c82 059/389: When opening debug window, always split horizontally, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 4e31010 062/389: mpv returns nil as "time-pos" in at least one circumstance, ELPA Syncer, 2021/12/03