487
95%

Ran 04 May 2020 08:28PM UTC

Jobs 1

Files 7

Run time 3s

Badge

Embed ▾

pending completion

Build # 487

Build Type

push

travis-ci-com

Committed by

ayazhafiz

Commit Message

Parse while loops

While adding this feature, I realized we have a problem in some of the
ways we parse expressions. Most expressions (e.g. binary operators)
return the token stream after the expression when they are finished
parsing, but block expressions stop at their terminating brace and the
returned token stream includes the close brace.

```
while true { doWork() }
      ^^^^              parsed expr
           ^^^^^^^^^^^^ rest of token stream

while if true {true} else {false} { doWork() }
      ^^^^^^^^^^^^^^^^^^^^^^^^^^^              parsed expr
                                ^^^^^^^^^^^^^^ rest of token stream

Also fails with parened expressions:
while (if true {true} else {false}) { doWork() }
      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^              parsed expr
                                  ^^^^^^^^^^^^^^ rest of token stream
```

This probably isn't a big deal for now, but I think the best way going
forward for consistency would be to parse the closing token with the
expression and require that blocks end with `!` in a statement context.
This could be kinda ugly though, so I'm interested in other ideas.

```
if true {
  doSomething()
} else {
  never()
}!
```

Part of #51

Run Details

447 of 464 relevant lines covered (96.34%)

0.96 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	487.1	04 May 2020 08:28PM UTC	0	96.34	Travis Job 487.1

Source Files on build 487

Detailed source file information is not available for this build.

ayazhafiz / hgb / 487
95%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 487

ayazhafiz / hgb / 487 95%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 487

ayazhafiz / hgb / 487
95%

README BADGES
x