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/dec_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/dec_sub.h')
| -rw-r--r-- | sub/dec_sub.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/sub/dec_sub.h b/sub/dec_sub.h index 7ccf513438..442cd870d8 100644 --- a/sub/dec_sub.h +++ b/sub/dec_sub.h @@ -76,7 +76,8 @@ static inline bool is_text_sub(int type) void sub_decode(struct sh_sub *sh, struct osd_state *osd, void *data, int data_len, double pts, double duration); -void sub_get_bitmaps(struct osd_state *osd, struct sub_bitmaps *res); +void sub_get_bitmaps(struct osd_state *osd, struct sub_render_params *params, + struct sub_bitmaps *res); void sub_init(struct sh_sub *sh, struct osd_state *osd); void sub_reset(struct sh_sub *sh, struct osd_state *osd); void sub_switchoff(struct sh_sub *sh, struct osd_state *osd); |