patch 9.0.1262: the did_set_string_option function is too long
Problem: The did_set_string_option function is too long.
Solution: Split off functionality to individual functions. (Yegappan
Lakshmanan, Lewis Russell, closes #11904)
diff --git a/src/ops.c b/src/ops.c
index f4059b8..138edc7 100644
--- a/src/ops.c
+++ b/src/ops.c
@@ -3405,10 +3405,13 @@
* Process the 'operatorfunc' option value.
* Returns OK or FAIL.
*/
- int
+ char *
set_operatorfunc_option(void)
{
- return option_set_callback_func(p_opfunc, &opfunc_cb);
+ if (option_set_callback_func(p_opfunc, &opfunc_cb) == FAIL)
+ return e_invalid_argument;
+
+ return NULL;
}
#if defined(EXITFREE) || defined(PROTO)