ilammy / sabre / 56

Ran 13 Jan 2018 08:42AM UTC

Jobs 1

Files 0

Run time –

Badge

Embed ▾

pending completion

Build # 56

Build Type

push

travis-ci

Committed by

web-flow

Commit Message

Merge pull request #17 from ilammy/mandatory-spans

Mandatory spans

This is a preparation for new macro expander and semantic analyzer. It
should simplify the new implementation which will keep macro expanders
in environments.

Previously we have had some spans marked as optional. The idea here was
that if the expressions come from macro expansion then they cannot have
spans (as they come from expressions 'invented' by macro expander).
However, a correct implementation should track at least the source of
the expression (the first macro). An ideal one would track the whole
macro expansion chain.

Currently we do not implement any macros, so we do not really need this.
Later we may replace `Span` with some `NonEmptyVec<Span>` or something.
But it certainly does not need to be an `Option`.

Run Details

Jobs

ID	Job ID	Ran	Files	Coverage
1	56.1	13 Jan 2018 08:42AM UTC	0		Travis Job 56.1

Source Files on build 56

Detailed source file information is not available for this build.

ilammy / sabre / 56

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 56

ilammy / sabre / 56

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 56

README BADGES
x