[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[nongnu] elpa/subed 150a781 084/389: Rename *-move-to-* functions to *-j
From: |
ELPA Syncer |
Subject: |
[nongnu] elpa/subed 150a781 084/389: Rename *-move-to-* functions to *-jump-to-* |
Date: |
Fri, 3 Dec 2021 11:00:02 -0500 (EST) |
branch: elpa/subed
commit 150a78146a9acf1fffd869644de5da651e3273c8
Author: Random User <rndusr@posteo.de>
Commit: Random User <rndusr@posteo.de>
Rename *-move-to-* functions to *-jump-to-*
---
subed/subed-config.el | 4 +-
subed/subed-srt.el | 86 ++++++++---------
subed/subed.el | 12 +--
tests/test-subed-srt.el | 252 ++++++++++++++++++++++++------------------------
4 files changed, 177 insertions(+), 177 deletions(-)
diff --git a/subed/subed-config.el b/subed/subed-config.el
index 2305aa3..3fec134 100644
--- a/subed/subed-config.el
+++ b/subed/subed-config.el
@@ -27,8 +27,8 @@
(let ((subed-mode-map (make-keymap)))
(define-key subed-mode-map (kbd "M-n") 'subed-forward-subtitle-text)
(define-key subed-mode-map (kbd "M-p") 'subed-backward-subtitle-text)
- (define-key subed-mode-map (kbd "C-M-a") 'subed-move-to-subtitle-text)
- (define-key subed-mode-map (kbd "C-M-e") 'subed-move-to-subtitle-end)
+ (define-key subed-mode-map (kbd "C-M-a") 'subed-jump-to-subtitle-text)
+ (define-key subed-mode-map (kbd "C-M-e") 'subed-jump-to-subtitle-end)
(define-key subed-mode-map (kbd "M-[") 'subed-decrease-start-time)
(define-key subed-mode-map (kbd "M-]") 'subed-increase-start-time)
(define-key subed-mode-map (kbd "M-{") 'subed-decrease-stop-time)
diff --git a/subed/subed-srt.el b/subed/subed-srt.el
index d93f738..341501a 100644
--- a/subed/subed-srt.el
+++ b/subed/subed-srt.el
@@ -62,7 +62,7 @@ Return nil if TIME-STRING doesn't match the pattern."
(defun subed-srt--subtitle-id ()
"Return the ID of subtitle at point or nil if there is no ID."
(save-excursion
- (when (subed-srt-move-to-subtitle-id)
+ (when (subed-srt-jump-to-subtitle-id)
(string-to-number (current-word)))))
(defun subed-srt--subtitle-id-at-msecs (msecs)
@@ -95,7 +95,7 @@ after MSECS if there is one and its start time is >= MSECS +
(defun subed-srt--subtitle-msecs-start (&optional sub-id)
"Subtitle start time in milliseconds or nil if it can't be found."
(let ((timestamp (save-excursion
- (when (subed-srt-move-to-subtitle-time-start sub-id)
+ (when (subed-srt-jump-to-subtitle-time-start sub-id)
(buffer-substring (point) (+ (point)
subed-srt--length-timestamp))))))
(when timestamp
(subed-srt--timestamp-to-msecs timestamp))))
@@ -103,7 +103,7 @@ after MSECS if there is one and its start time is >= MSECS +
(defun subed-srt--subtitle-msecs-stop (&optional sub-id)
"Subtitle stop time in milliseconds or nil if it can't be found."
(let ((timestamp (save-excursion
- (when (subed-srt-move-to-subtitle-time-stop sub-id)
+ (when (subed-srt-jump-to-subtitle-time-stop sub-id)
(buffer-substring (point) (+ (point)
subed-srt--length-timestamp))))))
(when timestamp
(subed-srt--timestamp-to-msecs timestamp))))
@@ -111,15 +111,15 @@ after MSECS if there is one and its start time is >=
MSECS +
(defun subed-srt--subtitle-text (&optional sub-id)
"Return subtitle's text."
(or (save-excursion
- (let ((beg (subed-srt-move-to-subtitle-text sub-id))
- (end (subed-srt-move-to-subtitle-end sub-id)))
+ (let ((beg (subed-srt-jump-to-subtitle-text sub-id))
+ (end (subed-srt-jump-to-subtitle-end sub-id)))
(when (and beg end)
(buffer-substring beg end)))) ""))
(defun subed-srt--subtitle-relative-point ()
"Point relative to subtitle's ID or nil if ID can't be found."
(let ((start-point (save-excursion
- (when (subed-srt-move-to-subtitle-id)
+ (when (subed-srt-jump-to-subtitle-id)
(point)))))
(when start-point
(- (point) start-point))))
@@ -127,7 +127,7 @@ after MSECS if there is one and its start time is >= MSECS +
;;; Traversing
-(defun subed-srt-move-to-subtitle-id (&optional sub-id)
+(defun subed-srt-jump-to-subtitle-id (&optional sub-id)
"Move to the ID of a subtitle and return point.
If SUBTITLE-ID is not given, focus the current subtitle's ID.
Return point or nil if no subtitle ID could be found."
@@ -157,59 +157,59 @@ Return point or nil if no subtitle ID could be found."
(when (looking-at "^\\([0-9]+\\)$")
(point))))
-(defun subed-srt-move-to-subtitle-id-at-msecs (msecs)
+(defun subed-srt-jump-to-subtitle-id-at-msecs (msecs)
"Move point to the ID of the subtitle that is playing at MSECS.
Return point or nil if point is still on the same subtitle.
See also `subed-srt--subtitle-id-at-msecs'."
(let ((current-sub-id (subed-srt--subtitle-id))
(target-sub-id (subed-srt--subtitle-id-at-msecs msecs)))
(when (and target-sub-id current-sub-id (not (= target-sub-id
current-sub-id)))
- (subed-srt-move-to-subtitle-id target-sub-id))))
+ (subed-srt-jump-to-subtitle-id target-sub-id))))
-(defun subed-srt-move-to-subtitle-time-start (&optional sub-id)
+(defun subed-srt-jump-to-subtitle-time-start (&optional sub-id)
"Move point to subtitle's start time.
Return point or nil if no start time could be found."
(interactive)
(save-match-data
- (when (subed-srt-move-to-subtitle-id sub-id)
+ (when (subed-srt-jump-to-subtitle-id sub-id)
(forward-line)
(when (looking-at subed-srt--regexp-timestamp)
(point)))))
-(defun subed-srt-move-to-subtitle-time-stop (&optional sub-id)
+(defun subed-srt-jump-to-subtitle-time-stop (&optional sub-id)
"Move point to subtitle's stop time.
Return point or nil if no stop time could be found."
(interactive)
(save-match-data
- (when (subed-srt-move-to-subtitle-id sub-id)
+ (when (subed-srt-jump-to-subtitle-id sub-id)
(forward-line 1)
(re-search-forward " +--> +" (point-at-eol) t)
(when (looking-at subed-srt--regexp-timestamp)
(point)))))
-(defun subed-srt-move-to-subtitle-text (&optional sub-id)
+(defun subed-srt-jump-to-subtitle-text (&optional sub-id)
"Move point on the first character of subtitle's text.
Return point."
(interactive)
- (when (subed-srt-move-to-subtitle-id sub-id)
+ (when (subed-srt-jump-to-subtitle-id sub-id)
(forward-line 2)
(point)))
-(defun subed-srt-move-to-subtitle-text-at-msecs (msecs)
+(defun subed-srt-jump-to-subtitle-text-at-msecs (msecs)
"Move point to the text of the subtitle that is playing at MSECS.
Return point or nil if point is still on the same subtitle.
See also `subed-srt--subtitle-id-at-msecs'."
- (when (subed-srt-move-to-subtitle-id-at-msecs msecs)
- (subed-srt-move-to-subtitle-text)))
+ (when (subed-srt-jump-to-subtitle-id-at-msecs msecs)
+ (subed-srt-jump-to-subtitle-text)))
-(defun subed-srt-move-to-subtitle-end (&optional sub-id)
+(defun subed-srt-jump-to-subtitle-end (&optional sub-id)
"Move point after the last character of the subtitle's text.
Return point or nil if point did not change or if no subtitle end
can be found."
(interactive)
(save-match-data
(let ((orig-point (point)))
- (subed-srt-move-to-subtitle-text sub-id)
+ (subed-srt-jump-to-subtitle-text sub-id)
;; Look for next separator or end of buffer. We can't use
;; `subed-srt--regexp-separator' here because if subtitle text is empty,
;; it may be the only empty line in the separator, i.e. there's only one
@@ -226,7 +226,7 @@ Return point or nil if there is no next subtitle."
(interactive)
(save-match-data
(when (re-search-forward (concat subed-srt--regexp-separator "[0-9]+\n")
nil t)
- (subed-srt-move-to-subtitle-id))))
+ (subed-srt-jump-to-subtitle-id))))
(defun subed-srt-backward-subtitle-id ()
"Move point to previous subtitle's ID.
@@ -234,64 +234,64 @@ Return point or nil if there is no previous subtitle."
(interactive)
(let ((orig-point (point))
(orig-sub-id (subed-srt--subtitle-id)))
- (when (subed-srt-move-to-subtitle-id)
+ (when (subed-srt-jump-to-subtitle-id)
(forward-line -1)
(if (= (subed-srt--subtitle-id) orig-sub-id)
(progn (goto-char orig-point)
nil)
- (subed-srt-move-to-subtitle-id)))))
+ (subed-srt-jump-to-subtitle-id)))))
(defun subed-srt-forward-subtitle-text ()
"Move point to next subtitle's text.
Return point or nil if there is no next subtitle."
(interactive)
(when (subed-srt-forward-subtitle-id)
- (subed-srt-move-to-subtitle-text)))
+ (subed-srt-jump-to-subtitle-text)))
(defun subed-srt-backward-subtitle-text ()
"Move point to previous subtitle's text.
Return point or nil if there is no previous subtitle."
(interactive)
(when (subed-srt-backward-subtitle-id)
- (subed-srt-move-to-subtitle-text)))
+ (subed-srt-jump-to-subtitle-text)))
(defun subed-srt-forward-subtitle-end ()
"Move point to end of next subtitle.
Return point or nil if there is no next subtitle."
(interactive)
(when (subed-srt-forward-subtitle-id)
- (subed-srt-move-to-subtitle-end)))
+ (subed-srt-jump-to-subtitle-end)))
(defun subed-srt-backward-subtitle-end ()
"Move point to end of previous subtitle.
Return point or nil if there is no previous subtitle."
(interactive)
(when (subed-srt-backward-subtitle-id)
- (subed-srt-move-to-subtitle-end)))
+ (subed-srt-jump-to-subtitle-end)))
(defun subed-srt-forward-subtitle-time-start ()
"Move point to next subtitle's start time."
(interactive)
(when (subed-srt-forward-subtitle-id)
- (subed-srt-move-to-subtitle-time-start)))
+ (subed-srt-jump-to-subtitle-time-start)))
(defun subed-srt-backward-subtitle-time-start ()
"Move point to previous subtitle's start time."
(interactive)
(when (subed-srt-backward-subtitle-id)
- (subed-srt-move-to-subtitle-time-start)))
+ (subed-srt-jump-to-subtitle-time-start)))
(defun subed-srt-forward-subtitle-time-stop ()
"Move point to next subtitle's stop time."
(interactive)
(when (subed-srt-forward-subtitle-id)
- (subed-srt-move-to-subtitle-time-stop)))
+ (subed-srt-jump-to-subtitle-time-stop)))
(defun subed-srt-backward-subtitle-time-stop ()
"Move point to previous subtitle's stop time."
(interactive)
(when (subed-srt-backward-subtitle-id)
- (subed-srt-move-to-subtitle-time-stop)))
+ (subed-srt-jump-to-subtitle-time-stop)))
;;; Manipulation
@@ -300,7 +300,7 @@ Return point or nil if there is no previous subtitle."
"Add MSECS milliseconds to start time (use negative value to subtract)."
(let ((msecs-new (+ (subed-srt--subtitle-msecs-start) msecs)))
(save-excursion
- (subed-srt-move-to-subtitle-time-start)
+ (subed-srt-jump-to-subtitle-time-start)
(delete-region (point) (+ (point) subed-srt--length-timestamp))
(insert (subed-srt--msecs-to-timestamp msecs-new)))
(when subed-subtitle-time-adjusted-hook
@@ -311,7 +311,7 @@ Return point or nil if there is no previous subtitle."
"Add MSECS milliseconds to stop time (use negative value to subtract)."
(let ((msecs-new (+ (subed-srt--subtitle-msecs-stop) msecs)))
(save-excursion
- (subed-srt-move-to-subtitle-time-stop)
+ (subed-srt-jump-to-subtitle-time-stop)
(delete-region (point) (+ (point) subed-srt--length-timestamp))
(insert (subed-srt--msecs-to-timestamp msecs-new)))
(when subed-subtitle-time-adjusted-hook
@@ -392,17 +392,17 @@ See also `subed-increase-start-time'."
;; Ensure number-of-subs is positive, now that we figured out
`insert-before'
(number-of-subs (abs number-of-subs)))
(subed-debug "Inserting %s subtitle(s) %s the current" number-of-subs
(if insert-before "before" "after"))
- (subed-srt-move-to-subtitle-id)
+ (subed-srt-jump-to-subtitle-id)
;; Move to the ID of the subtitle we're prepending subtitles to so that
we
;; can do (insert "<new subtitle>")
(if insert-before
- (subed-srt-move-to-subtitle-id)
+ (subed-srt-jump-to-subtitle-id)
(when (and (not (subed-srt-forward-subtitle-id)) ;; Appending after
last subtitle
(> (buffer-size) 0)) ;; Buffer is not empty
;; There is no ID because we're appending to the last subtitle. We
just
;; have to make sure there is a subtitle delimiter ("\n\n") after the
;; last subtitle and point is where the new ID will go.
- (subed-srt-move-to-subtitle-end)
+ (subed-srt-jump-to-subtitle-end)
(forward-line)
(insert "\n")))
;; Insert subtitles
@@ -447,16 +447,16 @@ See also `subed-increase-start-time'."
(forward-line -1)
(kill-whole-line)))
(subed-srt-regenerate-ids)
- (subed-srt-move-to-subtitle-text))))
+ (subed-srt-jump-to-subtitle-text))))
(defun subed-srt-subtitle-kill ()
"Remove subtitle at point."
(interactive)
(let ((beg (save-excursion
- (subed-srt-move-to-subtitle-id)
+ (subed-srt-jump-to-subtitle-id)
(point)))
(end (save-excursion
- (subed-srt-move-to-subtitle-id)
+ (subed-srt-jump-to-subtitle-id)
(when (subed-srt-forward-subtitle-id)
(point)))))
(if (not end)
@@ -464,7 +464,7 @@ See also `subed-increase-start-time'."
(let ((beg (save-excursion
(goto-char beg)
(subed-srt-backward-subtitle-text)
- (subed-srt-move-to-subtitle-end)
+ (subed-srt-jump-to-subtitle-end)
(1+ (point))))
(end (save-excursion
(goto-char (point-max)))))
@@ -483,7 +483,7 @@ each subtitle."
(save-match-data
(save-excursion
(goto-char (point-min))
- (subed-srt-move-to-subtitle-id)
+ (subed-srt-jump-to-subtitle-id)
(unless (string= (current-word) "1")
(kill-word 1)
(insert "1"))
@@ -527,7 +527,7 @@ each subtitle."
;; Two trailing newline if last subtitle text is empty,
;; one trailing newline otherwise
(goto-char (point-max))
- (subed-srt-move-to-subtitle-end)
+ (subed-srt-jump-to-subtitle-end)
(unless (looking-at "\n\\'")
(delete-region (point) (point-max))
(insert "\n"))
@@ -581,7 +581,7 @@ each subtitle."
(lambda () (unless (subed-srt-forward-subtitle-id)
(goto-char (point-max))))
;; endrecfun (move to end of current record/subtitle)
- 'subed-srt-move-to-subtitle-end
+ 'subed-srt-jump-to-subtitle-end
;; startkeyfun (return sort value of current record/subtitle)
'subed-srt--subtitle-msecs-start))
(subed-srt-regenerate-ids)))
diff --git a/subed/subed.el b/subed/subed.el
index 41bf8f0..86e26f3 100644
--- a/subed/subed.el
+++ b/subed/subed.el
@@ -39,10 +39,10 @@
(fset 'subed--subtitle-msecs-stop 'subed-srt--subtitle-msecs-stop)
(fset 'subed--subtitle-relative-point 'subed-srt--subtitle-relative-point)
-(fset 'subed-move-to-subtitle-id 'subed-srt-move-to-subtitle-id)
-(fset 'subed-move-to-subtitle-text-at-msecs
'subed-srt-move-to-subtitle-text-at-msecs)
-(fset 'subed-move-to-subtitle-text 'subed-srt-move-to-subtitle-text)
-(fset 'subed-move-to-subtitle-end 'subed-srt-move-to-subtitle-end)
+(fset 'subed-jump-to-subtitle-id 'subed-srt-jump-to-subtitle-id)
+(fset 'subed-jump-to-subtitle-text-at-msecs
'subed-srt-jump-to-subtitle-text-at-msecs)
+(fset 'subed-jump-to-subtitle-text 'subed-srt-jump-to-subtitle-text)
+(fset 'subed-jump-to-subtitle-end 'subed-srt-jump-to-subtitle-end)
(fset 'subed-forward-subtitle-id 'subed-srt-forward-subtitle-id)
(fset 'subed-backward-subtitle-id 'subed-srt-backward-subtitle-id)
@@ -179,7 +179,7 @@ playing subtitle."
(defun subed--sync-point-to-player (msecs)
"Move point to currently playing subtitle."
- (when (subed-move-to-subtitle-text-at-msecs msecs)
+ (when (subed-jump-to-subtitle-text-at-msecs msecs)
(subed-debug "Synchronized point to playback position: %s -> #%s"
(subed-srt--msecs-to-timestamp msecs) (subed--subtitle-id))
;; post-command-hook is not triggered because we didn't move interactively.
@@ -363,7 +363,7 @@ don't change."
`(let ((sub-id (subed--subtitle-id))
(sub-pos (subed--subtitle-relative-point)))
(progn ,@body)
- (subed-move-to-subtitle-id sub-id)
+ (subed-jump-to-subtitle-id sub-id)
;; Subtitle text may have changed and we may not be able to move to the
;; exact original position
(condition-case nil
diff --git a/tests/test-subed-srt.el b/tests/test-subed-srt.el
index 1fe3cf7..94f8ea6 100644
--- a/tests/test-subed-srt.el
+++ b/tests/test-subed-srt.el
@@ -20,7 +20,7 @@ Baz.
(it "returns the subtitle ID if it can be found."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text 2)
+ (subed-srt-jump-to-subtitle-text 2)
(expect (subed-srt--subtitle-id) :to-equal 2)))
(it "returns nil if no subtitle ID can be found."
(with-temp-buffer
@@ -34,7 +34,7 @@ Baz.
(let ((msecs
(subed-srt--subtitle-msecs-start target-id)))
(cl-loop for outset-id from 1 to 3 do
(progn
- (subed-srt-move-to-subtitle-id
outset-id)
+ (subed-srt-jump-to-subtitle-id
outset-id)
(expect
(subed-srt--subtitle-id-at-msecs msecs) :to-equal target-id)))))))
(it "returns subtitle ID if time is equal to stop time."
(with-temp-buffer
@@ -43,7 +43,7 @@ Baz.
(let ((msecs
(subed-srt--subtitle-msecs-stop target-id)))
(cl-loop for outset-id from 1 to 3 do
(progn
- (subed-srt-move-to-subtitle-id
outset-id)
+ (subed-srt-jump-to-subtitle-id
outset-id)
(expect
(subed-srt--subtitle-id-at-msecs msecs) :to-equal target-id)))))))
(it "returns subtitle ID if time is between start and stop
time."
(with-temp-buffer
@@ -52,7 +52,7 @@ Baz.
(let ((msecs (+ 1
(subed-srt--subtitle-msecs-start target-id))))
(cl-loop for outset-id from 1 to 3 do
(progn
- (subed-srt-move-to-subtitle-id
outset-id)
+ (subed-srt-jump-to-subtitle-id
outset-id)
(expect
(subed-srt--subtitle-id-at-msecs msecs) :to-equal target-id)))))))
(it "returns first subtitle ID if time is before the first
subtitle's start time."
(with-temp-buffer
@@ -62,7 +62,7 @@ Baz.
(subed-srt--subtitle-msecs-start))
1)))
(cl-loop for outset-id from 1 to 3 do
(progn
- (subed-srt-move-to-subtitle-id
outset-id)
+ (subed-srt-jump-to-subtitle-id
outset-id)
(expect
(subed-srt--subtitle-id-at-msecs msecs) :to-equal 1))))))
(it "returns last subtitle ID if time is after last
subtitle's start time."
(with-temp-buffer
@@ -72,7 +72,7 @@ Baz.
(subed-srt--subtitle-msecs-stop))
1)))
(cl-loop for outset-id from 1 to 3 do
(progn
- (subed-srt-move-to-subtitle-id
outset-id)
+ (subed-srt-jump-to-subtitle-id
outset-id)
(expect
(subed-srt--subtitle-id-at-msecs msecs) :to-equal 3))))))
(it "returns previous subtitle ID when time is between
subtitles."
(with-temp-buffer
@@ -81,19 +81,19 @@ Baz.
(let ((msecs (+
(subed-srt--subtitle-msecs-stop target-id) 1)))
(cl-loop for outset-id from 1 to 3 do
(progn
- (subed-srt-move-to-subtitle-id
outset-id)
+ (subed-srt-jump-to-subtitle-id
outset-id)
(expect
(subed-srt--subtitle-id-at-msecs msecs) :to-equal target-id))))
(let ((msecs (-
(subed-srt--subtitle-msecs-start (+ target-id 1)) 1)))
(cl-loop for outset-id from 1 to 3 do
(progn
- (subed-srt-move-to-subtitle-id
outset-id)
+ (subed-srt-jump-to-subtitle-id
outset-id)
(expect
(subed-srt--subtitle-id-at-msecs msecs) :to-equal target-id)))))))
(it "doesn't fail when start time is invalid."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-id 2)
+ (subed-srt-jump-to-subtitle-id 2)
(let ((msecs (- (subed-srt--subtitle-msecs-start)
1)))
- (subed-srt-move-to-subtitle-time-start)
+ (subed-srt-jump-to-subtitle-time-start)
(forward-char 8) (delete-char 1)
(expect (subed-srt--subtitle-id-at-msecs msecs)
:to-equal 2))))
)
@@ -101,7 +101,7 @@ Baz.
(it "returns the time in milliseconds."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text 2)
+ (subed-srt-jump-to-subtitle-text 2)
(expect (subed-srt--subtitle-msecs-start) :to-equal
(+ (* 2 60000) (* 2 1000) 234))
(expect (subed-srt--subtitle-msecs-stop) :to-equal
(+ (* 2 60000) (* 10 1000) 345))))
(it "returns nil if time can't be found."
@@ -114,31 +114,31 @@ Baz.
(it "and at the beginning with a trailing
newline."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text 1)
+ (subed-srt-jump-to-subtitle-text 1)
(kill-line)
(expect (subed-srt--subtitle-text)
:to-equal "")))
(it "and at the beginning without a trailing
newline."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text 1)
+ (subed-srt-jump-to-subtitle-text 1)
(kill-whole-line)
(expect (subed-srt--subtitle-text)
:to-equal "")))
(it "and in the middle."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text 2)
+ (subed-srt-jump-to-subtitle-text 2)
(kill-line)
(expect (subed-srt--subtitle-text)
:to-equal "")))
(it "and at the end with a trailing newline."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text 3)
+ (subed-srt-jump-to-subtitle-text 3)
(kill-line)
(expect (subed-srt--subtitle-text)
:to-equal "")))
(it "and at the end without a trailing newline."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text 3)
+ (subed-srt-jump-to-subtitle-text 3)
(kill-whole-line)
(expect (subed-srt--subtitle-text)
:to-equal "")))
)
@@ -146,12 +146,12 @@ Baz.
(it "and has no linebreaks."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text 2)
+ (subed-srt-jump-to-subtitle-text 2)
(expect (subed-srt--subtitle-text)
:to-equal "Bar.")))
(it "and has linebreaks."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text 2)
+ (subed-srt-jump-to-subtitle-text 2)
(insert "Bar.\n")
(expect (subed-srt--subtitle-text)
:to-equal "Bar.\nBar.")))
)
@@ -160,7 +160,7 @@ Baz.
(it "returns the relative point if we can find an ID."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-id 2)
+ (subed-srt-jump-to-subtitle-id 2)
(expect (subed-srt--subtitle-relative-point)
:to-equal 0)
(forward-line)
(expect (subed-srt--subtitle-relative-point)
:to-equal 2)
@@ -175,34 +175,34 @@ Baz.
(it "returns nil if we can't find an ID."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-id 1)
+ (subed-srt-jump-to-subtitle-id 1)
(insert "foo")
(expect (subed-srt--subtitle-relative-point)
:to-equal nil)))
)
)
-(describe "Moving"
+(describe "Jumping"
(describe "to current subtitle ID"
(it "returns ID's point when point is already on the ID."
(with-temp-buffer
(insert mock-srt-data)
(goto-char (point-min))
(expect (thing-at-point 'word) :to-equal "1")
- (expect (subed-srt-move-to-subtitle-id) :to-equal 1)
+ (expect (subed-srt-jump-to-subtitle-id) :to-equal 1)
(expect (thing-at-point 'word) :to-equal "1")))
(it "returns ID's point when point is on the duration."
(with-temp-buffer
(insert mock-srt-data)
(search-backward ",234")
(expect (thing-at-point 'word) :to-equal "02")
- (expect (subed-srt-move-to-subtitle-id) :to-equal 39)
+ (expect (subed-srt-jump-to-subtitle-id) :to-equal 39)
(expect (thing-at-point 'word) :to-equal "2")))
(it "returns ID's point when point is on the text."
(with-temp-buffer
(insert mock-srt-data)
(search-backward "Baz.")
(expect (thing-at-point 'word) :to-equal "Baz")
- (expect (subed-srt-move-to-subtitle-id) :to-equal 77)
+ (expect (subed-srt-jump-to-subtitle-id) :to-equal 77)
(expect (thing-at-point 'word) :to-equal "3")))
(it "returns ID's point when point is between subtitles."
(with-temp-buffer
@@ -210,18 +210,18 @@ Baz.
(goto-char (point-min))
(search-forward "Bar.\n")
(expect (thing-at-point 'line) :to-equal "\n")
- (expect (subed-srt-move-to-subtitle-id) :to-equal 39)
+ (expect (subed-srt-jump-to-subtitle-id) :to-equal 39)
(expect (thing-at-point 'word) :to-equal "2")))
(it "returns nil if buffer is empty."
(with-temp-buffer
(expect (buffer-string) :to-equal "")
- (expect (subed-srt-move-to-subtitle-id) :to-equal
nil)))
+ (expect (subed-srt-jump-to-subtitle-id) :to-equal
nil)))
(it "returns ID's point when buffer starts with blank
lines."
(with-temp-buffer
(insert (concat " \n \t \n" mock-srt-data))
(search-backward "Foo.")
(expect (thing-at-point 'line) :to-equal "Foo.\n")
- (expect (subed-srt-move-to-subtitle-id) :to-equal 7)
+ (expect (subed-srt-jump-to-subtitle-id) :to-equal 7)
(expect (thing-at-point 'word) :to-equal "1")))
(it "returns ID's point when subtitles are separated with
blank lines."
(with-temp-buffer
@@ -229,7 +229,7 @@ Baz.
(goto-char (point-min))
(search-forward "Foo.\n")
(insert " \n \t \n")
- (expect (subed-srt-move-to-subtitle-id) :to-equal 1)
+ (expect (subed-srt-jump-to-subtitle-id) :to-equal 1)
(expect (thing-at-point 'word) :to-equal "1")))
)
(describe "to specific subtitle ID"
@@ -237,11 +237,11 @@ Baz.
(with-temp-buffer
(insert mock-srt-data)
(goto-char (point-max))
- (expect (subed-srt-move-to-subtitle-id 2) :to-equal
39)
+ (expect (subed-srt-jump-to-subtitle-id 2) :to-equal
39)
(expect (thing-at-point 'word) :to-equal "2")
- (expect (subed-srt-move-to-subtitle-id 1) :to-equal
1)
+ (expect (subed-srt-jump-to-subtitle-id 1) :to-equal
1)
(expect (thing-at-point 'word) :to-equal "1")
- (expect (subed-srt-move-to-subtitle-id 3) :to-equal
77)
+ (expect (subed-srt-jump-to-subtitle-id 3) :to-equal
77)
(expect (thing-at-point 'word) :to-equal "3")))
(it "returns nil and does not move if wanted ID does not
exists."
(with-temp-buffer
@@ -249,7 +249,7 @@ Baz.
(goto-char (point-min))
(search-forward "Foo")
(let ((stored-point (point)))
- (expect (subed-srt-move-to-subtitle-id 4)
:to-equal nil)
+ (expect (subed-srt-jump-to-subtitle-id 4)
:to-equal nil)
(expect stored-point :to-equal (point)))))
)
(describe "to subtitle ID at specific time"
@@ -258,7 +258,7 @@ Baz.
(insert mock-srt-data)
(goto-char (point-max))
(spy-on 'subed-srt--subtitle-id-at-msecs
:and-return-value (point-min))
- (expect (subed-srt-move-to-subtitle-id-at-msecs
123456) :to-equal (point-min))
+ (expect (subed-srt-jump-to-subtitle-id-at-msecs
123456) :to-equal (point-min))
(expect (point) :to-equal (point-min))
(expect 'subed-srt--subtitle-id-at-msecs
:to-have-been-called-with 123456)
(expect 'subed-srt--subtitle-id-at-msecs
:to-have-been-called-times 1)))
@@ -267,7 +267,7 @@ Baz.
(insert mock-srt-data)
(goto-char 75)
(spy-on 'subed-srt--subtitle-id-at-msecs
:and-return-value 75)
- (expect (subed-srt-move-to-subtitle-id-at-msecs
123456) :to-equal nil)
+ (expect (subed-srt-jump-to-subtitle-id-at-msecs
123456) :to-equal nil)
(expect (point) :to-equal 75)
(expect 'subed-srt--subtitle-id-at-msecs
:to-have-been-called-with 123456)
(expect 'subed-srt--subtitle-id-at-msecs
:to-have-been-called-times 1)))
@@ -277,130 +277,130 @@ Baz.
(with-temp-buffer
(insert mock-srt-data)
(goto-char (point-min))
- (expect (subed-srt-move-to-subtitle-time-start)
:to-equal 3)
+ (expect (subed-srt-jump-to-subtitle-time-start)
:to-equal 3)
(expect (buffer-substring (point) (+ (point)
subed-srt--length-timestamp)) :to-equal "00:01:01,000")
(re-search-forward "\n\n")
- (expect (subed-srt-move-to-subtitle-time-start)
:to-equal 41)
+ (expect (subed-srt-jump-to-subtitle-time-start)
:to-equal 41)
(expect (buffer-substring (point) (+ (point)
subed-srt--length-timestamp)) :to-equal "00:02:02,234")
(re-search-forward "\n\n")
- (expect (subed-srt-move-to-subtitle-time-start)
:to-equal 79)
+ (expect (subed-srt-jump-to-subtitle-time-start)
:to-equal 79)
(expect (buffer-substring (point) (+ (point)
subed-srt--length-timestamp)) :to-equal "00:03:03,456")))
(it "returns nil if movement failed."
(with-temp-buffer
- (expect (subed-srt-move-to-subtitle-time-start)
:to-equal nil)))
+ (expect (subed-srt-jump-to-subtitle-time-start)
:to-equal nil)))
)
(describe "to subtitle stop time"
(it "returns stop time's point if movement was successful."
(with-temp-buffer
(insert mock-srt-data)
(goto-char (point-min))
- (expect (subed-srt-move-to-subtitle-time-stop)
:to-equal 20)
+ (expect (subed-srt-jump-to-subtitle-time-stop)
:to-equal 20)
(expect (buffer-substring (point) (+ (point)
subed-srt--length-timestamp)) :to-equal "00:01:05,123")
(re-search-forward "\n\n")
- (expect (subed-srt-move-to-subtitle-time-stop)
:to-equal 58)
+ (expect (subed-srt-jump-to-subtitle-time-stop)
:to-equal 58)
(expect (buffer-substring (point) (+ (point)
subed-srt--length-timestamp)) :to-equal "00:02:10,345")
(re-search-forward "\n\n")
- (expect (subed-srt-move-to-subtitle-time-stop)
:to-equal 96)
+ (expect (subed-srt-jump-to-subtitle-time-stop)
:to-equal 96)
(expect (buffer-substring (point) (+ (point)
subed-srt--length-timestamp)) :to-equal "00:03:15,567")))
(it "returns nil if movement failed."
(with-temp-buffer
- (expect (subed-srt-move-to-subtitle-time-stop)
:to-equal nil)))
+ (expect (subed-srt-jump-to-subtitle-time-stop)
:to-equal nil)))
)
(describe "to subtitle text"
(it "returns subtitle text's point if movement was
successful."
(with-temp-buffer
(insert mock-srt-data)
(goto-char (point-min))
- (expect (subed-srt-move-to-subtitle-text) :to-equal
33)
+ (expect (subed-srt-jump-to-subtitle-text) :to-equal
33)
(expect (point) :to-equal (save-excursion (goto-char
(point-max)) (search-backward "Foo.")))
(re-search-forward "\n\n")
- (expect (subed-srt-move-to-subtitle-text) :to-equal
71)
+ (expect (subed-srt-jump-to-subtitle-text) :to-equal
71)
(expect (point) :to-equal (save-excursion (goto-char
(point-max)) (search-backward "Bar.")))
(re-search-forward "\n\n")
- (expect (subed-srt-move-to-subtitle-text) :to-equal
109)
+ (expect (subed-srt-jump-to-subtitle-text) :to-equal
109)
(expect (point) :to-equal (save-excursion (goto-char
(point-max)) (search-backward "Baz.")))))
(it "returns nil if movement failed."
(with-temp-buffer
- (expect (subed-srt-move-to-subtitle-time-stop)
:to-equal nil)))
+ (expect (subed-srt-jump-to-subtitle-time-stop)
:to-equal nil)))
)
(describe "to end of subtitle text"
(it "returns point if subtitle end can be found."
(with-temp-buffer
(insert mock-srt-data)
(goto-char (point-min))
- (expect (subed-srt-move-to-subtitle-end) :to-be 37)
+ (expect (subed-srt-jump-to-subtitle-end) :to-be 37)
(expect (looking-back "^Foo.$") :to-be t)
(forward-char 2)
- (expect (subed-srt-move-to-subtitle-end) :to-be 75)
+ (expect (subed-srt-jump-to-subtitle-end) :to-be 75)
(expect (looking-back "^Bar.$") :to-be t)
(forward-char 2)
- (expect (subed-srt-move-to-subtitle-end) :to-be 113)
+ (expect (subed-srt-jump-to-subtitle-end) :to-be 113)
(expect (looking-back "^Baz.$") :to-be t)
(goto-char (point-max))
(backward-char 2)
- (expect (subed-srt-move-to-subtitle-end) :to-be 113)
+ (expect (subed-srt-jump-to-subtitle-end) :to-be 113)
(expect (looking-back "^Baz.$") :to-be t)))
(it "returns nil if subtitle end cannot be found."
(with-temp-buffer
- (expect (subed-srt-move-to-subtitle-end) :to-be
nil)))
+ (expect (subed-srt-jump-to-subtitle-end) :to-be
nil)))
(it "returns nil if point did not move."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text 1)
+ (subed-srt-jump-to-subtitle-text 1)
(kill-line)
- (expect (subed-srt-move-to-subtitle-end) :to-be
nil)))
+ (expect (subed-srt-jump-to-subtitle-end) :to-be
nil)))
(it "works if text is empty with trailing newline."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text 1)
+ (subed-srt-jump-to-subtitle-text 1)
(kill-line)
(backward-char)
- (expect (subed-srt-move-to-subtitle-end) :to-be 33)
+ (expect (subed-srt-jump-to-subtitle-end) :to-be 33)
(expect (looking-at "^$") :to-be t)
- (subed-srt-move-to-subtitle-text 2)
+ (subed-srt-jump-to-subtitle-text 2)
(kill-line)
(backward-char)
- (expect (subed-srt-move-to-subtitle-end) :to-be 67)
+ (expect (subed-srt-jump-to-subtitle-end) :to-be 67)
(expect (looking-at "^$") :to-be t)
- (subed-srt-move-to-subtitle-text 3)
+ (subed-srt-jump-to-subtitle-text 3)
(kill-line)
(backward-char)
- (expect (subed-srt-move-to-subtitle-end) :to-be 101)
+ (expect (subed-srt-jump-to-subtitle-end) :to-be 101)
(expect (looking-at "^$") :to-be t)))
(it "works if text is empty without trailing newline."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text 1)
+ (subed-srt-jump-to-subtitle-text 1)
(kill-whole-line)
- (expect (subed-srt-move-to-subtitle-end) :to-be nil)
+ (expect (subed-srt-jump-to-subtitle-end) :to-be nil)
(expect (looking-at "^$") :to-be t)
(goto-char (point-min))
- (expect (subed-srt-move-to-subtitle-end) :to-be 33)
+ (expect (subed-srt-jump-to-subtitle-end) :to-be 33)
(expect (looking-at "^$") :to-be t)
- (subed-srt-move-to-subtitle-text 2)
+ (subed-srt-jump-to-subtitle-text 2)
(kill-whole-line)
- (expect (subed-srt-move-to-subtitle-end) :to-be nil)
+ (expect (subed-srt-jump-to-subtitle-end) :to-be nil)
(expect (looking-at "^$") :to-be t)
(backward-char)
- (expect (subed-srt-move-to-subtitle-end) :to-be 66)
+ (expect (subed-srt-jump-to-subtitle-end) :to-be 66)
(expect (looking-at "^$") :to-be t)
- (subed-srt-move-to-subtitle-text 3)
+ (subed-srt-jump-to-subtitle-text 3)
(kill-whole-line)
- (expect (subed-srt-move-to-subtitle-end) :to-be nil)
+ (expect (subed-srt-jump-to-subtitle-end) :to-be nil)
(expect (looking-at "^$") :to-be t)
(backward-char)
- (expect (subed-srt-move-to-subtitle-end) :to-be 99)
+ (expect (subed-srt-jump-to-subtitle-end) :to-be 99)
(expect (looking-at "^$") :to-be t)))
)
(describe "to next subtitle ID"
(it "returns point when there is a next subtitle."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-id 1)
+ (subed-srt-jump-to-subtitle-id 1)
(expect (thing-at-point 'word) :to-equal "1")
(expect (subed-srt-forward-subtitle-id) :to-be 39)
(expect (thing-at-point 'word) :to-equal "2")
- (subed-srt-move-to-subtitle-time-start 2)
+ (subed-srt-jump-to-subtitle-time-start 2)
(expect (thing-at-point 'word) :to-equal "00")
(expect (subed-srt-forward-subtitle-id) :to-be 77)
(expect (thing-at-point 'word) :to-equal "3")))
@@ -410,23 +410,23 @@ Baz.
(expect (subed-srt-forward-subtitle-id) :to-be nil))
(with-temp-buffer
(insert (concat mock-srt-data))
- (subed-srt-move-to-subtitle-text 1)
+ (subed-srt-jump-to-subtitle-text 1)
(expect (thing-at-point 'word) :to-equal "Foo")
(expect (subed-srt-forward-subtitle-id) :to-be 39)
(expect (thing-at-point 'word) :to-equal "2")
- (subed-srt-move-to-subtitle-time-stop 2)
+ (subed-srt-jump-to-subtitle-time-stop 2)
(expect (thing-at-point 'word) :to-equal "00")
(expect (subed-srt-forward-subtitle-id) :to-be 77)
(expect (thing-at-point 'word) :to-equal "3"))
(with-temp-buffer
(insert (concat mock-srt-data))
- (subed-srt-move-to-subtitle-text 3)
+ (subed-srt-jump-to-subtitle-text 3)
(expect (thing-at-point 'word) :to-equal "Baz")
(expect (subed-srt-forward-subtitle-id) :to-be nil)
(expect (thing-at-point 'word) :to-equal "Baz"))
(with-temp-buffer
(insert (concat mock-srt-data "\n\n"))
- (subed-srt-move-to-subtitle-time-stop 3)
+ (subed-srt-jump-to-subtitle-time-stop 3)
(expect (thing-at-point 'word) :to-equal "00")
(expect (subed-srt-forward-subtitle-id) :to-be nil)
(expect (thing-at-point 'word) :to-equal "00")))
@@ -435,11 +435,11 @@ Baz.
(it "returns point when there is a previous subtitle."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text 2)
+ (subed-srt-jump-to-subtitle-text 2)
(expect (thing-at-point 'word) :to-equal "Bar")
(expect (subed-srt-backward-subtitle-id) :to-be 1)
(expect (thing-at-point 'word) :to-equal "1")
- (subed-srt-move-to-subtitle-time-stop 3)
+ (subed-srt-jump-to-subtitle-time-stop 3)
(expect (thing-at-point 'word) :to-equal "00")
(expect (subed-srt-backward-subtitle-id) :to-be 39)
(expect (thing-at-point 'word) :to-equal "2")))
@@ -448,19 +448,19 @@ Baz.
(expect (subed-srt-backward-subtitle-id) :to-be nil))
(with-temp-buffer
(insert (concat mock-srt-data))
- (subed-srt-move-to-subtitle-id 1)
+ (subed-srt-jump-to-subtitle-id 1)
(expect (thing-at-point 'word) :to-equal "1")
(expect (subed-srt-backward-subtitle-id) :to-be nil)
(expect (thing-at-point 'word) :to-equal "1"))
(with-temp-buffer
(insert (concat mock-srt-data))
- (subed-srt-move-to-subtitle-text 1)
+ (subed-srt-jump-to-subtitle-text 1)
(expect (thing-at-point 'word) :to-equal "Foo")
(expect (subed-srt-backward-subtitle-id) :to-be nil)
(expect (thing-at-point 'word) :to-equal "Foo"))
(with-temp-buffer
(insert (concat "\n\n\n" mock-srt-data))
- (subed-srt-move-to-subtitle-time-stop 1)
+ (subed-srt-jump-to-subtitle-time-stop 1)
(expect (thing-at-point 'word) :to-equal "00")
(expect (subed-srt-backward-subtitle-id) :to-be nil)
(expect (thing-at-point 'word) :to-equal "00")))
@@ -469,7 +469,7 @@ Baz.
(it "returns point when there is a next subtitle."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-id 1)
+ (subed-srt-jump-to-subtitle-id 1)
(expect (thing-at-point 'word) :to-equal "1")
(expect (subed-srt-forward-subtitle-text) :to-be 71)
(expect (thing-at-point 'word) :to-equal "Bar")))
@@ -477,7 +477,7 @@ Baz.
(with-temp-buffer
(goto-char (point-max))
(insert (concat mock-srt-data "\n\n"))
- (subed-srt-move-to-subtitle-id 3)
+ (subed-srt-jump-to-subtitle-id 3)
(expect (thing-at-point 'word) :to-equal "3")
(expect (subed-srt-forward-subtitle-text) :to-be nil)
(expect (thing-at-point 'word) :to-equal "3")))
@@ -486,7 +486,7 @@ Baz.
(it "returns point when there is a previous subtitle."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-id 3)
+ (subed-srt-jump-to-subtitle-id 3)
(expect (thing-at-point 'word) :to-equal "3")
(expect (subed-srt-backward-subtitle-text) :to-be 71)
(expect (thing-at-point 'word) :to-equal "Bar")))
@@ -502,14 +502,14 @@ Baz.
(it "returns point when there is a next subtitle."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text 2)
+ (subed-srt-jump-to-subtitle-text 2)
(expect (thing-at-point 'word) :to-equal "Bar")
(expect (subed-srt-forward-subtitle-end) :to-be 113)
(expect (thing-at-point 'word) :to-equal nil)))
(it "returns nil and doesn't move when there is no next
subtitle."
(with-temp-buffer
(insert (concat mock-srt-data "\n\n"))
- (subed-srt-move-to-subtitle-text 3)
+ (subed-srt-jump-to-subtitle-text 3)
(end-of-line)
(expect (thing-at-point 'word) :to-equal nil)
(expect (subed-srt-forward-subtitle-end) :to-be nil)
@@ -519,7 +519,7 @@ Baz.
(it "returns point when there is a previous subtitle."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-id 3)
+ (subed-srt-jump-to-subtitle-id 3)
(expect (thing-at-point 'word) :to-equal "3")
(expect (subed-srt-backward-subtitle-text) :to-be 71)
(expect (thing-at-point 'word) :to-equal "Bar")))
@@ -535,14 +535,14 @@ Baz.
(it "returns point when there is a next subtitle."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text 1)
+ (subed-srt-jump-to-subtitle-text 1)
(expect (thing-at-point 'word) :to-equal "Foo")
(expect (subed-srt-forward-subtitle-time-start)
:to-be 41)
(expect (thing-at-point 'word) :to-equal "00")))
(it "returns nil and doesn't move when there is no next
subtitle."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-id 3)
+ (subed-srt-jump-to-subtitle-id 3)
(expect (thing-at-point 'word) :to-equal "3")
(expect (subed-srt-forward-subtitle-time-start)
:to-be nil)
(expect (thing-at-point 'word) :to-equal "3")))
@@ -551,14 +551,14 @@ Baz.
(it "returns point when there is a previous subtitle."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-id 2)
+ (subed-srt-jump-to-subtitle-id 2)
(expect (thing-at-point 'word) :to-equal "2")
(expect (subed-srt-backward-subtitle-time-start)
:to-be 3)
(expect (thing-at-point 'word) :to-equal "00")))
(it "returns nil and doesn't move when there is no
previous subtitle."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-id 1)
+ (subed-srt-jump-to-subtitle-id 1)
(expect (thing-at-point 'word) :to-equal "1")
(expect (subed-srt-backward-subtitle-time-start)
:to-be nil)
(expect (thing-at-point 'word) :to-equal "1")))
@@ -567,14 +567,14 @@ Baz.
(it "returns point when there is a next subtitle."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text 1)
+ (subed-srt-jump-to-subtitle-text 1)
(expect (thing-at-point 'word) :to-equal "Foo")
(expect (subed-srt-forward-subtitle-time-stop)
:to-be 58)
(expect (thing-at-point 'word) :to-equal "00")))
(it "returns nil and doesn't move when there is no next
subtitle."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-id 3)
+ (subed-srt-jump-to-subtitle-id 3)
(expect (thing-at-point 'word) :to-equal "3")
(expect (subed-srt-forward-subtitle-time-stop)
:to-be nil)
(expect (thing-at-point 'word) :to-equal "3")))
@@ -583,14 +583,14 @@ Baz.
(it "returns point when there is a previous subtitle."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-id 3)
+ (subed-srt-jump-to-subtitle-id 3)
(expect (thing-at-point 'word) :to-equal "3")
(expect (subed-srt-backward-subtitle-time-stop)
:to-be 58)
(expect (thing-at-point 'word) :to-equal "00")))
(it "returns nil and doesn't move when there is no
previous subtitle."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-id 1)
+ (subed-srt-jump-to-subtitle-id 1)
(expect (thing-at-point 'word) :to-equal "1")
(expect (subed-srt-backward-subtitle-time-stop)
:to-be nil)
(expect (thing-at-point 'word) :to-equal "1")))
@@ -608,57 +608,57 @@ Baz.
(subed-srt-increase-start-time)
(expect 'foo :to-have-been-called-with 3 183556)
(expect 'foo :to-have-been-called-times 1)
- (subed-srt-move-to-subtitle-id 1)
+ (subed-srt-jump-to-subtitle-id 1)
(subed-srt-increase-stop-time)
(expect 'foo :to-have-been-called-with 1 65223)
(expect 'foo :to-have-been-called-times 2)
- (subed-srt-move-to-subtitle-end 2)
+ (subed-srt-jump-to-subtitle-end 2)
(subed-srt-decrease-start-time)
(expect 'foo :to-have-been-called-with 2 122134)
(expect 'foo :to-have-been-called-times 3)
- (subed-srt-move-to-subtitle-text 3)
+ (subed-srt-jump-to-subtitle-text 3)
(subed-srt-decrease-stop-time)
(expect 'foo :to-have-been-called-with 3 195467)
(expect 'foo :to-have-been-called-times 4))))
(it "adjusts the start/stop time."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-id 1)
+ (subed-srt-jump-to-subtitle-id 1)
(subed-srt-increase-start-time)
- (expect (save-excursion (subed-srt-move-to-subtitle-time-start)
+ (expect (save-excursion (subed-srt-jump-to-subtitle-time-start)
(thing-at-point 'line)) :to-equal
"00:01:01,100 --> 00:01:05,123\n")
(subed-srt-decrease-start-time)
(subed-srt-decrease-start-time)
- (expect (save-excursion (subed-srt-move-to-subtitle-time-start)
+ (expect (save-excursion (subed-srt-jump-to-subtitle-time-start)
(thing-at-point 'line)) :to-equal
"00:01:00,900 --> 00:01:05,123\n")
(subed-srt-increase-stop-time)
(subed-srt-increase-stop-time)
- (expect (save-excursion (subed-srt-move-to-subtitle-time-start)
+ (expect (save-excursion (subed-srt-jump-to-subtitle-time-start)
(thing-at-point 'line)) :to-equal
"00:01:00,900 --> 00:01:05,323\n")
(subed-srt-decrease-stop-time)
- (expect (save-excursion (subed-srt-move-to-subtitle-time-start)
+ (expect (save-excursion (subed-srt-jump-to-subtitle-time-start)
(thing-at-point 'line)) :to-equal
"00:01:00,900 --> 00:01:05,223\n")))
(it "sets the number of seconds when given an argument."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-id 1)
+ (subed-srt-jump-to-subtitle-id 1)
(subed-srt-increase-start-time 200)
(subed-srt-increase-start-time)
(subed-srt-increase-start-time)
- (expect (save-excursion (subed-srt-move-to-subtitle-time-start)
+ (expect (save-excursion (subed-srt-jump-to-subtitle-time-start)
(thing-at-point 'line)) :to-equal
"00:01:01,600 --> 00:01:05,123\n")
(subed-srt-decrease-start-time 50)
(subed-srt-decrease-start-time)
- (expect (save-excursion (subed-srt-move-to-subtitle-time-start)
+ (expect (save-excursion (subed-srt-jump-to-subtitle-time-start)
(thing-at-point 'line)) :to-equal
"00:01:01,500 --> 00:01:05,123\n")
(subed-srt-decrease-stop-time 1000)
(subed-srt-decrease-stop-time)
(subed-srt-decrease-stop-time)
- (expect (save-excursion (subed-srt-move-to-subtitle-time-start)
+ (expect (save-excursion (subed-srt-jump-to-subtitle-time-start)
(thing-at-point 'line)) :to-equal
"00:01:01,500 --> 00:01:02,123\n")
(subed-srt-increase-stop-time 2000)
(subed-srt-increase-stop-time)
- (expect (save-excursion (subed-srt-move-to-subtitle-time-start)
+ (expect (save-excursion (subed-srt-jump-to-subtitle-time-start)
(thing-at-point 'line)) :to-equal
"00:01:01,500 --> 00:01:06,123\n")))
)
@@ -666,7 +666,7 @@ Baz.
(it "removes it when it is the first one."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text 1)
+ (subed-srt-jump-to-subtitle-text 1)
(subed-srt-subtitle-kill)
(expect (buffer-string) :to-equal (concat "2\n"
"00:02:02,234 -->
00:02:10,345\n"
@@ -678,7 +678,7 @@ Baz.
(it "removes it when it is in the middle."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text 2)
+ (subed-srt-jump-to-subtitle-text 2)
(subed-srt-subtitle-kill)
(expect (buffer-string) :to-equal (concat "1\n"
"00:01:01,000 -->
00:01:05,123\n"
@@ -690,7 +690,7 @@ Baz.
(it "removes it when it is the last one."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text 3)
+ (subed-srt-jump-to-subtitle-text 3)
(subed-srt-subtitle-kill)
(expect (buffer-string) :to-equal (concat "1\n"
"00:01:01,000 -->
00:01:05,123\n"
@@ -702,7 +702,7 @@ Baz.
(it "removes the previous subtitle when point is right above an ID."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-id 3)
+ (subed-srt-jump-to-subtitle-id 3)
(backward-char)
(expect (looking-at "^\n3\n") :to-be t)
(subed-srt-subtitle-kill)
@@ -758,7 +758,7 @@ Baz.
(cl-loop for arg in (list '- -1 (list 4)) do
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text
2)
+ (subed-srt-jump-to-subtitle-text
2)
(subed-srt-subtitle-insert arg)
(expect (buffer-string) :to-equal
(concat "1\n"
"00:01:01,000 --> 00:01:05,123\n"
@@ -777,7 +777,7 @@ Baz.
(cl-loop for arg in (list -2 (list 16)) do
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text
2)
+ (subed-srt-jump-to-subtitle-text
2)
(subed-srt-subtitle-insert arg)
(expect (buffer-string) :to-equal
(concat "1\n"
"00:01:01,000 --> 00:01:05,123\n"
@@ -801,7 +801,7 @@ Baz.
(cl-loop for arg in (list nil 1) do
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text
1)
+ (subed-srt-jump-to-subtitle-text
1)
(subed-srt-subtitle-insert arg)
(expect (buffer-string) :to-equal
(concat "1\n"
"00:01:01,000 --> 00:01:05,123\n"
@@ -820,7 +820,7 @@ Baz.
(cl-loop for arg in (list 2) do
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text
1)
+ (subed-srt-jump-to-subtitle-text
1)
(subed-srt-subtitle-insert arg)
(expect (buffer-string) :to-equal
(concat "1\n"
"00:01:01,000 --> 00:01:05,123\n"
@@ -844,7 +844,7 @@ Baz.
(cl-loop for arg in (list '- -1 (list 4)) do
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text
1)
+ (subed-srt-jump-to-subtitle-text
1)
(subed-srt-subtitle-insert arg)
(expect (buffer-string) :to-equal
(concat "1\n"
"00:00:00,100 --> 00:01:00,900\n"
@@ -863,7 +863,7 @@ Baz.
(cl-loop for arg in (list -2 (list 16)) do
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text
1)
+ (subed-srt-jump-to-subtitle-text
1)
(subed-srt-subtitle-insert arg)
(expect (buffer-string) :to-equal
(concat "1\n"
"00:00:00,100 --> 00:00:30,450\n"
@@ -887,7 +887,7 @@ Baz.
(cl-loop for arg in (list nil 1) do
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text
3)
+ (subed-srt-jump-to-subtitle-text
3)
(subed-srt-subtitle-insert arg)
(expect (buffer-string) :to-equal
(concat "1\n"
"00:01:01,000 --> 00:01:05,123\n"
@@ -906,7 +906,7 @@ Baz.
(cl-loop for arg in (list 2) do
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text
3)
+ (subed-srt-jump-to-subtitle-text
3)
(subed-srt-subtitle-insert arg)
(expect (buffer-string) :to-equal
(concat "1\n"
"00:01:01,000 --> 00:01:05,123\n"
@@ -935,7 +935,7 @@ Baz.
(it "reports invalid IDs."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-id 1)
+ (subed-srt-jump-to-subtitle-id 1)
(insert "x")
(expect (subed-srt-validate) :to-throw
'error '("Found invalid subtitle ID: \"x1\""))
@@ -943,7 +943,7 @@ Baz.
(it "reports invalid start time."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-time-start 1)
+ (subed-srt-jump-to-subtitle-time-start 1)
(forward-char 5)
(delete-char 1)
(expect (subed-srt-validate) :to-throw
@@ -952,7 +952,7 @@ Baz.
(it "reports invalid stop time."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-time-stop 1)
+ (subed-srt-jump-to-subtitle-time-stop 1)
(forward-char 10)
(insert "3")
(expect (subed-srt-validate) :to-throw
@@ -961,7 +961,7 @@ Baz.
(it "reports invalid time separator."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-time-stop 1)
+ (subed-srt-jump-to-subtitle-time-stop 1)
(delete-char -1)
(expect (subed-srt-validate) :to-throw
'error '("Found invalid separator between start and
stop time: \"00:01:01,000 -->00:01:05,123\""))
@@ -969,7 +969,7 @@ Baz.
(it "does not report error when last subtitle text is empty."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text 3)
+ (subed-srt-jump-to-subtitle-text 3)
(kill-whole-line)
(forward-char -2)
(subed-srt-validate)
@@ -977,7 +977,7 @@ Baz.
(it "preserves point if there is no error."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text 2)
+ (subed-srt-jump-to-subtitle-text 2)
(forward-char 2)
(subed-srt-validate)
(expect (point) :to-equal 73)))
@@ -1030,7 +1030,7 @@ Baz.
(it "ensures double newline between subtitles if text of previous
subtitle is empty."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text 1)
+ (subed-srt-jump-to-subtitle-text 1)
(kill-whole-line)
(expect (buffer-string) :to-equal (concat "1\n"
"00:01:01,000 -->
00:01:05,123\n"
@@ -1079,7 +1079,7 @@ Baz.
(it "ensures single newline after last subtitle if text is empty."
(with-temp-buffer
(insert mock-srt-data)
- (subed-srt-move-to-subtitle-text 3)
+ (subed-srt-jump-to-subtitle-text 3)
(kill-whole-line)
(expect (buffer-string) :to-equal (concat "1\n"
"00:01:01,000 -->
00:01:05,123\n"
- [nongnu] elpa/subed 33f27c5 063/389: Fix docstring, (continued)
- [nongnu] elpa/subed 33f27c5 063/389: Fix docstring, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 7932aee 066/389: subed-srt-sort: Sanitize before validating, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed dc771c8 067/389: Fix/Add tests, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 23156e3 071/389: subed-srt-subtitle-insert: Make argument optional, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 0e05076 072/389: subed-srt-sanitize: Properly trim trailing newlines, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 49e8e72 073/389: subed-srt-move-to-subtitle-end: Adjust docstring, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 473f041 077/389: Add ogv to video file extensions, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed bd4c44d 081/389: Fix typo, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed bd2c0e4 082/389: Add LICENSE, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 07c6b2b 083/389: Show more helpful error message if mpv isn't available, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 150a781 084/389: Rename *-move-to-* functions to *-jump-to-*,
ELPA Syncer <=
- [nongnu] elpa/subed d2d3532 085/389: Rename get-adjust-time to get-seconds-adjust, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 73d303d 087/389: Adjust some docstrings, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed f76f7ea 092/389: Add keybindings C-M-n and C-M-p to move subtitles, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed d07bde4 093/389: Add moving subtitles to feature list and reword some stuff, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 9ccfef7 096/389: Add subed--right-pad, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed d4b1ddf 103/389: Rename subed-seconds-adjust/move to subed-milliseconds-adjust/move, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed bdcdb0a 100/389: Remove deprecated variables, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 181b9f0 099/389: Test for lacking digits in millisecond, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 03dd3c6 106/389: Make subed-move-subtitle-forward/backward work with region (sort of), ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 0f385d6 110/389: Keep logging debug messages when debugging is disabled, ELPA Syncer, 2021/12/03