vim-patch:2328a39: runtime(netrw): do not double escape Vim special characters

This double escaping was likely introduced because it was the only way
to make :Open work with hashes/percent signs despite shellescape(..., 1)
supposedly taking care of it, but then breaks the gx mapping
on MSYS2 as reported at [0]

Since special characters in the URL following :Open can be escaped,
whereas gx simply breaks and is more common, no longer double escape

[0]: https://github.com/vim/vim/issues/16252

fixes: vim/vim#16252
closes: vim/vim#16265

2328a39a54

Co-authored-by: Konfekt <Konfekt@users.noreply.github.com>
This commit is contained in:
Jaehwang Jung
2025-09-13 00:36:36 +09:00
parent 99d16af86a
commit bf7d20bb1d

View File

@@ -5145,8 +5145,7 @@ fun! netrw#BrowseX(fname,remote)
endif
endif
" although shellescape(..., 1) is used in netrw#Open(), it's insufficient
call netrw#Open(escape(fname, '#%'))
call netrw#Open(fname)
" cleanup: remove temporary file,
" delete current buffer if success with handler,