673
78%

Ran 14 May 2017 04:56PM UTC

Jobs 2

Files 15

Run time 3s

Badge

Embed ▾

pending completion

Build # 673

Build Type

push

travis-ci

Committed by

mvdan

Commit Message

syntax: error on unescaped $ literal

The only context in which it could make sense is a double-quoted string
that needs to use expansions too. For example:

	sed "s/$foo bar$/with/"

In that case, the second '$' is a literal and not a parameter expansion,
as it is not followed by a literal nor a character that makes its token
longer like '{' or '('.

The program should instead be written as:

	sed "s/$foo bar\$/with/"

In all other cases where expansions are not necessary, string literals
should be used instead, i.e. single quotes.

This gets rid of split literals without extra complexity and forces
programs to be saner and easier to read for both the human eye and the
parser. It also means that some possible bugs may be caught, like the
use of $[foo] in POSIX mode that will now error.

Coverage Stats

4597 of 4757 relevant lines covered (96.64%)

1505.04 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	673.1	14 May 2017 04:56PM UTC	0	96.64	Travis Job 673.1
2	673.2	14 May 2017 04:56PM UTC	0	96.64	Travis Job 673.2

Source Files on build 673

Detailed source file information is not available for this build.

mvdan / sh / 673
78%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 673

mvdan / sh / 673 78%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 673

mvdan / sh / 673
78%

README BADGES
x