Guard against nil properties, ensure that worker messages are for the current video, update osc

TheAMM / mpv_thumbnail_script

A Lua script to show preview thumbnails in mpv's OSC seekbar, sans external dependencies

GNU General Public License v3.0

679 stars 68 forks source link

Guard against nil properties, ensure that worker messages are for the current video, update osc #55

Open blankname opened 4 years ago

blankname commented 4 years ago

Should be a more complete fix for the race condition issues in #54.

tonycpsu commented 4 years ago

This seems much more stable than any previous attempt. I did have one time where the OSC went away, but I wasn't connected to a terminal at the time so I couldn't get a traceback or anything. But this PR definitely improves the situation significantly, and I've noticed no problems with it, so I recommend merging.

Czarnodziej commented 3 years ago

No more OSC crashes with these changes.