[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[nongnu] elpa/subed b1cb470 134/389: Deduplicate code by adding subed-mo
From: |
ELPA Syncer |
Subject: |
[nongnu] elpa/subed b1cb470 134/389: Deduplicate code by adding subed-move-subtitles function |
Date: |
Fri, 3 Dec 2021 11:00:12 -0500 (EST) |
branch: elpa/subed
commit b1cb470b6b0d677087fb3a6e7b35fe05cddf0cad
Author: Random User <rndusr@posteo.de>
Commit: Random User <rndusr@posteo.de>
Deduplicate code by adding subed-move-subtitles function
---
subed/subed.el | 34 ++++++++++++++++++----------------
1 file changed, 18 insertions(+), 16 deletions(-)
diff --git a/subed/subed.el b/subed/subed.el
index bf3c521..efeda0b 100644
--- a/subed/subed.el
+++ b/subed/subed.el
@@ -175,6 +175,22 @@ Before BODY is run, point is placed on the subtitle's ID."
;;; Moving subtitles
+(defun subed-move-subtitles (msecs &optional beg end)
+ "Move subtitles between BEG and END MSECS milliseconds forward.
+Use a negative value for MSECS to move subtitles backward.
+If END is nil, move all subtitles from BEG to end of buffer.
+If BEG is nil, move only the current subtitle.
+After subtitles are moved is done, replay the first moved
+subtitle if replaying is enabled."
+ (subed--with-subtitle-replay-disabled
+ (subed--for-each-subtitle beg end
+ (subed--adjust-subtitle-start-relative msecs)
+ (subed--adjust-subtitle-stop-relative msecs)))
+ (when (subed-replay-adjusted-subtitle-p)
+ (save-excursion
+ (when beg (goto-char beg))
+ (subed-mpv-jump (subed--subtitle-msecs-start)))))
+
(defun subed-move-subtitle-forward (&optional arg)
"Move subtitle `subed-milliseconds-adjust' forward in time
while preserving its duration, i.e. increase start and stop time
@@ -200,14 +216,7 @@ Example usage:
(msecs (subed--get-milliseconds-adjust arg))
(beg (when (use-region-p) (region-beginning)))
(end (when (use-region-p) (region-end))))
- (subed--with-subtitle-replay-disabled
- (subed--for-each-subtitle beg end
- (subed--adjust-subtitle-start-relative msecs)
- (subed--adjust-subtitle-stop-relative msecs)))
- (when (subed-replay-adjusted-subtitle-p)
- (save-excursion
- (when beg (goto-char beg))
- (subed-mpv-jump (subed--subtitle-msecs-start))))))
+ (subed-move-subtitles msecs beg end)))
(defun subed-move-subtitle-backward (&optional arg)
"Move subtitle `subed-milliseconds-adjust' backward in time
@@ -220,14 +229,7 @@ See `subed-move-subtitle-forward'."
(msecs (* -1 (subed--get-milliseconds-adjust arg)))
(beg (when (use-region-p) (region-beginning)))
(end (when (use-region-p) (region-end))))
- (subed--with-subtitle-replay-disabled
- (subed--for-each-subtitle beg end
- (subed--adjust-subtitle-start-relative msecs)
- (subed--adjust-subtitle-stop-relative msecs)))
- (when (subed-replay-adjusted-subtitle-p)
- (save-excursion
- (when beg (goto-char beg))
- (subed-mpv-jump (subed--subtitle-msecs-start))))))
+ (subed-move-subtitles msecs beg end)))
;;; Replay time-adjusted subtitle
- [nongnu] elpa/subed 1ef2dbb 245/389: Add more tests for inserting subtitles adjacent with restricted space, (continued)
- [nongnu] elpa/subed 1ef2dbb 245/389: Add more tests for inserting subtitles adjacent with restricted space, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 239deaf 246/389: subed-copy-player-pos-to-start/stop-time: Use generic functions, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 0924465 261/389: subed-srt--prepend-subtitle: Fix docstring, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed f62fdb9 263/389: Remove empty line, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed ce6bf04 271/389: Point to subed-mode docstring in "Installation", ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 464d0cb 284/389: Add defcustom: subed-enforce-time-boundaries, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed a1c66c5 116/389: Make if clause easier to grasp, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed f31424e 115/389: Show message when enabling/disable features manually, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed b30293d 120/389: subed--replay-adjusted-subtitle: Fix docstring, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 34caefb 117/389: Explain usage of prefix argument when adjusting/moving, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed b1cb470 134/389: Deduplicate code by adding subed-move-subtitles function,
ELPA Syncer <=
- [nongnu] elpa/subed f7c04d2 167/389: Fix tests for inserting subtitle(s), ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed c1ee129 181/389: Add functions to adjust start/stop timestamps to player's ones, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 9bf3fcc 213/389: Make timestamp adjustment limitations generic, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 6fd0468 193/389: Use (defalias ...) instead of (fset ...), ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed af4d876 211/389: use emacs 25 for e.g. alist-get, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 9bc42c6 192/389: arg instead of (not (eq nil arg)), ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 7b5c5e0 205/389: Add/Fix tests for moving subtitles, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 3b69f02 225/389: Simplify basic test for moving subtitle, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed b6e484f 204/389: Move subtitles in active region as a unit, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 07a3774 218/389: Add generic function subed-subtitle-id-max, ELPA Syncer, 2021/12/03