]> granicus.if.org Git - vim/commitdiff
patch 8.1.0114: confusing variable name v8.1.0114
authorBram Moolenaar <Bram@vim.org>
Sun, 24 Jun 2018 21:53:28 +0000 (23:53 +0200)
committerBram Moolenaar <Bram@vim.org>
Sun, 24 Jun 2018 21:53:28 +0000 (23:53 +0200)
Problem:    Confusing variable name.
Solution:   Rename new_ts to new_vts_array.  Change zero to NULL.

src/ex_cmds.c
src/option.c
src/version.c

index 2ed2bdb4d31b65ba5f72893e8509fda76ab724a0..994ef0c5e9634eca7f5fe7026e240a77decc45ec 100644 (file)
@@ -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
index d7d411772f7a4fd638397e669cde172a15dfce16..1ba8a8c2e426073e2a9cd69879614e5dff625948 100644 (file)
@@ -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;
index 433881c3b1cb14be66d97ac30d4d23d7df631f5c..076778ac3cf63a67eb14d47c00ed5c0f6be242cd 100644 (file)
@@ -789,6 +789,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    114,
 /**/
     113,
 /**/