aniani/vim
0
0
Fork 1
mirror of https://github.com/vim/vim.git synced 2025-09-23 03:43:49 -04:00
Code Activity

patch 8.0.1426: "gf" and <cfile> don't accept ? and & in URL

Browse Source 
Problem:    "gf" and <cfile> don't accept ? and & in URL. (Dmitrii Tcyganok)
Solution:   Check for a URL and allow for extra characters. (closes #2493)
This commit is contained in:
Bram Moolenaar
2017-12-25 14:29:18 +01:00
parent 2095148277
commit 9e3dfc6501
3 changed files with 22 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/testdir/test_gf.vim
View File

@@ -7,7 +7,8 @@ func Test_gf_url()
\ "first test for URL://machine.name/tmp/vimtest2a and other text", \ "first test for URL://machine.name/tmp/vimtest2a and other text",
\ "second test for URL://machine.name/tmp/vimtest2b. And other text", \ "second test for URL://machine.name/tmp/vimtest2b. And other text",
\ "third test for URL:\\\\machine.name\\vimtest2c and other text", \ "third test for URL:\\\\machine.name\\vimtest2c and other text",
\ "fourth test for URL:\\\\machine.name\\tmp\\vimtest2d, and other text" \ "fourth test for URL:\\\\machine.name\\tmp\\vimtest2d, and other text",
\ "fifth test for URL://machine.name/tmp?q=vim&opt=yes and other text",
\ ]) \ ])
call cursor(1,1) call cursor(1,1)
call search("^first") call search("^first")
@@ -28,6 +29,10 @@ func Test_gf_url()
call search("URL") call search("URL")
call assert_equal("URL:\\\\machine.name\\tmp\\vimtest2d", expand("<cfile>")) call assert_equal("URL:\\\\machine.name\\tmp\\vimtest2d", expand("<cfile>"))
call search("^fifth")
call search("URL")
call assert_equal("URL://machine.name/tmp?q=vim&opt=yes", expand("<cfile>"))
set isf&vim set isf&vim
enew! enew!
endfunc endfunc

2
src/version.c
View File

@@ -771,6 +771,8 @@ static char *(features[]) =
static int included_patches[] = static int included_patches[] =
{ /* Add new patch number below this line */ { /* Add new patch number below this line */
/**/
1426,
/**/ /**/
1425, 1425,
/**/ /**/

16
src/window.c
View File

@@ -6081,7 +6081,6 @@ grab_file_name(long count, linenr_T *file_lnum)
count, curbuf->b_ffname); count, curbuf->b_ffname);
} }
return file_name_at_cursor(options | FNAME_HYP, count, file_lnum); return file_name_at_cursor(options | FNAME_HYP, count, file_lnum);
} }
/* /*
@@ -6120,6 +6119,8 @@ file_name_in_line(
{ {
char_u *ptr; char_u *ptr;
int len; int len;
int in_type = TRUE;
int is_url = FALSE;
/* /*
* search forward for what could be the start of a file name * search forward for what could be the start of a file name
@@ -6158,8 +6159,19 @@ file_name_in_line(
*/ */
len = 0; len = 0;
while (vim_isfilec(ptr[len]) || (ptr[len] == '\\' && ptr[len + 1] == ' ') while (vim_isfilec(ptr[len]) || (ptr[len] == '\\' && ptr[len + 1] == ' ')
|| ((options & FNAME_HYP) && path_is_url(ptr + len))) || ((options & FNAME_HYP) && path_is_url(ptr + len))
|| (is_url && vim_strchr((char_u *)"?&=", ptr[len]) != NULL))
{ {
/* After type:// we also include ?, & and = as valid characters, so that
* http://google.com?q=this&that=ok works. */
if ((ptr[len] >= 'A' && ptr[len] <= 'Z') || (ptr[len] >= 'a' && ptr[len] <= 'z'))
{
if (in_type && path_is_url(ptr + len + 1))
is_url = TRUE;
}
else
in_type = FALSE;
if (ptr[len] == '\\') if (ptr[len] == '\\')
/* Skip over the "\" in "\ ". */ /* Skip over the "\" in "\ ". */
++len; ++len;