home *** CD-ROM | disk | FTP | other *** search
/ vim.ftp.fu-berlin.de / 2015-02-03.vim.ftp.fu-berlin.de.tar / vim.ftp.fu-berlin.de / patches / 6.0.134 < prev    next >
Encoding:
Internet Message Format  |  2002-01-14  |  7.8 KB
  1. To: vim-dev@vim.org
  2. Subject: Patch 6.0.134
  3. Fcc: outbox
  4. From: Bram Moolenaar <Bram@moolenaar.net>
  5. MIME-Version: 1.0
  6. Content-Type: text/plain; charset=ISO-8859-1
  7. Content-Transfer-Encoding: 8bit
  8. ------------
  9.  
  10. Patch 6.0.134
  11. Problem:    When completing ":set tags=" a path with an embedded space causes
  12.         the completion to stop. (Sektor van Skijlen)
  13. Solution:   Escape spaces with backslashes, like for ":set path=".  Also take
  14.         backslashes into account when searching for the start of the path
  15.         to complete (e.g., for 'backupdir' and 'cscopeprg').
  16. Files:        src/ex_docmd.c, src/ex_getln.c, src/option.c, src/structs.h
  17.  
  18.  
  19. *** ../vim60.133/src/ex_docmd.c    Mon Dec 31 17:47:49 2001
  20. --- src/ex_docmd.c    Wed Jan  2 13:08:49 2002
  21. ***************
  22. *** 2153,2159 ****
  23.   
  24.       xp->xp_pattern = buff;
  25.       xp->xp_context = EXPAND_COMMANDS;    /* Default until we get past command */
  26. !     xp->xp_set_path = FALSE;
  27.   
  28.   /*
  29.    * 2. skip comment lines and leading space, colons or bars
  30. --- 2153,2159 ----
  31.   
  32.       xp->xp_pattern = buff;
  33.       xp->xp_context = EXPAND_COMMANDS;    /* Default until we get past command */
  34. !     xp->xp_backslash = XP_BS_NONE;
  35.   
  36.   /*
  37.    * 2. skip comment lines and leading space, colons or bars
  38. *** ../vim60.133/src/ex_getln.c    Wed Oct 31 20:51:33 2001
  39. --- src/ex_getln.c    Wed Jan  2 13:22:16 2002
  40. ***************
  41. *** 2605,2611 ****
  42.           for (i = 0; i < numfiles; ++i)
  43.           {
  44.           /* for ":set path=" we need to escape spaces twice */
  45. !         if (xp->xp_set_path)
  46.           {
  47.               p = vim_strsave_escaped(files[i], (char_u *)" ");
  48.               if (p != NULL)
  49. --- 2605,2611 ----
  50.           for (i = 0; i < numfiles; ++i)
  51.           {
  52.           /* for ":set path=" we need to escape spaces twice */
  53. !         if (xp->xp_backslash == XP_BS_THREE)
  54.           {
  55.               p = vim_strsave_escaped(files[i], (char_u *)" ");
  56.               if (p != NULL)
  57. ***************
  58. *** 2643,2649 ****
  59.               }
  60.           }
  61.           }
  62. !         xp->xp_set_path = FALSE;
  63.       }
  64.       else if (xp->xp_context == EXPAND_TAGS)
  65.       {
  66. --- 2643,2649 ----
  67.               }
  68.           }
  69.           }
  70. !         xp->xp_backslash = XP_BS_NONE;
  71.       }
  72.       else if (xp->xp_context == EXPAND_TAGS)
  73.       {
  74. ***************
  75. *** 3103,3138 ****
  76.       if (options & WILD_SILENT)
  77.       flags |= EW_SILENT;
  78.   
  79. !     if (xp->xp_context == EXPAND_FILES)
  80.       {
  81.       /*
  82. !      * Expand file names.
  83. !      */
  84. !     return expand_wildcards(1, &pat, num_file, file, flags|EW_FILE);
  85. !     }
  86. !     else if (xp->xp_context == EXPAND_DIRECTORIES)
  87. !     {
  88. !     /*
  89. !      * Expand directory names.
  90.        */
  91.       int    free_pat = FALSE;
  92.       int    i;
  93.   
  94. !     /* for ":set path=" we need to remove backslashes for escaped space */
  95. !     if (xp->xp_set_path)
  96.       {
  97.           free_pat = TRUE;
  98.           pat = vim_strsave(pat);
  99.           for (i = 0; pat[i]; ++i)
  100. !         if (pat[i] == '\\'
  101. !             && pat[i + 1] == '\\'
  102. !             && pat[i + 2] == '\\'
  103. !             && pat[i + 3] == ' ')
  104. !             STRCPY(pat + i, pat + i + 3);
  105.       }
  106.   
  107. !     ret = expand_wildcards(1, &pat, num_file, file,
  108. !                          (flags | EW_DIR) & ~EW_FILE);
  109.       if (free_pat)
  110.           vim_free(pat);
  111.       return ret;
  112. --- 3103,3141 ----
  113.       if (options & WILD_SILENT)
  114.       flags |= EW_SILENT;
  115.   
  116. !     if (xp->xp_context == EXPAND_FILES || xp->xp_context == EXPAND_DIRECTORIES)
  117.       {
  118.       /*
  119. !      * Expand file or directory names.
  120.        */
  121.       int    free_pat = FALSE;
  122.       int    i;
  123.   
  124. !     /* for ":set path=" and ":set tags=" halve backslashes for escaped
  125. !      * space */
  126. !     if (xp->xp_backslash != XP_BS_NONE)
  127.       {
  128.           free_pat = TRUE;
  129.           pat = vim_strsave(pat);
  130.           for (i = 0; pat[i]; ++i)
  131. !         if (pat[i] == '\\')
  132. !         {
  133. !             if (xp->xp_backslash == XP_BS_THREE
  134. !                 && pat[i + 1] == '\\'
  135. !                 && pat[i + 2] == '\\'
  136. !                 && pat[i + 3] == ' ')
  137. !             STRCPY(pat + i, pat + i + 3);
  138. !             if (xp->xp_backslash == XP_BS_ONE
  139. !                 && pat[i + 1] == ' ')
  140. !             STRCPY(pat + i, pat + i + 1);
  141. !         }
  142.       }
  143.   
  144. !     if (xp->xp_context == EXPAND_FILES)
  145. !         flags |= EW_FILE;
  146. !     else
  147. !         flags = (flags | EW_DIR) & ~EW_FILE;
  148. !     ret = expand_wildcards(1, &pat, num_file, file, flags);
  149.       if (free_pat)
  150.           vim_free(pat);
  151.       return ret;
  152. *** ../vim60.133/src/option.c    Tue Jan 15 14:28:35 2002
  153. --- src/option.c    Tue Jan 15 14:24:07 2002
  154. ***************
  155. *** 7603,7609 ****
  156.       int        opt_idx = 0;    /* init for GCC */
  157.       char_u    *p;
  158.       char_u    *s;
  159. -     char_u    *after_blank = NULL;
  160.       int        is_term_option = FALSE;
  161.       int        key;
  162.   
  163. --- 7603,7608 ----
  164. ***************
  165. *** 7636,7646 ****
  166.           ++p;
  167.           break;
  168.       }
  169. -     /* remember possible start of file name to expand */
  170. -     if (after_blank == NULL
  171. -         && ((*p == ' ' && (p - s) < 2)
  172. -             || (*p == ',' && p == s)))
  173. -         after_blank = p + 1;
  174.       --p;
  175.       }
  176.       if (STRNCMP(p, "no", 2) == 0)
  177. --- 7635,7640 ----
  178. ***************
  179. *** 7733,7742 ****
  180.       xp->xp_context = EXPAND_NOTHING;
  181.       if (is_term_option || (flags & P_NUM))
  182.       return;
  183. !     if (after_blank != NULL)
  184. !     xp->xp_pattern = after_blank;
  185. !     else
  186. !     xp->xp_pattern = p + 1;
  187.       if (flags & P_EXPAND)
  188.       {
  189.       p = options[opt_idx].var;
  190. --- 7727,7735 ----
  191.       xp->xp_context = EXPAND_NOTHING;
  192.       if (is_term_option || (flags & P_NUM))
  193.       return;
  194. ! 
  195. !     xp->xp_pattern = p + 1;
  196. ! 
  197.       if (flags & P_EXPAND)
  198.       {
  199.       p = options[opt_idx].var;
  200. ***************
  201. *** 7758,7768 ****
  202.               || p == (char_u *)&p_cdpath
  203.   #endif
  204.              )
  205. !         xp->xp_set_path = TRUE;
  206.       }
  207.       else
  208.           xp->xp_context = EXPAND_FILES;
  209.       }
  210.       return;
  211.   }
  212.   
  213. --- 7751,7790 ----
  214.               || p == (char_u *)&p_cdpath
  215.   #endif
  216.              )
  217. !         xp->xp_backslash = XP_BS_THREE;
  218. !         else
  219. !         xp->xp_backslash = XP_BS_ONE;
  220.       }
  221.       else
  222. +     {
  223.           xp->xp_context = EXPAND_FILES;
  224. +         /* for 'tags' need three backslashes for a space */
  225. +         if (p == (char_u *)&p_tags)
  226. +         xp->xp_backslash = XP_BS_THREE;
  227. +         else
  228. +         xp->xp_backslash = XP_BS_ONE;
  229. +     }
  230. +     }
  231. + 
  232. +     /* For an option that is a list of file names, find the start of the
  233. +      * last file name. */
  234. +     for (p = arg + STRLEN(arg) - 1; p > xp->xp_pattern; --p)
  235. +     {
  236. +     /* count number of backslashes before ' ' or ',' */
  237. +     if (*p == ' ' || *p == ',')
  238. +     {
  239. +         s = p;
  240. +         while (s > xp->xp_pattern && *(s - 1) == '\\')
  241. +         --s;
  242. +         if ((*p == ' ' && (xp->xp_backslash == XP_BS_THREE && (p - s) < 3))
  243. +             || (*p == ',' && (flags & P_COMMA) && ((p - s) & 1) == 0))
  244. +         {
  245. +         xp->xp_pattern = p + 1;
  246. +         break;
  247. +         }
  248. +     }
  249.       }
  250. + 
  251.       return;
  252.   }
  253.   
  254. *** ../vim60.133/src/structs.h    Fri Nov  2 16:29:44 2001
  255. --- src/structs.h    Wed Jan  2 13:16:36 2002
  256. ***************
  257. *** 364,371 ****
  258.   {
  259.       int        xp_context;        /* type of expansion */
  260.       char_u    *xp_pattern;        /* start of item to expand */
  261. !     int        xp_set_path;        /* ":set path=/dir/<Tab>" */
  262.   } expand_T;
  263.   
  264.   /*
  265.    * Command modifiers ":vertical", ":browse", ":confirm" and ":hide" set a flag.
  266. --- 364,376 ----
  267.   {
  268.       int        xp_context;        /* type of expansion */
  269.       char_u    *xp_pattern;        /* start of item to expand */
  270. !     int        xp_backslash;        /* one of the XP_BS_ values */
  271.   } expand_T;
  272. + 
  273. + /* values for xp_backslash */
  274. + #define XP_BS_NONE    0    /* nothing special for backslashes */
  275. + #define XP_BS_ONE    1    /* uses one backslash before a space */
  276. + #define XP_BS_THREE    2    /* uses three backslashes before a space */
  277.   
  278.   /*
  279.    * Command modifiers ":vertical", ":browse", ":confirm" and ":hide" set a flag.
  280. *** ../vim60.133/src/version.c    Tue Jan 15 14:28:35 2002
  281. --- src/version.c    Tue Jan 15 14:30:42 2002
  282. ***************
  283. *** 608,609 ****
  284. --- 608,611 ----
  285.   {   /* Add new patch number below this line */
  286. + /**/
  287. +     134,
  288.   /**/
  289.  
  290. -- 
  291. Every engineer dreams about saving the universe and having sex with aliens.
  292. This is much more glamorous than the real life of an engineer, which consists
  293. of hiding from the universe and having sex without the participation of other
  294. life forms.
  295.                 (Scott Adams - The Dilbert principle)
  296.  
  297.  ///  Bram Moolenaar -- Bram@moolenaar.net -- http://www.moolenaar.net  \\\
  298. (((   Creator of Vim -- http://vim.sf.net -- ftp://ftp.vim.org/pub/vim   )))
  299.  \\\  Help me helping AIDS orphans in Uganda - http://iccf-holland.org  ///
  300.