search

dyphire / mpv-config

本项目为 windows 下 mpv 播放器的配置文件 (This project is the configuration file of mpv player on Windows)
Other
728 stars 35 forks source link

auto-save-state 修复问题 #104

Closed zhongfly closed 4 months ago

zhongfly commented 4 months ago
zhongfly commented 4 months ago

也许应该把can_save的检查放到save()里，更明确，以防之后出错

dyphire commented 4 months ago

删除demuxer-hysteresis-secs

zhongfly commented 4 months ago

删除demuxer-hysteresis-secs

mp.commandv("change-list", "watch-later-options", "remove", "start") 这些命令应该也不用执行了吧

dyphire commented 4 months ago

删除demuxer-hysteresis-secs

mp.commandv("change-list", "watch-later-options", "remove", "start") 这些命令应该也不用执行了吧

需要执行，因为在 watch-later-options 行为变化后只需要在播放完成后移除 start 参数即可清除进度记录，不必再禁用整个 watch-later 功能，这样可以正常记录用户想要保存的其他 watch-later-options 更改

zhongfly commented 4 months ago

有些搞不明白了…有的地方不做兼容性判断，直接删除了整个watch-later；有的做了判断，删除start或整个watch-later。

https://github.com/dyphire/mpv-config/blob/7d1cecca568eb20da913381d68e7c991e959bb3f/scripts/auto-save-state.lua#L55-L58

那我就只加can_save这部分，其他的你改吧

dyphire commented 4 months ago

有些搞不明白了…有的地方不做兼容性判断，直接删除了整个watch-later；有的做了判断，删除start或整个watch-later。

https://github.com/dyphire/mpv-config/blob/7d1cecca568eb20da913381d68e7c991e959bb3f/scripts/auto-save-state.lua#L55-L58

那我就只加can_save这部分，其他的你改吧

这里的处理单纯漏掉了，不怎么关注这个脚本所以没注意到。可以把兼容性处理提取为单独函数调用，或者测试下 eof-reached 事件是不是不必要的