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 / 7.3 / 7.3.1295 < prev    next >
Encoding:
Internet Message Format  |  2013-07-02  |  10.5 KB
  1. To: vim_dev@googlegroups.com
  2. Subject: Patch 7.3.1295
  3. Fcc: outbox
  4. From: Bram Moolenaar <Bram@moolenaar.net>
  5. Mime-Version: 1.0
  6. Content-Type: text/plain; charset=UTF-8
  7. Content-Transfer-Encoding: 8bit
  8. ------------
  9.  
  10. Patch 7.3.1295
  11. Problem:    glob() and globpath() do not handle escaped special characters
  12.         properly.
  13. Solution:   Handle escaped characters differently. (Adnan Zafar)
  14. Files:        src/testdir/Makefile, src/testdir/test97.in,
  15.         src/testdir/test97.ok, src/testdir/Make_amiga.mak,
  16.         src/testdir/Make_dos.mak, src/testdir/Make_ming.mak,
  17.         src/testdir/Make_os2.mak, src/testdir/Make_vms.mms, src/fileio.c,
  18.         src/misc1.c
  19.  
  20.  
  21. *** ../vim-7.3.1294/src/testdir/Makefile    2013-07-01 21:24:40.000000000 +0200
  22. --- src/testdir/Makefile    2013-07-03 16:20:47.000000000 +0200
  23. ***************
  24. *** 29,35 ****
  25.           test79.out test80.out test81.out test82.out test83.out \
  26.           test84.out test85.out test86.out test87.out test88.out \
  27.           test89.out test90.out test91.out test92.out test93.out \
  28. !         test94.out test95.out test96.out
  29.   
  30.   SCRIPTS_GUI = test16.out
  31.   
  32. --- 29,35 ----
  33.           test79.out test80.out test81.out test82.out test83.out \
  34.           test84.out test85.out test86.out test87.out test88.out \
  35.           test89.out test90.out test91.out test92.out test93.out \
  36. !         test94.out test95.out test96.out test97.out
  37.   
  38.   SCRIPTS_GUI = test16.out
  39.   
  40. *** ../vim-7.3.1294/src/testdir/test97.in    2013-07-03 16:52:09.000000000 +0200
  41. --- src/testdir/test97.in    2013-07-03 16:24:24.000000000 +0200
  42. ***************
  43. *** 0 ****
  44. --- 1,17 ----
  45. + Test whether glob()/globpath() return correct results with certain escaped
  46. + characters.
  47. + 
  48. + STARTTEST
  49. + :so small.vim
  50. + :set shell=doesnotexist
  51. + :e test.out
  52. + :put =glob('Xxx\{')
  53. + :put =glob('Xxx\$')
  54. + :w! Xxx{
  55. + :w! Xxx\$
  56. + :put =glob('Xxx\{')
  57. + :put =glob('Xxx\$')
  58. + :w
  59. + :qa!
  60. + ENDTEST
  61. + 
  62. *** ../vim-7.3.1294/src/testdir/test97.ok    2013-07-03 16:52:09.000000000 +0200
  63. --- src/testdir/test97.ok    2013-07-03 16:14:50.000000000 +0200
  64. ***************
  65. *** 0 ****
  66. --- 1,5 ----
  67. + 
  68. + 
  69. + 
  70. + Xxx{
  71. + Xxx$
  72. *** ../vim-7.3.1294/src/testdir/Make_amiga.mak    2013-07-01 21:24:40.000000000 +0200
  73. --- src/testdir/Make_amiga.mak    2013-07-03 16:21:39.000000000 +0200
  74. ***************
  75. *** 33,39 ****
  76.           test76.out test77.out test78.out test79.out test80.out \
  77.           test81.out test82.out test83.out test84.out test88.out \
  78.           test89.out test90.out test91.out test92.out test93.out \
  79. !         test94.out test95.out test96.out
  80.   
  81.   .SUFFIXES: .in .out
  82.   
  83. --- 33,39 ----
  84.           test76.out test77.out test78.out test79.out test80.out \
  85.           test81.out test82.out test83.out test84.out test88.out \
  86.           test89.out test90.out test91.out test92.out test93.out \
  87. !         test94.out test95.out test96.out test97.out
  88.   
  89.   .SUFFIXES: .in .out
  90.   
  91. ***************
  92. *** 146,148 ****
  93. --- 146,149 ----
  94.   test94.out: test94.in
  95.   test95.out: test95.in
  96.   test96.out: test96.in
  97. + test97.out: test97.in
  98. *** ../vim-7.3.1294/src/testdir/Make_dos.mak    2013-07-01 21:24:40.000000000 +0200
  99. --- src/testdir/Make_dos.mak    2013-07-03 16:21:46.000000000 +0200
  100. ***************
  101. *** 32,38 ****
  102.           test79.out test80.out test81.out test82.out test83.out \
  103.           test84.out test85.out test86.out test87.out test88.out \
  104.           test89.out test90.out test91.out test92.out test93.out \
  105. !         test94.out test95.out test96.out
  106.   
  107.   SCRIPTS32 =    test50.out test70.out
  108.   
  109. --- 32,38 ----
  110.           test79.out test80.out test81.out test82.out test83.out \
  111.           test84.out test85.out test86.out test87.out test88.out \
  112.           test89.out test90.out test91.out test92.out test93.out \
  113. !         test94.out test95.out test96.out test97.out
  114.   
  115.   SCRIPTS32 =    test50.out test70.out
  116.   
  117. *** ../vim-7.3.1294/src/testdir/Make_ming.mak    2013-07-01 21:24:40.000000000 +0200
  118. --- src/testdir/Make_ming.mak    2013-07-03 16:21:49.000000000 +0200
  119. ***************
  120. *** 52,58 ****
  121.           test79.out test80.out test81.out test82.out test83.out \
  122.           test84.out test85.out test86.out test87.out test88.out \
  123.           test89.out test90.out test91.out test92.out test93.out \
  124. !         test94.out test95.out test96.out
  125.   
  126.   SCRIPTS32 =    test50.out test70.out
  127.   
  128. --- 52,58 ----
  129.           test79.out test80.out test81.out test82.out test83.out \
  130.           test84.out test85.out test86.out test87.out test88.out \
  131.           test89.out test90.out test91.out test92.out test93.out \
  132. !         test94.out test95.out test96.out test97.out
  133.   
  134.   SCRIPTS32 =    test50.out test70.out
  135.   
  136. *** ../vim-7.3.1294/src/testdir/Make_os2.mak    2013-07-01 21:24:40.000000000 +0200
  137. --- src/testdir/Make_os2.mak    2013-07-03 16:21:51.000000000 +0200
  138. ***************
  139. *** 33,39 ****
  140.           test76.out test77.out test78.out test79.out test80.out \
  141.           test81.out test82.out test83.out test84.out test88.out \
  142.           test89.out test90.out test91.out test92.out test93.out \
  143. !         test94.out test95.out test96.out
  144.   
  145.   .SUFFIXES: .in .out
  146.   
  147. --- 33,39 ----
  148.           test76.out test77.out test78.out test79.out test80.out \
  149.           test81.out test82.out test83.out test84.out test88.out \
  150.           test89.out test90.out test91.out test92.out test93.out \
  151. !         test94.out test95.out test96.out test97.out
  152.   
  153.   .SUFFIXES: .in .out
  154.   
  155. *** ../vim-7.3.1294/src/testdir/Make_vms.mms    2013-07-01 21:24:40.000000000 +0200
  156. --- src/testdir/Make_vms.mms    2013-07-03 16:21:54.000000000 +0200
  157. ***************
  158. *** 4,10 ****
  159.   # Authors:    Zoltan Arpadffy, <arpadffy@polarhome.com>
  160.   #        Sandor Kopanyi,  <sandor.kopanyi@mailbox.hu>
  161.   #
  162. ! # Last change:  2013 Jul 01
  163.   #
  164.   # This has been tested on VMS 6.2 to 8.3 on DEC Alpha, VAX and IA64.
  165.   # Edit the lines in the Configuration section below to select.
  166. --- 4,10 ----
  167.   # Authors:    Zoltan Arpadffy, <arpadffy@polarhome.com>
  168.   #        Sandor Kopanyi,  <sandor.kopanyi@mailbox.hu>
  169.   #
  170. ! # Last change:  2013 Jul 03
  171.   #
  172.   # This has been tested on VMS 6.2 to 8.3 on DEC Alpha, VAX and IA64.
  173.   # Edit the lines in the Configuration section below to select.
  174. ***************
  175. *** 78,84 ****
  176.        test77.out test78.out test79.out test80.out test81.out \
  177.        test82.out test83.out test84.out test88.out test89.out \
  178.        test90.out test91.out test92.out test93.out test94.out \
  179. !      test95.out test96.out
  180.   
  181.   # Known problems:
  182.   # Test 30: a problem around mac format - unknown reason
  183. --- 78,84 ----
  184.        test77.out test78.out test79.out test80.out test81.out \
  185.        test82.out test83.out test84.out test88.out test89.out \
  186.        test90.out test91.out test92.out test93.out test94.out \
  187. !      test95.out test96.out test97.out
  188.   
  189.   # Known problems:
  190.   # Test 30: a problem around mac format - unknown reason
  191. *** ../vim-7.3.1294/src/fileio.c    2013-06-26 19:17:58.000000000 +0200
  192. --- src/fileio.c    2013-07-03 16:27:10.000000000 +0200
  193. ***************
  194. *** 10301,10307 ****
  195.            * foo\,bar -> foo,bar
  196.            * foo\ bar -> foo bar
  197.            * Don't unescape \, * and others that are also special in a
  198. !          * regexp. */
  199.           if (*++p == '?'
  200.   #ifdef BACKSLASH_IN_FILENAME
  201.               && no_bslash
  202. --- 10301,10310 ----
  203.            * foo\,bar -> foo,bar
  204.            * foo\ bar -> foo bar
  205.            * Don't unescape \, * and others that are also special in a
  206. !          * regexp.
  207. !          * An escaped { must be unescaped since we use magic not
  208. !          * verymagic.
  209. !          */
  210.           if (*++p == '?'
  211.   #ifdef BACKSLASH_IN_FILENAME
  212.               && no_bslash
  213. ***************
  214. *** 10309,10315 ****
  215.               )
  216.               reg_pat[i++] = '?';
  217.           else
  218. !             if (*p == ',' || *p == '%' || *p == '#' || *p == ' ')
  219.               reg_pat[i++] = *p;
  220.               else
  221.               {
  222. --- 10312,10319 ----
  223.               )
  224.               reg_pat[i++] = '?';
  225.           else
  226. !             if (*p == ',' || *p == '%' || *p == '#'
  227. !                             || *p == ' ' || *p == '{')
  228.               reg_pat[i++] = *p;
  229.               else
  230.               {
  231. *** ../vim-7.3.1294/src/misc1.c    2013-06-17 19:26:29.000000000 +0200
  232. --- src/misc1.c    2013-07-03 16:39:46.000000000 +0200
  233. ***************
  234. *** 10457,10462 ****
  235. --- 10457,10510 ----
  236.   }
  237.   #endif
  238.   
  239. + static int has_env_var __ARGS((char_u *p));
  240. + 
  241. + /*
  242. +  * Return TRUE if "p" contains what looks like an environment variable.
  243. +  * Allowing for escaping.
  244. +  */
  245. +     static int
  246. + has_env_var(p)
  247. +     char_u *p;
  248. + {
  249. +     for ( ; *p; mb_ptr_adv(p))
  250. +     {
  251. +     if (*p == '\\' && p[1] != NUL)
  252. +         ++p;
  253. +     else if (vim_strchr((char_u *)
  254. + #if defined(MSDOS) || defined(MSWIN) || defined(OS2)
  255. +                     "$%"
  256. + #else
  257. +                     "$"
  258. + #endif
  259. +                     , *p) != NULL)
  260. +         return TRUE;
  261. +     }
  262. +     return FALSE;
  263. + }
  264. + 
  265. + #ifdef SPECIAL_WILDCHAR
  266. + static int has_special_wildchar __ARGS((char_u *p));
  267. + 
  268. + /*
  269. +  * Return TRUE if "p" contains a special wildcard character.
  270. +  * Allowing for escaping.
  271. +  */
  272. +     static int
  273. + has_special_wildchar(p)
  274. +     char_u  *p;
  275. + {
  276. +     for ( ; *p; mb_ptr_adv(p))
  277. +     {
  278. +     if (*p == '\\' && p[1] != NUL)
  279. +         ++p;
  280. +     else if (vim_strchr((char_u *)SPECIAL_WILDCHAR, *p) != NULL)
  281. +         return TRUE;
  282. +     }
  283. +     return FALSE;
  284. + }
  285. + #endif
  286. + 
  287.   /*
  288.    * Generic wildcard expansion code.
  289.    *
  290. ***************
  291. *** 10507,10513 ****
  292.        */
  293.       for (i = 0; i < num_pat; i++)
  294.       {
  295. !     if (vim_strpbrk(pat[i], (char_u *)SPECIAL_WILDCHAR) != NULL
  296.   # ifdef VIM_BACKTICK
  297.           && !(vim_backtick(pat[i]) && pat[i][1] == '=')
  298.   # endif
  299. --- 10555,10561 ----
  300.        */
  301.       for (i = 0; i < num_pat; i++)
  302.       {
  303. !     if (has_special_wildchar(pat[i])
  304.   # ifdef VIM_BACKTICK
  305.           && !(vim_backtick(pat[i]) && pat[i][1] == '=')
  306.   # endif
  307. ***************
  308. *** 10537,10543 ****
  309.           /*
  310.            * First expand environment variables, "~/" and "~user/".
  311.            */
  312. !         if (vim_strchr(p, '$') != NULL || *p == '~')
  313.           {
  314.           p = expand_env_save_opt(p, TRUE);
  315.           if (p == NULL)
  316. --- 10585,10591 ----
  317.           /*
  318.            * First expand environment variables, "~/" and "~user/".
  319.            */
  320. !         if (has_env_var(p) || *p == '~')
  321.           {
  322.           p = expand_env_save_opt(p, TRUE);
  323.           if (p == NULL)
  324. ***************
  325. *** 10548,10554 ****
  326.            * variable, use the shell to do that.  Discard previously
  327.            * found file names and start all over again.
  328.            */
  329. !         else if (vim_strchr(p, '$') != NULL || *p == '~')
  330.           {
  331.               vim_free(p);
  332.               ga_clear_strings(&ga);
  333. --- 10596,10602 ----
  334.            * variable, use the shell to do that.  Discard previously
  335.            * found file names and start all over again.
  336.            */
  337. !         else if (has_env_var(p) || *p == '~')
  338.           {
  339.               vim_free(p);
  340.               ga_clear_strings(&ga);
  341. *** ../vim-7.3.1294/src/version.c    2013-07-03 15:46:59.000000000 +0200
  342. --- src/version.c    2013-07-03 15:51:54.000000000 +0200
  343. ***************
  344. *** 730,731 ****
  345. --- 730,733 ----
  346.   {   /* Add new patch number below this line */
  347. + /**/
  348. +     1295,
  349.   /**/
  350.  
  351. -- 
  352. When I look deep into your eyes, I see JPEG artifacts.
  353. I can tell by the pixels that we're wrong for each other.  (xkcd)
  354.  
  355.  /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
  356. ///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
  357. \\\  an exciting new programming language -- http://www.Zimbu.org        ///
  358.  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///
  359.