diff options
author | Bram Moolenaar <Bram@vim.org> | 2018-06-24 23:53:28 +0200 |
---|---|---|
committer | Bram Moolenaar <Bram@vim.org> | 2018-06-24 23:53:28 +0200 |
commit | 0119a59ffdfb21cf1c0a56e7ed6105e875150163 (patch) | |
tree | b034a9624b677b5edb568134a041b995f4090f53 /src | |
parent | 675e8d6adb88d5b38087870dfda51c0205070741 (diff) |
patch 8.1.0114: confusing variable namev8.1.0114
Problem: Confusing variable name.
Solution: Rename new_ts to new_vts_array. Change zero to NULL.
Diffstat (limited to 'src')
-rw-r--r-- | src/ex_cmds.c | 31 | ||||
-rw-r--r-- | src/option.c | 5 | ||||
-rw-r--r-- | src/version.c | 2 |
3 files changed, 20 insertions, 18 deletions
diff --git a/src/ex_cmds.c b/src/ex_cmds.c index 2ed2bdb4d3..994ef0c5e9 100644 --- a/src/ex_cmds.c +++ b/src/ex_cmds.c @@ -678,7 +678,7 @@ ex_retab(exarg_T *eap) char_u *new_line = (char_u *)1; /* init to non-NULL */ int did_undo; /* called u_save for current line */ #ifdef FEAT_VARTABS - int *new_ts = 0; + int *new_vts_array = NULL; char_u *new_ts_str; /* string value of tab argument */ #else int temp; @@ -693,16 +693,17 @@ ex_retab(exarg_T *eap) #ifdef FEAT_VARTABS new_ts_str = eap->arg; - if (!tabstop_set(eap->arg, &new_ts)) + if (!tabstop_set(eap->arg, &new_vts_array)) return; while (vim_isdigit(*(eap->arg)) || *(eap->arg) == ',') ++(eap->arg); - // This ensures that either new_ts and new_ts_str are freshly allocated, - // or new_ts points to an existing array and new_ts_str is null. - if (new_ts == 0) + // This ensures that either new_vts_array and new_ts_str are freshly + // allocated, or new_vts_array points to an existing array and new_ts_str + // is null. + if (new_vts_array == NULL) { - new_ts = curbuf->b_p_vts_array; + new_vts_array = curbuf->b_p_vts_array; new_ts_str = NULL; } else @@ -753,9 +754,7 @@ ex_retab(exarg_T *eap) int t, s; tabstop_fromto(start_vcol, vcol, - tabstop_count(new_ts)? 0: curbuf->b_p_ts, - new_ts, - &t, &s); + curbuf->b_p_ts, new_vts_array, &t, &s); num_tabs = t; num_spaces = s; #else @@ -829,11 +828,11 @@ ex_retab(exarg_T *eap) // If a single value was given then it can be considered equal to // either the value of 'tabstop' or the value of 'vartabstop'. if (tabstop_count(curbuf->b_p_vts_array) == 0 - && tabstop_count(new_ts) == 1 - && curbuf->b_p_ts == tabstop_first(new_ts)) + && tabstop_count(new_vts_array) == 1 + && curbuf->b_p_ts == tabstop_first(new_vts_array)) ; /* not changed */ else if (tabstop_count(curbuf->b_p_vts_array) > 0 - && tabstop_eq(curbuf->b_p_vts_array, new_ts)) + && tabstop_eq(curbuf->b_p_vts_array, new_vts_array)) ; /* not changed */ else redraw_curbuf_later(NOT_VALID); @@ -853,20 +852,20 @@ ex_retab(exarg_T *eap) // than one tabstop then update 'vartabstop'. int *old_vts_ary = curbuf->b_p_vts_array; - if (tabstop_count(old_vts_ary) > 0 || tabstop_count(new_ts) > 1) + if (tabstop_count(old_vts_ary) > 0 || tabstop_count(new_vts_array) > 1) { set_string_option_direct((char_u *)"vts", -1, new_ts_str, OPT_FREE|OPT_LOCAL, 0); vim_free(new_ts_str); - curbuf->b_p_vts_array = new_ts; + curbuf->b_p_vts_array = new_vts_array; vim_free(old_vts_ary); } else { // 'vartabstop' wasn't in use and a single value was given to // retab then update 'tabstop'. - curbuf->b_p_ts = tabstop_first(new_ts); - vim_free(new_ts); + curbuf->b_p_ts = tabstop_first(new_vts_array); + vim_free(new_vts_array); } } #else diff --git a/src/option.c b/src/option.c index d7d411772f..1ba8a8c2e4 100644 --- a/src/option.c +++ b/src/option.c @@ -12844,7 +12844,7 @@ tabstop_start(colnr_T col, int ts, int *vts) int t; int excess; - if (vts == 0 || vts[0] == 0) + if (vts == NULL || vts[0] == 0) return (col / ts) * ts; tabcount = vts[0]; @@ -12878,10 +12878,11 @@ tabstop_fromto( int tabcount; int t; - if (vts == 0 || vts[0] == 0) + if (vts == NULL || vts[0] == 0) { int tabs = 0; int initspc = ts - (start_col % ts); + if (spaces >= initspc) { spaces -= initspc; diff --git a/src/version.c b/src/version.c index 433881c3b1..076778ac3c 100644 --- a/src/version.c +++ b/src/version.c @@ -790,6 +790,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 114, +/**/ 113, /**/ 112, |