Skip to content

Commit df0f448

Browse files
committed
main: Highlight path after globbing checks
Closes zsh-users#468.
1 parent 2181247 commit df0f448

File tree

3 files changed

+48
-9
lines changed

3 files changed

+48
-9
lines changed

highlighters/main/main-highlighter.zsh

+11-8
Original file line numberDiff line numberDiff line change
@@ -695,19 +695,14 @@ _zsh_highlight_highlighter_main_paint()
695695
elif (( in_redirection == 2 )); then
696696
style=redirection
697697
else
698-
if _zsh_highlight_main_highlighter_check_path; then
699-
style=$REPLY
700-
else
701-
_zsh_highlight_main_highlighter_highlight_argument
702-
already_added=1
703-
fi
698+
_zsh_highlight_main_highlighter_highlight_argument
699+
already_added=1
704700
fi
705701
;;
706702
esac
707703
fi
708704
if ! (( already_added )); then
709705
_zsh_highlight_main_add_region_highlight $start_pos $end_pos $style
710-
[[ $style == path || $style == path_prefix ]] && _zsh_highlight_main_highlighter_highlight_path_separators
711706
fi
712707
if [[ -n ${(M)ZSH_HIGHLIGHT_TOKENS_COMMANDSEPARATOR:#"$arg"} ]]; then
713708
if [[ $arg == ';' ]] && $in_array_assignment; then
@@ -802,7 +797,8 @@ _zsh_highlight_main_highlighter_check_path()
802797
# This command will at least highlight start_pos to end_pos with the default style
803798
_zsh_highlight_main_highlighter_highlight_argument()
804799
{
805-
local i
800+
local i path_eligible style
801+
path_eligible=1
806802

807803
_zsh_highlight_main_add_region_highlight $start_pos $end_pos default
808804
for (( i = 1 ; i <= end_pos - start_pos ; i += 1 )); do
@@ -827,11 +823,18 @@ _zsh_highlight_main_highlighter_highlight_argument()
827823
[*?])
828824
if $highlight_glob; then
829825
_zsh_highlight_main_add_region_highlight $start_pos $end_pos globbing
826+
path_eligible=0
830827
break
831828
fi;;
832829
*) continue;;
833830
esac
834831
done
832+
833+
if (( path_eligible )) && _zsh_highlight_main_highlighter_check_path; then
834+
style=$REPLY
835+
_zsh_highlight_main_add_region_highlight $start_pos $end_pos $style
836+
_zsh_highlight_main_highlighter_highlight_path_separators
837+
fi
835838
}
836839

837840
# Quote Helper Functions
+36
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,36 @@
1+
# -------------------------------------------------------------------------------------------------
2+
# Copyright (c) 2018 zsh-syntax-highlighting contributors
3+
# All rights reserved.
4+
#
5+
# Redistribution and use in source and binary forms, with or without modification, are permitted
6+
# provided that the following conditions are met:
7+
#
8+
# * Redistributions of source code must retain the above copyright notice, this list of conditions
9+
# and the following disclaimer.
10+
# * Redistributions in binary form must reproduce the above copyright notice, this list of
11+
# conditions and the following disclaimer in the documentation and/or other materials provided
12+
# with the distribution.
13+
# * Neither the name of the zsh-syntax-highlighting contributors nor the names of its contributors
14+
# may be used to endorse or promote products derived from this software without specific prior
15+
# written permission.
16+
#
17+
# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND ANY EXPRESS OR
18+
# IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND
19+
# FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR
20+
# CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
21+
# DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
22+
# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER
23+
# IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT
24+
# OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
25+
# -------------------------------------------------------------------------------------------------
26+
# -*- mode: zsh; sh-indentation: 2; indent-tabs-mode: nil; sh-basic-offset: 2; -*-
27+
# vim: ft=zsh sw=2 ts=2 et
28+
# -------------------------------------------------------------------------------------------------
29+
30+
touch \*
31+
32+
BUFFER='noglob echo *'
33+
34+
expected_region_highlight=(
35+
"13 13 path" # *
36+
)

highlighters/main/test-data/order-path-before-globbing.zsh

+1-1
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,6 @@ BUFFER=': * \*'
3333

3434
expected_region_highlight=(
3535
'1 1 builtin' # :
36-
'3 3 globbing "issue #468"' # *
36+
'3 3 globbing' # *
3737
'5 6 path' # \*
3838
)

0 commit comments

Comments
 (0)