From f0fd6633205fe0b477032d2be5028a204c327e0b Mon Sep 17 00:00:00 2001 From: wm4 Date: Fri, 23 Sep 2016 21:04:20 +0200 Subject: m_config, command: remove some minor code duplication I would have been fine with this, but now I want to add another flag, and the duplication would become more messy than having a strange function for deduplication. --- options/m_config.c | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) (limited to 'options/m_config.c') diff --git a/options/m_config.c b/options/m_config.c index 08465df97f..5c8a99b372 100644 --- a/options/m_config.c +++ b/options/m_config.c @@ -676,6 +676,12 @@ static int handle_set_opt_flags(struct m_config *config, return set ? 2 : 1; } +void m_config_mark_co_flags(struct m_config_option *co, int flags) +{ + if (flags & M_SETOPT_FROM_CMDLINE) + co->is_set_from_cmdline = true; +} + // Unlike m_config_set_option_raw() this does not go through the property layer // via config.option_set_callback. int m_config_set_option_raw_direct(struct m_config *config, @@ -696,9 +702,7 @@ int m_config_set_option_raw_direct(struct m_config *config, m_option_copy(co->opt, co->data, data); - if (flags & M_SETOPT_FROM_CMDLINE) - co->is_set_from_cmdline = true; - + m_config_mark_co_flags(co, flags); m_config_notify_change_co(config, co); return 0; -- cgit v1.2.3