[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[nongnu] elpa/subed 0968590 344/389: Fix separator search in subed-srt--
From: |
ELPA Syncer |
Subject: |
[nongnu] elpa/subed 0968590 344/389: Fix separator search in subed-srt--validate |
Date: |
Fri, 3 Dec 2021 11:00:54 -0500 (EST) |
branch: elpa/subed
commit 0968590ad30f2699afa0efa7b5648ffadaacd886
Author: Jun Inoue <jun.lambda@gmail.com>
Commit: Jun Inoue <jun.lambda@gmail.com>
Fix separator search in subed-srt--validate
Before this fix, subed-srt--validate was not scanning most of the
file. The reason is that [[^\\']] matches the literal characters [,
^, \, or ', followed by a ]. The intent was probably to reject
end-of-buffer, but that's already handled by the subsequent
looking-at.
---
subed/subed-srt.el | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/subed/subed-srt.el b/subed/subed-srt.el
index d738ea8..3a12a2d 100644
--- a/subed/subed-srt.el
+++ b/subed/subed-srt.el
@@ -513,7 +513,7 @@ scheduled call is canceled and another call is scheduled in
(save-match-data
(let ((orig-point (point)))
(goto-char (point-min))
- (while (and (re-search-forward (format "\\(%s[[^\\']]\\|\\`\\)"
subed-srt--regexp-separator) nil t)
+ (while (and (re-search-forward (format "\\(%s\\|\\`\\)"
subed-srt--regexp-separator) nil t)
(looking-at "[[:alnum:]]"))
(unless (looking-at "^[0-9]+$")
(error "Found invalid subtitle ID: %S" (substring (or
(thing-at-point 'line :no-properties) "\n") 0 -1)))
- [nongnu] elpa/subed e8a68c2 322/389: subed-srt--msecs-to-timestamp: Explain save-match-data, (continued)
- [nongnu] elpa/subed e8a68c2 322/389: subed-srt--msecs-to-timestamp: Explain save-match-data, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 5c39418 323/389: Copy subed-srt--msecs-to-timestamp to subed-common.el, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed ecfec2b 325/389: Move debugging functions to subed-debug.el, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed f69f769 327/389: Set subed-mode-map in subed.el, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed d110f79 329/389: Stop disabling point-to-player sync on file-loaded event, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed dd4dc8c 332/389: Automatically disable point-to-player sync while looping, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 5957cbe 333/389: Set default subed-loop-seconds-after/before to 1, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed e71a64b 339/389: Refactor common code into subed-mpv-play-video, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed daaf480 340/389: subed-vtt: Fix navigation code, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 9642945 346/389: Further fixes to let tests pass, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 0968590 344/389: Fix separator search in subed-srt--validate,
ELPA Syncer <=
- [nongnu] elpa/subed 9e661cc 351/389: New command subed-split-subtitle splits at point with MPV timestamp, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed a63c362 252/389: More work on README, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed b5da89e 254/389: Move defgroup to the top, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 7d5c543 255/389: Add function: subed-srt--init, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 3036e7c 256/389: Remove stale comment, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed df51c44 258/389: Rename test file for common functions, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed c1e05c6 259/389: Remove empty line, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 9f0c939 260/389: subed-for-each-subtitle: Make all arguments mandatory, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 3c97061 264/389: Add subed--init-alist variable to track supported formats, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed f6acfdd 266/389: Re-order function suffixes, ELPA Syncer, 2021/12/03