stonewareslord/zsh-syntax-highlighting
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

'main': Don't highlight bare '$foo' as a filename, as it's a parameter expansion.

Browse Source 
Fixes #474.
This commit is contained in:
Daniel Shahaf 2018-01-20 15:40:11 +00:00 committed by Daniel Shahaf
parent 382be9096d
commit 03692831ad
2 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
highlighters/main/main-highlighter.zsh
View File

@ -823,7 +823,9 @@ _zsh_highlight_main_highlighter_highlight_argument()
highlights+=($reply) highlights+=($reply)
;; ;;
'$') '$')
path_eligible=0
if [[ $arg[i+1] == "'" ]]; then if [[ $arg[i+1] == "'" ]]; then
path_eligible=1
_zsh_highlight_main_highlighter_highlight_dollar_quote $i _zsh_highlight_main_highlighter_highlight_dollar_quote $i
(( i = REPLY )) (( i = REPLY ))
highlights+=($reply) highlights+=($reply)

2
highlighters/main/test-data/order-path-after-dollar.zsh
View File

@ -33,6 +33,6 @@ BUFFER=': $foo \$foo'
expected_region_highlight=( expected_region_highlight=(
'1 1 builtin' # : '1 1 builtin' # :
'3 6 default "issue #474"' # $foo - if we add a "unquoted parameter expansion" style then this expectation should change '3 6 default' # $foo - if we add a "unquoted parameter expansion" style then this expectation should change
'8 12 path' # \$foo '8 12 path' # \$foo
) )