889
78%

Ran 02 Oct 2017 12:37PM UTC

Jobs 2

Files 20

Run time 25s

Badge

Embed ▾

pending completion

Build # 889

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%)

1924.71 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	889.1	02 Oct 2017 12:38PM UTC	0	94.35	Travis Job 889.1
2	889.2	02 Oct 2017 12:37PM UTC	0	94.35	Travis Job 889.2

Source Files on build 889

Detailed source file information is not available for this build.

mvdan / sh / 889
78%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 889

mvdan / sh / 889 78%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 889

mvdan / sh / 889
78%

README BADGES
x