diff options
| author | wm4 <wm4@nowhere> | 2016-08-30 23:45:58 +0200 |
|---|---|---|
| committer | wm4 <wm4@nowhere> | 2016-08-30 23:45:58 +0200 |
| commit | 3bb134969eb624e859a546fd7142fe2eae076346 (patch) | |
| tree | e2a6ae7887661c420252c8eeae7ecbc12caa4a94 /options/m_config.c | |
| parent | 6f6d2eb7705db6e315f7df72db8f81ef7e7d3f35 (diff) | |
| download | mpv-3bb134969eb624e859a546fd7142fe2eae076346.tar.bz2 mpv-3bb134969eb624e859a546fd7142fe2eae076346.tar.xz | |
m_option: remove M_OPT_TYPE_DYNAMIC flag
It's actually redundant with whether m_option_type.free is set. Some
option types were flagged inconsistently. Its only use was for running
an additional sanity check without any real functionality.
Diffstat (limited to 'options/m_config.c')
| -rw-r--r-- | options/m_config.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/options/m_config.c b/options/m_config.c index 106a8f20ee..d2fe3fdcfa 100644 --- a/options/m_config.c +++ b/options/m_config.c @@ -424,7 +424,7 @@ static void m_config_add_option(struct m_config *config, } else { // Initialize options if (co.data && co.default_data) { - if (arg->type->flags & M_OPT_TYPE_DYNAMIC) { + if (arg->type->free) { // Would leak memory by overwriting *co.data repeatedly. for (int i = 0; i < config->num_opts; i++) { if (co.data == config->opts[i].data) |