2
78%
master: 78%

Ran 02 Oct 2017 12:37PM UTC

Files 20

Run time 1s

Badge

Embed ▾

Committed 02 Oct 2017 12:33PM UTC coverage: 94.351% (-0.01%) from 94.361%

Job # 889.2

Build Type

push

travis-ci

Committed by

mvdan

Commit Message

syntax: parse valid parameter names only

When parsing $foo. We stopped on certain characters, i.e. a blacklist.
Yet the POSIX spec says the opposite:

	If the parameter is not enclosed in braces, and is a name, the
	expansion shall use the longest valid name (see XBD Name)

In other words, we should use a whitelist of characters that are allowed
to advance the lexer. This means that we should stop on any symbol and
any non-ASCII letter, for example. Do it this way and add more tests.

This also means we can simplify quite a bit of the code dealing with
these in the ever growing advanceLitOther.

Coverage Stats

5562 of 5895 relevant lines covered (94.35%)

961.51 hits per line

mvdan / sh / 889 / 2
78%
master: 78%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 889.2

mvdan / sh / 889 / 2 78% master: 78%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 889.2

mvdan / sh / 889 / 2
78%
master: 78%

README BADGES
x