Patch 9.0.1359
Problem: Too many "else if" statements in handling options.
Solution: Add more functions for handling option changes. (Yegappan
Lakshmanan, closes #12060)
Files: src/map.c, src/optiondefs.h, src/optionstr.c,
src/proto/optionstr.pro, src/structs.h,
src/testdir/gen_opt_test.vim, src/testdir/test_options.vim
*** ../vim-9.0.1358/src/map.c 2023-02-20 12:16:33.332269406 +0000
--- src/map.c 2023-02-27 12:44:30.742555348 +0000
***************
*** 3088,3098 ****
}
if (to == NUL)
{
! // TODO: Need to use errbuf argument for this error message
! // and return it.
! semsg(_(e_langmap_matching_character_missing_for_str),
! transchar(from));
! return NULL;
}
if (from >= 256)
--- 3088,3097 ----
}
if (to == NUL)
{
! sprintf(args->os_errbuf,
! _(e_langmap_matching_character_missing_for_str),
! transchar(from));
! return args->os_errbuf;
}
if (from >= 256)
***************
*** 3112,3121 ****
{
if (p[0] != ',')
{
! // TODO: Need to use errbuf argument for this error
! // message and return it.
!
semsg(_(e_langmap_extra_characters_after_semicolon_str), p);
! return NULL;
}
++p;
}
--- 3111,3120 ----
{
if (p[0] != ',')
{
! sprintf(args->os_errbuf,
!
_(e_langmap_extra_characters_after_semicolon_str),
! p);
! return args->os_errbuf;
}
++p;
}
*** ../vim-9.0.1358/src/optiondefs.h 2023-02-25 11:59:28.866874078 +0000
--- src/optiondefs.h 2023-02-27 12:44:30.746555345 +0000
***************
*** 939,945 ****
#endif
(char_u *)0L} SCTX_INIT},
{"filetype", "ft",
P_STRING|P_EXPAND|P_ALLOCED|P_VI_DEF|P_NOGLOB|P_NFNAME,
! (char_u *)&p_ft, PV_FT, NULL,
{(char_u *)"", (char_u *)0L}
SCTX_INIT},
{"fillchars", "fcs", P_STRING|P_VI_DEF|P_RALL|P_ONECOMMA|P_NODUP,
--- 939,946 ----
#endif
(char_u *)0L} SCTX_INIT},
{"filetype", "ft",
P_STRING|P_EXPAND|P_ALLOCED|P_VI_DEF|P_NOGLOB|P_NFNAME,
! (char_u *)&p_ft, PV_FT,
! did_set_filetype_or_syntax,
{(char_u *)"", (char_u *)0L}
SCTX_INIT},
{"fillchars", "fcs", P_STRING|P_VI_DEF|P_RALL|P_ONECOMMA|P_NODUP,
***************
*** 1411,1417 ****
(char_u *)&p_im, PV_NONE, did_set_insertmode,
{(char_u *)FALSE, (char_u *)0L} SCTX_INIT},
{"isfname", "isf", P_STRING|P_VI_DEF|P_COMMA|P_NODUP,
! (char_u *)&p_isf, PV_NONE, NULL,
{
#ifdef BACKSLASH_IN_FILENAME
// Excluded are: & and ^ are special in cmd.exe
--- 1412,1418 ----
(char_u *)&p_im, PV_NONE, did_set_insertmode,
{(char_u *)FALSE, (char_u *)0L} SCTX_INIT},
{"isfname", "isf", P_STRING|P_VI_DEF|P_COMMA|P_NODUP,
! (char_u *)&p_isf, PV_NONE, did_set_isopt,
{
#ifdef BACKSLASH_IN_FILENAME
// Excluded are: & and ^ are special in cmd.exe
***************
*** 1428,1434 ****
#endif
(char_u *)0L} SCTX_INIT},
{"isident", "isi", P_STRING|P_VI_DEF|P_COMMA|P_NODUP,
! (char_u *)&p_isi, PV_NONE, NULL,
{
#if defined(MSWIN)
(char_u *)"@,48-57,_,128-167,224-235",
--- 1429,1435 ----
#endif
(char_u *)0L} SCTX_INIT},
{"isident", "isi", P_STRING|P_VI_DEF|P_COMMA|P_NODUP,
! (char_u *)&p_isi, PV_NONE, did_set_isopt,
{
#if defined(MSWIN)
(char_u *)"@,48-57,_,128-167,224-235",
***************
*** 1437,1443 ****
#endif
(char_u *)0L} SCTX_INIT},
{"iskeyword", "isk", P_STRING|P_ALLOCED|P_VIM|P_COMMA|P_NODUP,
! (char_u *)&p_isk, PV_ISK, NULL,
{
(char_u *)"@,48-57,_",
#if defined(MSWIN)
--- 1438,1444 ----
#endif
(char_u *)0L} SCTX_INIT},
{"iskeyword", "isk", P_STRING|P_ALLOCED|P_VIM|P_COMMA|P_NODUP,
! (char_u *)&p_isk, PV_ISK, did_set_isopt,
{
(char_u *)"@,48-57,_",
#if defined(MSWIN)
***************
*** 1447,1453 ****
#endif
} SCTX_INIT},
{"isprint", "isp", P_STRING|P_VI_DEF|P_RALL|P_COMMA|P_NODUP,
! (char_u *)&p_isp, PV_NONE, NULL,
{
#if defined(MSWIN) || defined(VMS)
(char_u *)"@,~-255",
--- 1448,1454 ----
#endif
} SCTX_INIT},
{"isprint", "isp", P_STRING|P_VI_DEF|P_RALL|P_COMMA|P_NODUP,
! (char_u *)&p_isp, PV_NONE, did_set_isopt,
{
#if defined(MSWIN) || defined(VMS)
(char_u *)"@,~-255",
***************
*** 1469,1475 ****
SCTX_INIT},
{"keymap", "kmp",
P_STRING|P_ALLOCED|P_VI_DEF|P_RBUF|P_RSTAT|P_NFNAME|P_PRI_MKRC,
#ifdef FEAT_KEYMAP
! (char_u *)&p_keymap, PV_KMAP, NULL,
{(char_u *)"", (char_u *)0L}
#else
(char_u *)NULL, PV_NONE, NULL,
--- 1470,1476 ----
SCTX_INIT},
{"keymap", "kmp",
P_STRING|P_ALLOCED|P_VI_DEF|P_RBUF|P_RSTAT|P_NFNAME|P_PRI_MKRC,
#ifdef FEAT_KEYMAP
! (char_u *)&p_keymap, PV_KMAP, did_set_keymap,
{(char_u *)"", (char_u *)0L}
#else
(char_u *)NULL, PV_NONE, NULL,
***************
*** 2095,2101 ****
{(char_u *)FALSE, (char_u *)0L} SCTX_INIT},
{"rightleftcmd", "rlc", P_STRING|P_ALLOCED|P_VI_DEF|P_RWIN,
#ifdef FEAT_RIGHTLEFT
! (char_u *)VAR_WIN, PV_RLC, NULL,
{(char_u *)"search", (char_u *)NULL}
#else
(char_u *)NULL, PV_NONE, NULL,
--- 2096,2102 ----
{(char_u *)FALSE, (char_u *)0L} SCTX_INIT},
{"rightleftcmd", "rlc", P_STRING|P_ALLOCED|P_VI_DEF|P_RWIN,
#ifdef FEAT_RIGHTLEFT
! (char_u *)VAR_WIN, PV_RLC, did_set_rightleftcmd,
{(char_u *)"search", (char_u *)NULL}
#else
(char_u *)NULL, PV_NONE, NULL,
***************
*** 2419,2425 ****
(char_u *)&p_swf, PV_SWF, did_set_swapfile,
{(char_u *)TRUE, (char_u *)0L} SCTX_INIT},
{"swapsync", "sws", P_STRING|P_VI_DEF,
! (char_u *)&p_sws, PV_NONE, NULL,
{(char_u *)"fsync", (char_u *)0L} SCTX_INIT},
{"switchbuf", "swb", P_STRING|P_VI_DEF|P_ONECOMMA|P_NODUP,
(char_u *)&p_swb, PV_NONE, did_set_switchbuf,
--- 2420,2426 ----
(char_u *)&p_swf, PV_SWF, did_set_swapfile,
{(char_u *)TRUE, (char_u *)0L} SCTX_INIT},
{"swapsync", "sws", P_STRING|P_VI_DEF,
! (char_u *)&p_sws, PV_NONE, did_set_swapsync,
{(char_u *)"fsync", (char_u *)0L} SCTX_INIT},
{"switchbuf", "swb", P_STRING|P_VI_DEF|P_ONECOMMA|P_NODUP,
(char_u *)&p_swb, PV_NONE, did_set_switchbuf,
***************
*** 2435,2441 ****
SCTX_INIT},
{"syntax", "syn",
P_STRING|P_ALLOCED|P_VI_DEF|P_NOGLOB|P_NFNAME,
#ifdef FEAT_SYN_HL
! (char_u *)&p_syn, PV_SYN, NULL,
{(char_u *)"", (char_u *)0L}
#else
(char_u *)NULL, PV_NONE, NULL,
--- 2436,2443 ----
SCTX_INIT},
{"syntax", "syn",
P_STRING|P_ALLOCED|P_VI_DEF|P_NOGLOB|P_NFNAME,
#ifdef FEAT_SYN_HL
! (char_u *)&p_syn, PV_SYN,
! did_set_filetype_or_syntax,
{(char_u *)"", (char_u *)0L}
#else
(char_u *)NULL, PV_NONE, NULL,
*** ../vim-9.0.1358/src/optionstr.c 2023-02-25 11:59:28.866874078 +0000
--- src/optionstr.c 2023-02-27 12:44:30.746555345 +0000
***************
*** 97,102 ****
--- 97,103 ----
static char *(p_twt_values[]) = {"winpty", "conpty", "", NULL};
#endif
static char *(p_sloc_values[]) = {"last", "statusline", "tabline", NULL};
+ static char *(p_sws_values[]) = {"fsync", "sync", NULL};
static int check_opt_strings(char_u *val, char **values, int list);
static int opt_strings_flags(char_u *val, char **values, unsigned *flagp, int
list);
***************
*** 761,775 ****
* The 'isident' or the 'iskeyword' or the 'isprint' or the 'isfname' option
is
* changed.
*/
! static char *
! did_set_isopt(int *did_chartab)
{
// 'isident', 'iskeyword', 'isprint or 'isfname' option: refill
g_chartab[]
// If the new option is invalid, use old value.
// 'lisp' option: refill g_chartab[] for '-' char.
if (init_chartab() == FAIL)
{
! *did_chartab = TRUE; // need to restore it below
return e_invalid_argument; // error in value
}
--- 762,776 ----
* The 'isident' or the 'iskeyword' or the 'isprint' or the 'isfname' option
is
* changed.
*/
! char *
! did_set_isopt(optset_T *args)
{
// 'isident', 'iskeyword', 'isprint or 'isfname' option: refill
g_chartab[]
// If the new option is invalid, use old value.
// 'lisp' option: refill g_chartab[] for '-' char.
if (init_chartab() == FAIL)
{
! args->os_restore_chartab = TRUE;// need to restore the chartab.
return e_invalid_argument; // error in value
}
***************
*** 929,934 ****
--- 930,944 ----
}
/*
+ * The 'swapsync' option is changed.
+ */
+ char *
+ did_set_swapsync(optset_T *args UNUSED)
+ {
+ return did_set_opt_strings(p_sws, p_sws_values, FALSE);
+ }
+
+ /*
* The 'switchbuf' option is changed.
*/
char *
***************
*** 1224,1239 ****
}
#endif
! #ifdef FEAT_KEYMAP
/*
* The 'keymap' option is changed.
*/
! static char *
! did_set_keymap(char_u **varp, int opt_flags, int *value_checked)
{
char *errmsg = NULL;
! if (!valid_filetype(*varp))
errmsg = e_invalid_argument;
else
{
--- 1234,1249 ----
}
#endif
! #if defined(FEAT_KEYMAP) || defined(PROTO)
/*
* The 'keymap' option is changed.
*/
! char *
! did_set_keymap(optset_T *args)
{
char *errmsg = NULL;
! if (!valid_filetype(args->os_varp))
errmsg = e_invalid_argument;
else
{
***************
*** 1250,1256 ****
// Since we check the value, there is no need to set P_INSECURE,
// even when the value comes from a modeline.
! *value_checked = TRUE;
}
if (errmsg == NULL)
--- 1260,1266 ----
// Since we check the value, there is no need to set P_INSECURE,
// even when the value comes from a modeline.
! args->os_value_checked = TRUE;
}
if (errmsg == NULL)
***************
*** 1270,1276 ****
if (curbuf->b_p_imsearch == B_IMODE_LMAP)
curbuf->b_p_imsearch = B_IMODE_USE_INSERT;
}
! if ((opt_flags & OPT_LOCAL) == 0)
{
set_iminsert_global();
set_imsearch_global();
--- 1280,1286 ----
if (curbuf->b_p_imsearch == B_IMODE_LMAP)
curbuf->b_p_imsearch = B_IMODE_USE_INSERT;
}
! if ((args->os_flags & OPT_LOCAL) == 0)
{
set_iminsert_global();
set_imsearch_global();
***************
*** 2573,2579 ****
char *
did_set_lispoptions(optset_T *args)
{
! if (*args->os_varp != NUL && STRCMP(args->os_varp, "expr:0") != 0
&& STRCMP(args->os_varp, "expr:1") != 0)
return e_invalid_argument;
--- 2583,2590 ----
char *
did_set_lispoptions(optset_T *args)
{
! if (*args->os_varp != NUL
! && STRCMP(args->os_varp, "expr:0") != 0
&& STRCMP(args->os_varp, "expr:1") != 0)
return e_invalid_argument;
***************
*** 2594,2617 ****
}
#endif
/*
* The 'filetype' or the 'syntax' option is changed.
*/
! static char *
! did_set_filetype_or_syntax(
! char_u **varp,
! char_u *oldval,
! int *value_checked,
! int *value_changed)
{
! if (!valid_filetype(*varp))
return e_invalid_argument;
! *value_changed = STRCMP(oldval, *varp) != 0;
// Since we check the value, there is no need to set P_INSECURE,
// even when the value comes from a modeline.
! *value_checked = TRUE;
return NULL;
}
--- 2605,2640 ----
}
#endif
+ #if defined(FEAT_RIGHTLEFT) || defined(PROTO)
+ /*
+ * The 'rightleftcmd' option is changed.
+ */
+ char *
+ did_set_rightleftcmd(optset_T *args)
+ {
+ // Currently only "search" is a supported value.
+ if (*args->os_varp != NUL && STRCMP(args->os_varp, "search") != 0)
+ return e_invalid_argument;
+
+ return NULL;
+ }
+ #endif
+
/*
* The 'filetype' or the 'syntax' option is changed.
*/
! char *
! did_set_filetype_or_syntax(optset_T *args)
{
! if (!valid_filetype(args->os_varp))
return e_invalid_argument;
! args->os_value_changed =
! STRCMP(args->os_oldval.string, args->os_varp) != 0;
// Since we check the value, there is no need to set P_INSECURE,
// even when the value comes from a modeline.
! args->os_value_checked = TRUE;
return NULL;
}
***************
*** 3008,3014 ****
// need to set P_INSECURE
{
char *errmsg = NULL;
! int did_chartab = FALSE;
char_u **gvarp;
long_u free_oldval = (get_option_flags(opt_idx) & P_ALLOCED);
int value_changed = FALSE;
--- 3031,3037 ----
// need to set P_INSECURE
{
char *errmsg = NULL;
! int restore_chartab = FALSE;
char_u **gvarp;
long_u free_oldval = (get_option_flags(opt_idx) & P_ALLOCED);
int value_changed = FALSE;
***************
*** 3037,3067 ****
args.os_flags = opt_flags;
args.os_oldval.string = oldval;
args.os_newval.string = value;
args.os_errbuf = errbuf;
errmsg = did_set_cb(&args);
#ifdef FEAT_EVAL
! // When processing the '*expr' options (e.g. diffexpr, foldexpr, etc.),
! // the did_set_cb() function may modify '*varp'.
if (errmsg == NULL && is_expr_option(varp, gvarp))
*varp = args.os_varp;
#endif
}
else if (varp == &T_NAME) // 'term'
errmsg = did_set_term(&opt_idx, &free_oldval);
- else if ( varp == &p_isi // 'isident'
- || varp == &(curbuf->b_p_isk) // 'iskeyword'
- || varp == &p_isp // 'isprint'
- || varp == &p_isf) // 'isfname'
- errmsg = did_set_isopt(&did_chartab);
else if ( varp == &p_enc // 'encoding'
|| gvarp == &p_fenc // 'fileencoding'
|| varp == &p_tenc // 'termencoding'
|| gvarp == &p_menc) // 'makeencoding'
errmsg = did_set_encoding(varp, gvarp, opt_flags);
- #ifdef FEAT_KEYMAP
- else if (varp == &curbuf->b_p_keymap) // 'keymap'
- errmsg = did_set_keymap(varp, opt_flags, value_checked);
- #endif
else if ( varp == &p_lcs // global 'listchars'
|| varp == &p_fcs) // global 'fillchars'
errmsg = did_set_global_listfillchars(varp, opt_flags);
--- 3060,3096 ----
args.os_flags = opt_flags;
args.os_oldval.string = oldval;
args.os_newval.string = value;
+ args.os_value_checked = FALSE;
+ args.os_value_changed = FALSE;
+ args.os_restore_chartab = FALSE;
args.os_errbuf = errbuf;
+ // Invoke the option specific callback function to validate and apply
+ // the new option value.
errmsg = did_set_cb(&args);
+
#ifdef FEAT_EVAL
! // The '*expr' option (e.g. diffexpr, foldexpr, etc.), callback
! // functions may modify '*varp'.
if (errmsg == NULL && is_expr_option(varp, gvarp))
*varp = args.os_varp;
#endif
+ // The 'filetype' and 'syntax' option callback functions may change
+ // the os_value_changed field.
+ value_changed = args.os_value_changed;
+ // The 'keymap', 'filetype' and 'syntax' option callback functions
+ // may change the os_value_checked field.
+ *value_checked = args.os_value_checked;
+ // The 'isident', 'iskeyword', 'isprint' and 'isfname' options may
+ // change the character table. On failure, this needs to be restored.
+ restore_chartab = args.os_restore_chartab;
}
else if (varp == &T_NAME) // 'term'
errmsg = did_set_term(&opt_idx, &free_oldval);
else if ( varp == &p_enc // 'encoding'
|| gvarp == &p_fenc // 'fileencoding'
|| varp == &p_tenc // 'termencoding'
|| gvarp == &p_menc) // 'makeencoding'
errmsg = did_set_encoding(varp, gvarp, opt_flags);
else if ( varp == &p_lcs // global 'listchars'
|| varp == &p_fcs) // global 'fillchars'
errmsg = did_set_global_listfillchars(varp, opt_flags);
***************
*** 3072,3085 ****
// terminal options
else if (istermoption_idx(opt_idx) && full_screen)
did_set_term_option(varp, &did_swaptcap);
- else if (gvarp == &p_ft) // 'filetype'
- errmsg = did_set_filetype_or_syntax(varp, oldval, value_checked,
- &value_changed);
- #ifdef FEAT_SYN_HL
- else if (gvarp == &p_syn) // 'syntax'
- errmsg = did_set_filetype_or_syntax(varp, oldval, value_checked,
- &value_changed);
- #endif
// If an error is detected, restore the previous value.
if (errmsg != NULL)
--- 3101,3106 ----
***************
*** 3087,3093 ****
free_string_option(*varp);
*varp = oldval;
// When resetting some values, need to act on it.
! if (did_chartab)
(void)init_chartab();
if (varp == &p_hl)
(void)highlight_changed();
--- 3108,3114 ----
free_string_option(*varp);
*varp = oldval;
// When resetting some values, need to act on it.
! if (restore_chartab)
(void)init_chartab();
if (varp == &p_hl)
(void)highlight_changed();
*** ../vim-9.0.1358/src/proto/optionstr.pro 2023-02-25 11:59:28.866874078
+0000
--- src/proto/optionstr.pro 2023-02-27 12:44:30.746555345 +0000
***************
*** 12,17 ****
--- 12,18 ----
char *did_set_backupcopy(optset_T *args);
char *did_set_backupext_or_patchmode(optset_T *args);
char *did_set_breakindentopt(optset_T *args);
+ char *did_set_isopt(optset_T *args);
char *did_set_helpfile(optset_T *args);
char *did_set_colorcolumn(optset_T *args);
char *did_set_cursorlineopt(optset_T *args);
***************
*** 23,28 ****
--- 24,30 ----
char *did_set_selectmode(optset_T *args);
char *did_set_showcmdloc(optset_T *args);
char *did_set_splitkeep(optset_T *args);
+ char *did_set_swapsync(optset_T *args);
char *did_set_switchbuf(optset_T *args);
char *did_set_sessionoptions(optset_T *args);
char *did_set_viewoptions(optset_T *args);
***************
*** 36,41 ****
--- 38,44 ----
char *did_set_eventignore(optset_T *args);
char *did_set_printencoding(optset_T *args);
char *did_set_imactivatekey(optset_T *args);
+ char *did_set_keymap(optset_T *args);
char *did_set_fileformat(optset_T *args);
char *did_set_fileformats(optset_T *args);
char *did_set_cryptkey(optset_T *args);
***************
*** 96,101 ****
--- 99,106 ----
char *did_set_cinoptions(optset_T *args);
char *did_set_lispoptions(optset_T *args);
char *did_set_renderoptions(optset_T *args);
+ char *did_set_rightleftcmd(optset_T *args);
+ char *did_set_filetype_or_syntax(optset_T *args);
char *did_set_termwinkey(optset_T *args);
char *did_set_termwinsize(optset_T *args);
char *did_set_termwintype(optset_T *args);
*** ../vim-9.0.1358/src/structs.h 2023-02-25 11:59:28.866874078 +0000
--- src/structs.h 2023-02-27 12:44:30.746555345 +0000
***************
*** 4820,4825 ****
--- 4820,4836 ----
// Currently only used for boolean options.
int os_doskip;
+ // Option value was checked to be safe, no need to set P_INSECURE
+ // Used for the 'keymap', 'filetype' and 'syntax' options.
+ int os_value_checked;
+ // Option value changed. Used for the 'filetype' and 'syntax' options.
+ int os_value_changed;
+
+ // Used by the 'isident', 'iskeyword', 'isprint' and 'isfname' options.
+ // Set to TRUE if the character table is modified when processing the
+ // option and need to be restored because of a failure.
+ int os_restore_chartab;
+
// If the value specified for an option is not valid and the error message
// is parameterized, then the "os_errbuf" buffer is used to store the
error
// message (when it is not NULL).
*** ../vim-9.0.1358/src/testdir/gen_opt_test.vim 2023-02-25
11:59:28.866874078 +0000
--- src/testdir/gen_opt_test.vim 2023-02-27 12:44:30.746555345 +0000
***************
*** 130,135 ****
--- 130,136 ----
\ 'printoptions': [['', 'header:0', 'left:10pc,top:5pc'], ['xxx']],
\ 'scrollopt': [['', 'ver', 'ver,hor'], ['xxx']],
\ 'renderoptions': [[''], ['xxx']],
+ \ 'rightleftcmd': [['search'], ['xxx']],
\ 'selection': [['old', 'inclusive'], ['', 'xxx']],
\ 'selectmode': [['', 'mouse', 'key,cmd'], ['xxx']],
\ 'sessionoptions': [['', 'blank', 'help,options,slash'], ['xxx']],
***************
*** 140,145 ****
--- 141,147 ----
\ 'spelloptions': [['', 'camel'], ['xxx']],
\ 'spellsuggest': [['', 'best', 'double,33'], ['xxx']],
\ 'splitkeep': [['cursor', 'screen', 'topline'], ['xxx']],
+ \ 'swapsync': [['', 'sync', 'fsync'], ['xxx']],
\ 'switchbuf': [['', 'useopen', 'split,newtab'], ['xxx']],
\ 'tagcase': [['smart', 'match'], ['', 'xxx', 'smart,match']],
\ 'term': [[], []],
*** ../vim-9.0.1358/src/testdir/test_options.vim 2023-02-23
15:05:18.917676841 +0000
--- src/testdir/test_options.vim 2023-02-27 12:44:30.746555345 +0000
***************
*** 46,51 ****
--- 46,55 ----
set isfname=
call assert_equal("~X", expand("~X"))
set isfname&
+ " Test for setting 'isfname' to an unsupported character
+ let save_isfname = &isfname
+ call assert_fails('exe $"set isfname+={"\u1234"}"', 'E474:')
+ call assert_equal(save_isfname, &isfname)
endfunc
" Test for getting the value of 'pastetoggle'
***************
*** 1548,1553 ****
--- 1552,1558 ----
\ ['fileencoding', 'utf-8', 'a123,'],
\ ['fileformat', 'mac', 'a123'],
\ ['fileformats', 'mac', 'a123'],
+ \ ['filetype', 'abc', 'a^b'],
\ ['fillchars', 'diff:~', 'a123'],
\ ['foldclose', 'all', 'a123'],
\ ['foldmarker', '[[[,]]]', '[[['],
***************
*** 1556,1562 ****
\ ['formatoptions', 'an', '*'],
\ ['guicursor', 'n-v-c:block-Cursor/lCursor', 'n-v-c'],
\ ['helplang', 'en', 'a'],
! \ ['highlight', '!:CursorColumn', '8:']
\ ]
if has('gui')
call add(optlist, ['browsedir', 'buffer', 'a123'])
--- 1561,1609 ----
\ ['formatoptions', 'an', '*'],
\ ['guicursor', 'n-v-c:block-Cursor/lCursor', 'n-v-c'],
\ ['helplang', 'en', 'a'],
! \ ['highlight', '!:CursorColumn', '8:'],
! \ ['keymodel', 'stopsel', 'a123'],
! \ ['keyprotocol', 'kitty:kitty', 'kitty:'],
! \ ['lispoptions', 'expr:1', 'a123'],
! \ ['listchars', 'tab:->', 'tab:'],
! \ ['matchpairs', '<:>', '<:'],
! \ ['mkspellmem', '100000,1000,100', '100000'],
! \ ['mouse', 'nvi', 'z'],
! \ ['mousemodel', 'extend', 'a123'],
! \ ['nrformats', 'alpha', 'a123'],
! \ ['omnifunc', 'MyOmniFunc', '1a-'],
! \ ['operatorfunc', 'MyOpFunc', '1a-'],
! \ ['previewpopup', 'width:20', 'a123'],
! \ ['printoptions', 'paper:A4', 'a123:'],
! \ ['quickfixtextfunc', 'MyQfFunc', '1a-'],
! \ ['rulerformat', '%l', '%['],
! \ ['scrollopt', 'hor,jump', 'a123'],
! \ ['selection', 'exclusive', 'a123'],
! \ ['selectmode', 'cmd', 'a123'],
! \ ['sessionoptions', 'options', 'a123'],
! \ ['shortmess', 'w', '2'],
! \ ['showbreak', '>>', "\x01"],
! \ ['showcmdloc', 'statusline', 'a123'],
! \ ['signcolumn', 'no', 'a123'],
! \ ['spellcapcheck', '[.?!]\+', '%\{'],
! \ ['spellfile', 'MySpell.en.add', "\x01"],
! \ ['spelllang', 'en', "#"],
! \ ['spelloptions', 'camel', 'a123'],
! \ ['spellsuggest', 'double', 'a123'],
! \ ['splitkeep', 'topline', 'a123'],
! \ ['statusline', '%f', '%['],
! \ ['swapsync', 'sync', 'a123'],
! \ ['switchbuf', 'usetab', 'a123'],
! \ ['syntax', 'abc', 'a^b'],
! \ ['tabline', '%f', '%['],
! \ ['tagcase', 'ignore', 'a123'],
! \ ['tagfunc', 'MyTagFunc', '1a-'],
! \ ['thesaurusfunc', 'MyThesaurusFunc', '1a-'],
! \ ['viewoptions', 'options', 'a123'],
! \ ['virtualedit', 'onemore', 'a123'],
! \ ['whichwrap', '<,>', '{,}'],
! \ ['wildmode', 'list', 'a123'],
! \ ['wildoptions', 'pum', 'a123']
\ ]
if has('gui')
call add(optlist, ['browsedir', 'buffer', 'a123'])
***************
*** 1571,1581 ****
call add(optlist, ['cscopequickfix', 't-', 'z-'])
endif
if !has('win32')
! call add(optlist, ['imactivatefunc', 'MyCmplFunc', '1a-'])
endif
for opt in optlist
exe $"let save_opt = &{opt[0]}"
! exe $"let &{opt[0]} = '{opt[1]}'"
call assert_fails($"let &{opt[0]} = '{opt[2]}'", '', opt[0])
call assert_equal(opt[1], eval($"&{opt[0]}"), opt[0])
exe $"let &{opt[0]} = save_opt"
--- 1618,1666 ----
call add(optlist, ['cscopequickfix', 't-', 'z-'])
endif
if !has('win32')
! call add(optlist, ['imactivatefunc', 'MyActFunc', '1a-'])
! call add(optlist, ['imstatusfunc', 'MyStatusFunc', '1a-'])
! endif
! if has('keymap')
! call add(optlist, ['keymap', 'greek', '[]'])
! endif
! if has('mouseshape')
! call add(optlist, ['mouseshape', 'm:no', 'a123:'])
! endif
! if has('win32') && has('gui')
! call add(optlist, ['renderoptions', 'type:directx', 'type:directx,a123'])
! endif
! if has('rightleft')
! call add(optlist, ['rightleftcmd', 'search', 'a123'])
! endif
! if has('terminal')
! call add(optlist, ['termwinkey', '<C-L>', '<C'])
! call add(optlist, ['termwinsize', '24x80', '100'])
! endif
! if has('win32') && has('terminal')
! call add(optlist, ['termwintype', 'winpty', 'a123'])
! endif
! if has('+toolbar')
! call add(optlist, ['toolbar', 'text', 'a123'])
! call add(optlist, ['toolbariconsize', 'medium', 'a123'])
! endif
! if has('+mouse')
! call add(optlist, ['ttymouse', 'xterm', 'a123'])
! endif
! if has('+vartabs')
! call add(optlist, ['varsofttabstop', '12', 'a123'])
! call add(optlist, ['vartabstop', '4,20', '4,'])
! endif
! if has('gui')
! call add(optlist, ['winaltkeys', 'no', 'a123'])
endif
for opt in optlist
exe $"let save_opt = &{opt[0]}"
! try
! exe $"let &{opt[0]} = '{opt[1]}'"
! catch
! call assert_report($"Caught {v:exception} with {opt->string()}")
! endtry
call assert_fails($"let &{opt[0]} = '{opt[2]}'", '', opt[0])
call assert_equal(opt[1], eval($"&{opt[0]}"), opt[0])
exe $"let &{opt[0]} = save_opt"
*** ../vim-9.0.1358/src/version.c 2023-02-27 08:07:10.877872837 +0000
--- src/version.c 2023-02-27 12:46:03.590490404 +0000
***************
*** 697,698 ****
--- 697,700 ----
{ /* Add new patch number below this line */
+ /**/
+ 1359,
/**/
--
hundred-and-one symptoms of being an internet addict:
187. You promise yourself that you'll only stay online for another
15 minutes...at least once every hour.
/// Bram Moolenaar -- [email protected] -- http://www.Moolenaar.net \\\
/// \\\
\\\ sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
--
--
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php
---
You received this message because you are subscribed to the Google Groups
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/vim_dev/20230227124846.D00AD1C0B1F%40moolenaar.net.
