diff options
author | wm4 <wm4@nowhere> | 2012-10-04 17:16:36 +0200 |
---|---|---|
committer | wm4 <wm4@nowhere> | 2012-10-16 07:26:32 +0200 |
commit | 05f4f00e24caf23646a2b551b8e1a1a1abe76de7 (patch) | |
tree | f7f5684385025a9a7dfe22082209ddc266a3059b /sub/sub.h | |
parent | 17f5019b468d5269408b7dae53a24e17426de7d5 (diff) | |
download | mpv-05f4f00e24caf23646a2b551b8e1a1a1abe76de7.tar.bz2 mpv-05f4f00e24caf23646a2b551b8e1a1a1abe76de7.tar.xz |
sub: cleanup: don't pass parameters via global variables
Passing parameters from caller to subtitle renderer was done by
temporarily setting certain members in the osd_state struct (which for
all practical purposes are as good as global variables). This was the
only purpose of these members.
Rather than using such a messy way to pass parameter, put these into a
struct sub_render_params. The struct was already introduced in earlier
commits, and this commit just removes the parameter passing hack.
Diffstat (limited to 'sub/sub.h')
-rw-r--r-- | sub/sub.h | 8 |
1 files changed, 1 insertions, 7 deletions
@@ -67,10 +67,6 @@ struct osd_state { double sub_offset; double vo_sub_pts; - double sub_pts; - struct mp_eosd_res dim; - double normal_scale; - double vsfilter_scale; bool support_rgba; bool render_subs_in_filter; @@ -80,9 +76,7 @@ struct osd_state { char *osd_text; // OSDTYPE_OSD int progbar_type, progbar_value; // OSDTYPE_PROGBAR - // temporary for sub decoders - int bitmap_id; - int bitmap_pos_id; + int switch_sub_id; struct MPOpts *opts; |