[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[nongnu] elpa/subed 433f7b4 003/389: subed-disable-sync-point-to-player-
From: |
ELPA Syncer |
Subject: |
[nongnu] elpa/subed 433f7b4 003/389: subed-disable-sync-point-to-player-temporarily: Remove debug msgs |
Date: |
Fri, 3 Dec 2021 10:59:45 -0500 (EST) |
branch: elpa/subed
commit 433f7b4f9cbc9522ab18d69aec7167b7a40986c7
Author: Random User <rndusr@posteo.de>
Commit: Random User <rndusr@posteo.de>
subed-disable-sync-point-to-player-temporarily: Remove debug msgs
---
subed/subed.el | 16 +++-------------
tests/test-subed.el | 3 +--
2 files changed, 4 insertions(+), 15 deletions(-)
diff --git a/subed/subed.el b/subed/subed.el
index 997dbc0..d1eaedb 100644
--- a/subed/subed.el
+++ b/subed/subed.el
@@ -199,26 +199,16 @@ playing subtitle."
"If point is synced to playback position, temporarily disable
that for `subed-point-sync-delay-after-motion' seconds."
(if subed--point-sync-delay-after-motion-timer
- (progn
- (subed-debug "Cancelling old timer (should be nil: %s)"
(subed-sync-point-to-player-p))
- (cancel-timer subed--point-sync-delay-after-motion-timer))
- (progn
- (setq subed--point-was-synced (subed-sync-point-to-player-p))
- (subed-debug "Remembering whether point was originally synced: %s"
subed--point-was-synced)))
-
+ (cancel-timer subed--point-sync-delay-after-motion-timer)
+ (setq subed--point-was-synced (subed-sync-point-to-player-p)))
(when subed--point-was-synced
- (subed-debug "Temporarily disabling point-to-player syncing (should be t:
%s)"
- (subed-sync-point-to-player-p))
(subed-disable-sync-point-to-player))
-
(when subed--point-was-synced
- (subed-debug "Re-enabling point-to-player syncing in %s seconds"
subed-point-sync-delay-after-motion)
(setq subed--point-sync-delay-after-motion-timer
(run-at-time subed-point-sync-delay-after-motion nil
(lambda ()
(setq subed--point-sync-delay-after-motion-timer nil)
- (subed-enable-sync-point-to-player)
- (subed-debug "Re-added: %s"
subed-mpv-playback-position-hook))))))
+ (subed-enable-sync-point-to-player))))))
;;; Sync player-to-point
diff --git a/tests/test-subed.el b/tests/test-subed.el
index b4b450e..35b0849 100644
--- a/tests/test-subed.el
+++ b/tests/test-subed.el
@@ -81,8 +81,7 @@
subed-point-sync-delay-after-motion nil
(lambda ()
(setq
subed--point-sync-delay-after-motion-timer nil)
- (subed-enable-sync-point-to-player)
- (subed-debug "Re-added: %s"
subed-mpv-playback-position-hook))))
+ (subed-enable-sync-point-to-player))))
(it "cancels previously scheduled re-enabling of
point-to-player syncing."
(subed-disable-sync-point-to-player-temporarily)
(expect 'cancel-timer :not :to-have-been-called-with
"mock timer")
- [nongnu] branch elpa/subed created (now d0dfa1a), ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 6a12f22 002/389: When debugging, don't autoscroll if debug-window is hidden, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed d8bf4ed 001/389: Initial commit, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 2ff368b 004/389: Move subed-subtitle-time-adjusted-hook where it belongs, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 9d54f39 005/389: Be more robust when getting start/stop time, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 433f7b4 003/389: subed-disable-sync-point-to-player-temporarily: Remove debug msgs,
ELPA Syncer <=
- [nongnu] elpa/subed 70e6cea 015/389: README: Test unicode non-breaking space, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed d98325c 011/389: Add tests for getting subtitle start/stop time, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 73419ca 021/389: Make forward/backward movement tests pass, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 99a235a 019/389: Spoon trailing parens, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 46aeb10 020/389: Add more tests for forward/backward movement, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed fd7c2ba 017/389: Add tests for getting relative point within subtitle, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed ac4bc87 025/389: Add more tests for moving to subtitle ID, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 7d4caaa 018/389: Move time adjustment tests, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed f8a7c6a 029/389: Fix test that failed because of lexical scoping, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed b3f2776 032/389: Adjust some test descriptions and apply cosmetics, ELPA Syncer, 2021/12/03