[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[nongnu] elpa/subed 4e50796 052/389: Fix subed-srt-backward-subtitle-id
From: |
ELPA Syncer |
Subject: |
[nongnu] elpa/subed 4e50796 052/389: Fix subed-srt-backward-subtitle-id |
Date: |
Fri, 3 Dec 2021 10:59:55 -0500 (EST) |
branch: elpa/subed
commit 4e507962eb02f4422e02d402d57b08cbf746d264
Author: Random User <rndusr@posteo.de>
Commit: Random User <rndusr@posteo.de>
Fix subed-srt-backward-subtitle-id
Point wasn't preserved if there was no previous subtitle, and it always
ended up
at the current subtitle ID.
---
subed/subed-srt.el | 12 +++++++-----
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/subed/subed-srt.el b/subed/subed-srt.el
index d289185..a117b40 100644
--- a/subed/subed-srt.el
+++ b/subed/subed-srt.el
@@ -229,13 +229,15 @@ Return point or nil if there is no next subtitle."
(defun subed-srt-backward-subtitle-id ()
"Move point to previous subtitle's ID.
-Return point or nil if point didn't change (e.g. if called on the
-first subtitle)."
+Return point or nil if there is no previous subtitle."
(interactive)
- (when (subed-srt-move-to-subtitle-id)
- (let ((orig-point (point)))
+ (let ((orig-point (point))
+ (orig-sub-id (subed-srt--subtitle-id)))
+ (when (subed-srt-move-to-subtitle-id)
(forward-line -1)
- (unless (= (point) orig-point)
+ (if (= (subed-srt--subtitle-id) orig-sub-id)
+ (progn (goto-char orig-point)
+ nil)
(subed-srt-move-to-subtitle-id)))))
(defun subed-srt-forward-subtitle-text ()
- [nongnu] elpa/subed 7e04e16 013/389: Use C-u ... when inserting subtitles, (continued)
- [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, 2021/12/03
- [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 <=
- [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
- [nongnu] elpa/subed 1c8e0d1 064/389: Sanitize separator between start and stop time, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 3c1a1b4 065/389: Don't validate anything if buffer is empty, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed ce14b0d 074/389: Adjust comment, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 8fa0f6c 078/389: Remove debugging messages, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 96e7fcf 086/389: Add subed--get-seconds-move, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 189e5aa 088/389: Make more functions available format-agnosticly, ELPA Syncer, 2021/12/03