652
78%

Ran 29 Apr 2017 10:52PM UTC

Jobs 2

Files 15

Run time 14s

Badge

Embed ▾

pending completion

Build # 652

Build Type

push

travis-ci

Committed by

mvdan

Commit Message

syntax: remove some _LitWord fast branches

The point of these was to save space, since we would allocate for a
[]WordPart of length 1 instead of 4. But we always do 1 now, so that
doesn't make a difference.

Removing them simplifies the code, and one branch less improves
performance a tiny bit.

Keep the one in the callExpr token switch, as it's a cheaper branch and
it's hit often, actually giving a ~2% gain in Cmds+Nested.

name                   old time/op    new time/op    delta
Parse/LongStrs-4         5.28µs ± 2%    5.26µs ± 1%    ~     (p=0.589 n=6+6)
Parse/Cmds+Nested-4      6.97µs ± 2%    6.88µs ± 1%  -1.26%  (p=0.041 n=6+6)
Parse/Vars+Clauses-4     5.63µs ± 1%    5.68µs ± 1%    ~     (p=0.180 n=6+6)
Parse/Binary+Redirs-4    6.53µs ± 1%    6.56µs ± 1%    ~     (p=0.394 n=6+6)

name                   old alloc/op   new alloc/op   delta
Parse/LongStrs-4         1.70kB ± 0%    1.70kB ± 0%    ~     (all equal)
Parse/Cmds+Nested-4      5.47kB ± 0%    5.47kB ± 0%    ~     (p=1.000 n=6+6)
Parse/Vars+Clauses-4     3.20kB ± 0%    3.20kB ± 0%    ~     (all equal)
Parse/Binary+Redirs-4    3.84kB ± 0%    3.84kB ± 0%    ~     (all equal)

Coverage Stats

4503 of 4664 relevant lines covered (96.55%)

1512.83 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	652.1	29 Apr 2017 10:53PM UTC	0	96.55	Travis Job 652.1
2	652.2	29 Apr 2017 10:52PM UTC	0	96.55	Travis Job 652.2

Source Files on build 652

Detailed source file information is not available for this build.

mvdan / sh / 652
78%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 652

mvdan / sh / 652 78%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 652

mvdan / sh / 652
78%

README BADGES
x