From 7f4a09bb8534dfafd83099d773adf2e33c64e267 Mon Sep 17 00:00:00 2001 From: wm4 Date: Sat, 18 Jan 2014 01:19:20 +0100 Subject: sub: uglify OSD code path with locking Do two things: 1. add locking to struct osd_state 2. make struct osd_state opaque While 1. is somewhat simple, 2. is quite horrible. Lots of code accesses lots of osd_state (and osd_object) members. To make sure everything is accessed synchronously, I prefer making osd_state opaque, even if it means adding pretty dumb accessors. All of this is meant to allow running VO in their own threads. Eventually, VOs will request OSD on their own, which means osd_state will be accessed from foreign threads. --- player/core.h | 2 ++ 1 file changed, 2 insertions(+) (limited to 'player/core.h') diff --git a/player/core.h b/player/core.h index edfdaec2e1..1018ea9df3 100644 --- a/player/core.h +++ b/player/core.h @@ -23,6 +23,7 @@ #include "common/common.h" #include "options/options.h" +#include "sub/osd.h" // definitions used internally by the core player code @@ -158,6 +159,7 @@ typedef struct MPContext { int osd_function; double osd_function_visible; double osd_last_update; + struct osd_progbar_state osd_progbar; struct playlist *playlist; char *filename; // currently playing file -- cgit v1.2.3