diff options
| author | ZyX <kp-pav@yandex.ru> | 2016-06-24 17:23:29 +0300 |
|---|---|---|
| committer | ZyX <kp-pav@yandex.ru> | 2016-06-24 17:23:29 +0300 |
| commit | 7900e38a8fd9bfef841ea24af42b817509406da8 (patch) | |
| tree | 6f30faab99adfdb9e614bf0c3ee12ae28401226f | |
| parent | 458a4d04447e57c198210fb8f743a261e13b9855 (diff) | |
| download | rneovim-7900e38a8fd9bfef841ea24af42b817509406da8.tar.gz rneovim-7900e38a8fd9bfef841ea24af42b817509406da8.tar.bz2 rneovim-7900e38a8fd9bfef841ea24af42b817509406da8.zip | |
option: Make all pointers in set_string_option constant
| -rw-r--r-- | src/nvim/option.c | 14 |
1 files changed, 5 insertions, 9 deletions
diff --git a/src/nvim/option.c b/src/nvim/option.c index c1d040afeb..020a119fd3 100644 --- a/src/nvim/option.c +++ b/src/nvim/option.c @@ -2340,15 +2340,11 @@ static char *set_string_option(const int opt_idx, const char *const value, char *const oldval = *varp; *varp = s; - char *saved_oldval = NULL; - if (!starting) { - saved_oldval = xstrdup(oldval); - } + char *const saved_oldval = (starting ? NULL : xstrdup(oldval)); - char *r = NULL; - if ((r = (char *)did_set_string_option(opt_idx, (char_u **)varp, (int)true, - (char_u *)oldval, NULL, opt_flags)) - == NULL) { + char *const r = (char *)did_set_string_option( + opt_idx, (char_u **)varp, (int)true, (char_u *)oldval, NULL, opt_flags); + if (r == NULL) { did_set_option(opt_idx, opt_flags, true); } @@ -2357,7 +2353,7 @@ static char *set_string_option(const int opt_idx, const char *const value, char buf_type[7]; vim_snprintf(buf_type, ARRAY_SIZE(buf_type), "%s", (opt_flags & OPT_LOCAL) ? "local" : "global"); - set_vim_var_string(VV_OPTION_NEW, (char *) (*varp), -1); + set_vim_var_string(VV_OPTION_NEW, (char *)(*varp), -1); set_vim_var_string(VV_OPTION_OLD, saved_oldval, -1); set_vim_var_string(VV_OPTION_TYPE, buf_type, -1); apply_autocmds(EVENT_OPTIONSET, |