[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[nongnu] elpa/subed f2f11ca 305/389: subed-mpv-find-video: Gracefully ha
From: |
ELPA Syncer |
Subject: |
[nongnu] elpa/subed f2f11ca 305/389: subed-mpv-find-video: Gracefully handle new file |
Date: |
Fri, 3 Dec 2021 11:00:46 -0500 (EST) |
branch: elpa/subed
commit f2f11ca123076415305ecd8e5fff68b51070e9bf
Author: Random User <rndusr@posteo.de>
Commit: Random User <rndusr@posteo.de>
subed-mpv-find-video: Gracefully handle new file
If the subtitle file we're editing doesn't exist yet, but the video file
does,
we must add the subtitles after the first save.
---
subed/subed-mpv.el | 11 ++++++++++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/subed/subed-mpv.el b/subed/subed-mpv.el
index 8eff234..632c844 100644
--- a/subed/subed-mpv.el
+++ b/subed/subed-mpv.el
@@ -348,7 +348,16 @@ Video files are expected to have any of the extensions
listed in
(subed-debug "Opening video file: %s" filepath)
(subed-mpv--client-connect subed-mpv--retry-delays)
(subed-mpv--client-send `(loadfile ,filepath replace))
- (subed-mpv--client-send `(sub-add ,(buffer-file-name) select))
+ (if (file-exists-p (buffer-file-name))
+ (subed-mpv--client-send `(sub-add ,(buffer-file-name) select))
+ ;; mpv won't add the subtitles if the file doesn't exist yet, so we add
+ ;; it via after-save-hook. We don't want to add the subtitles on every
+ ;; save, so we remove and undefine ourselves after the first save.
+ (defun subed-mpv--sub-add-after-save ()
+ (subed-mpv--client-send `(sub-add ,(buffer-file-name) select))
+ (remove-hook 'after-save-hook #'subed-mpv--sub-add-after-save :local)
+ (fmakunbound 'subed-mpv--sub-add-after-save))
+ (add-hook 'after-save-hook #'subed-mpv--sub-add-after-save :append
:local))
(subed-mpv--client-send `(observe_property 1 time-pos))
(subed-mpv-playback-speed subed-playback-speed-while-not-typing))))
- [nongnu] elpa/subed e3d4f46 283/389: Reword test description, (continued)
- [nongnu] elpa/subed e3d4f46 283/389: Reword test description, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed fcc1951 288/389: Add subed-mpv-file-loaded-hook, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed d87e2cd 290/389: Add save-place-local-mode to example config, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed e7ae26e 292/389: Add target: clean, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 73e11d4 291/389: Define subed-mode as derived mode, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed df96747 297/389: Rename subed-mpv--handle-event -> subed-mpv--client-handle-event, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 4ef1646 298/389: subed-mpv-find-video: Kill previous mpv instance if existing, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 0872ab7 299/389: Remove "subed-mode enabled" message, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 6215cb1 300/389: Add test-compile target, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 9078a46 303/389: subed-srt--validate: Don't complain if buffer contains only whitespace, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed f2f11ca 305/389: subed-mpv-find-video: Gracefully handle new file,
ELPA Syncer <=
- [nongnu] elpa/subed fabc7dc 306/389: Move function around, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed aecaf3d 309/389: Better support for fill-paragraph, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed d34bdec 310/389: Fix mismatched parens in example config, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 66889a1 311/389: Fix test: Renumbering does not modify the kill-ring, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed e1fd29b 312/389: subed-srt--regenerate-ids: Do not modify empty buffer, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed ca015d4 313/389: Fix one compile warning, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 317c411 314/389: Remove declarations, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 72020af 315/389: Use long options, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed be43268 316/389: Fix test-compile target, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 3753706 318/389: Fix warning about missing function: subed-mpv--sub-add-after-save, ELPA Syncer, 2021/12/03