Skip to content

Commit 1150e3e

Browse files
danielshahafdanielshahaf
committed
'main': Highlight pipes inside array assignments as errors
1 parent 533bfa0 commit 1150e3e

File tree

5 files changed

+137
-3
lines changed

5 files changed

+137
-3
lines changed

‎highlighters/main/main-highlighter.zsh

+19-2
Original file line numberDiff line numberDiff line change
@@ -683,18 +683,35 @@ _zsh_highlight_main_highlighter_highlight_list()
683683

684684
# The Great Fork: is this a command word? Is this a non-command word?
685685
if [[ -n ${(M)ZSH_HIGHLIGHT_TOKENS_COMMANDSEPARATOR:#"$arg"} ]]; then
686+
# First, determine the style of the command separator itself.
686687
if _zsh_highlight_main__stack_pop T || _zsh_highlight_main__stack_pop Q; then
687688
# Missing closing square bracket(s)
688689
style=unknown-token
689-
elif [[ $this_word == *':regular:'* ]]; then
690+
elif $in_array_assignment && [[ $arg == ';' ]] && [[ $this_word == *':regular:'* ]]; then
691+
# Okay, this is hairy.
692+
#
693+
# Literal newlines inside array assignment are just fine.
694+
#
695+
# Literal semicolons inside array assignments are accepted by zsh (don't ask),
696+
# but we want to highlight them as errors anyway, because _nobody_ expects
697+
# them to behave as they do.
698+
#
699+
# However, BOTH cases are reported as ";" by ${(z)}, so we have to
700+
# actually check the buffer:
701+
case "${buf[start_pos+1,end_pos]}" in
702+
(';') style=unknown-token;;
703+
($'\n') style=default;;
704+
esac
705+
elif [[ $this_word == *':regular:'* ]] && ! $in_array_assignment; then
690706
# This highlights empty commands (semicolon follows nothing) as an error.
691707
# Zsh accepts them, though.
692708
style=commandseparator
693709
else
694710
style=unknown-token
695711
fi
712+
# Second, determine the style of next_word.
696713
if [[ $arg == ';' ]] && $in_array_assignment; then
697-
# literal newline inside an array assignment
714+
# Literal semicolon or literal newline inside an array assignment
698715
next_word=':regular:'
699716
else
700717
next_word=':start:'

‎highlighters/main/test-data/array-cmdsep1.zsh

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

‎highlighters/main/test-data/array-cmdsep2.zsh

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

‎highlighters/main/test-data/array-cmdsep3.zsh

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

‎highlighters/main/test-data/multiline-array-assignment1.zsh

+1-1
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ BUFFER=$'foo=(\nbar) env'
3232

3333
expected_region_highlight=(
3434
'1 5 assign' # foo=(
35-
'6 6 commandseparator' # \n
35+
'6 6 default' # \n
3636
'7 9 default' # bar
3737
'10 10 assign' # )
3838
'12 14 command' # env

0 commit comments

Comments
 (0)