From f2e30d0c448b9754d0d4daa901b51fbbf4c30747 Mon Sep 17 00:00:00 2001 From: Yegappan Lakshmanan Date: Mon, 30 Jan 2023 13:04:42 +0000 Subject: 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) --- src/gui_xim.c | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) (limited to 'src/gui_xim.c') diff --git a/src/gui_xim.c b/src/gui_xim.c index 163cbdaa0f..86273b5c3d 100644 --- a/src/gui_xim.c +++ b/src/gui_xim.c @@ -73,16 +73,22 @@ xim_log(char *s, ...) static callback_T imaf_cb; // 'imactivatefunc' callback function static callback_T imsf_cb; // 'imstatusfunc' callback function - int + char * set_imactivatefunc_option(void) { - return option_set_callback_func(p_imaf, &imaf_cb); + if (option_set_callback_func(p_imaf, &imaf_cb) == FAIL) + return e_invalid_argument; + + return NULL; } - int + char * set_imstatusfunc_option(void) { - return option_set_callback_func(p_imsf, &imsf_cb); + if (option_set_callback_func(p_imsf, &imsf_cb) == FAIL) + return e_invalid_argument; + + return NULL; } static void -- cgit v1.2.3