903
78%

Ran 22 Oct 2017 05:25PM UTC

Jobs 2

Files 20

Run time 40s

Badge

Embed ▾

pending completion

Build # 903

Build Type

push

travis-ci

Committed by

mvdan

Commit Message

syntax: remove need for a rune var in p.next

This was added for a noticeable performance improvement of around 3%
when parsing strings, due to how advancing ASCII runes is a very hot
path.

However, as it turns out, the compiler can realise this with a bit of
help - if we write p.r just before reading it. Then it can know that
it's safe to reuse the same value without the memory read.

Confirmed that the benchmarks do not change with the code
simplification.

Coverage Stats

5624 of 5967 relevant lines covered (94.25%)

2128.49 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	903.1	22 Oct 2017 05:25PM UTC	0	94.25	Travis Job 903.1
2	903.2	22 Oct 2017 05:25PM UTC	0	94.25	Travis Job 903.2

Source Files on build 903

Detailed source file information is not available for this build.

mvdan / sh / 903
78%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 903

mvdan / sh / 903 78%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 903

mvdan / sh / 903
78%

README BADGES
x