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 / unreleased / patches / old / 5.4o.2 < prev    next >
Encoding:
Internet Message Format  |  1999-07-12  |  2.8 KB
  1. To: "Graham, Scott" <Sgraham@ea.com>
  2. In-Reply-To: <C7A217699E9ED2118D7A0008C7B115C9687B62@eac-exch3.eac.ea.com>
  3. Cc: vim-dev@vim.org
  4. Subject: patch 5.4o.2 (was: vim54n - 'gf' causes IPF)
  5. From: Bram Moolenaar <Bram@moolenaar.net>
  6. Date: Tue, 13 Jul 1999 10:32:45 +0200
  7. Sender: mool@masaka.moolenaar.net
  8.  
  9.  
  10. Scott Graham wrote:
  11. > Doing 'gf' on top of file.c in the following line:
  12. > 
  13. > file.c://comment here
  14. > 
  15. > causes an IPF (it was grep output). It seems to be related to the ':'
  16. > specifically, but I haven't narrowed it down any further.
  17.  
  18. Indeed.  It is easy to reproduce.
  19.  
  20. > - output of bugreport.vim follows (sorry it's so large, wasn't sure if you'd
  21. >   want any or all of it):
  22.  
  23. Didn't need it in this case.  It often helps to find specific info, although
  24. it is indeed a bit large and it can be difficult to find the item that
  25. matters.  If you can reproduce the problem with "vim -u NONE" I mostly don't
  26. need the bugreport.
  27.  
  28.  
  29. Patch 5.4o.2
  30. Problem:    Crash when using "gf" on "file.c://comment here". (Scott Graham)
  31. Solution:   Fix wrong use of pointers in get_file_name_in_path().
  32. Files:        src/window.c
  33.  
  34.  
  35. *** ../vim-5.4o/src/window.c    Sun Jul 11 20:10:36 1999
  36. --- src/window.c    Tue Jul 13 10:24:22 1999
  37. ***************
  38. *** 1852,1857 ****
  39. --- 1852,1858 ----
  40.   {
  41.       char_u  *ptr;
  42.       char_u  *file_name;
  43. +     char_u  *path;
  44.       int        len;
  45.   
  46.       /*
  47. ***************
  48. *** 1894,1908 ****
  49.        * Such a link looks like "type://machine/path". Only "/path" is used.
  50.        * First search for the string "://", then for the extra '/'
  51.        */
  52. !     if ((file_name = vim_strchr(ptr, ':')) != NULL &&
  53. !         ((path_is_url(file_name) == URL_SLASH &&
  54. !           (file_name = vim_strchr(file_name + 3, '/')) != NULL) ||
  55. !          (path_is_url(file_name) == URL_BACKSLASH &&
  56. !           (file_name = vim_strchr(file_name + 3, '\\')) != NULL)) &&
  57. !         file_name < ptr + len)
  58.       {
  59. !         len -= file_name - ptr;
  60. !         ptr = file_name;
  61.           if (ptr[1] == '~')        /* skip '/' for /~user/path */
  62.           {
  63.           ++ptr;
  64. --- 1895,1909 ----
  65.        * Such a link looks like "type://machine/path". Only "/path" is used.
  66.        * First search for the string "://", then for the extra '/'
  67.        */
  68. !     if ((file_name = vim_strchr(ptr, ':')) != NULL
  69. !         && ((path_is_url(file_name) == URL_SLASH
  70. !             && (path = vim_strchr(file_name + 3, '/')) != NULL)
  71. !             || (path_is_url(file_name) == URL_BACKSLASH
  72. !             && (path = vim_strchr(file_name + 3, '\\')) != NULL))
  73. !         && path < ptr + len)
  74.       {
  75. !         len -= path - ptr;
  76. !         ptr = path;
  77.           if (ptr[1] == '~')        /* skip '/' for /~user/path */
  78.           {
  79.           ++ptr;
  80.  
  81. --
  82. hundred-and-one symptoms of being an internet addict:
  83. 66. You create a homepage with the impression to cure the afflicted...but
  84.     your hidden agenda is to receive more e-mail.
  85.  
  86. --/-/---- Bram Moolenaar ---- Bram@moolenaar.net ---- Bram@vim.org ---\-\--
  87.   \ \    www.vim.org/iccf      www.moolenaar.net       www.vim.org    / /
  88.