Commit: runtime(netrw): small fixes to netrw#BrowseX

0 views
Skip to first unread message

Christian Brabandt

unread,
Nov 14, 2024, 3:45:14 PM11/14/24
to vim...@googlegroups.com
runtime(netrw): small fixes to netrw#BrowseX

Commit: https://github.com/vim/vim/commit/460799d885feeabd57b4e9336c513e53b2c7c86f
Author: Konfekt <Kon...@users.noreply.github.com>
Date: Thu Nov 14 21:42:27 2024 +0100

runtime(netrw): small fixes to netrw#BrowseX

- do not enter dir in Netrw on opening
- double quotes after start cause error on Windows

See @chrisbra 's comments at
[0]

[0]: https://github.com/techntools/vim/commit/ff82e7a9dbef1cf98525be3186d65f450390e3b7#diff-39baf27d8f62071617bbef12f874cce31c0ebd02ec99e7b119474ca870c636a3R5279

Signed-off-by: Konfekt <Kon...@users.noreply.github.com>
Signed-off-by: Christian Brabandt <c...@256bit.org>

diff --git a/runtime/autoload/netrw.vim b/runtime/autoload/netrw.vim
index d11c43a49..89aad816f 100644
--- a/runtime/autoload/netrw.vim
+++ b/runtime/autoload/netrw.vim
@@ -35,6 +35,7 @@
" 2024 Nov 07 by Vim Project: use keeppatterns to prevent polluting the search history
" 2024 Nov 07 by Vim Project: fix a few issues with netrw tree listing (#15996)
" 2024 Nov 10 by Vim Project: directory symlink not resolved in tree view (#16020)
+" 2024 Nov 14 by Vim Project: small fixes to netrw#BrowseX (#16056)
" }}}
" Former Maintainer: Charles E Campbell
" GetLatestVimScripts: 1075 1 :AutoInstall: netrw.vim
@@ -5004,7 +5005,7 @@ if has('unix')
let args = a:args
exe 'silent !' ..
\ ((args =~? ' < +\.(exe|com|bat|cmd)>') ?
- \ 'cmd.exe /c start "" /b ' .. args :
+ \ 'cmd.exe /c start /b ' .. args :
\ 'nohup ' .. args .. ' ' .. s:redir() .. ' &')
\ | redraw!
endfun
@@ -5083,10 +5084,7 @@ endif
" given filename; typically this means given their extension.
" 0=local, 1=remote
fun! netrw#BrowseX(fname,remote)
- if a:remote == 0 && isdirectory(a:fname)
- " if its really just a local directory, then do a "gf" instead
- exe "e ".a:fname
- elseif a:remote == 1 && a:fname !~ '^https\=:' && a:fname =~ '/$'
+ if a:remote == 1 && a:fname !~ '^https\=:' && a:fname =~ '/$'
" remote directory, not a webpage access, looks like an attempt to do a directory listing
norm! gf
endif
Reply all
Reply to author
Forward
0 new messages