2
72%
master: 63%

Ran 25 Aug 2019 05:16PM UTC

Files 7

Run time 1s

Badge

Embed ▾

Committed 25 Aug 2019 05:13PM UTC coverage: 71.357% (+1.3%) from 70.098%

Job # 867.2

Build Type

Pull #276

travis-ci-com

Committed by

web-flow

Commit Message

Negative indexes, closes #274

This PR adds support for negative indexes in arrays / strings.

``` bash
[1,2,3][-1] #3
```

There is one change that might alter existing ABS scripts, and that's
non-existing indexes, for strings, returning an empty string rather than
`null`.

I believe it's an ok thing to break as both will evaluate to `false`
when casted to boolean, and to check whether an index exists one can
simply:

* check the length of the string (`s.len()`)
* check the boolean value of the index (`!!s[idx]`)

both these examples do not break with these changes. I instead admit
that it would be weird to see code such as:

``` bash
if s[idx] == null {
  ...
}
```

rather than

```
if !s[idx] {
  ...
}
```

So I guess this is a change that can go through, as it really shouldn't
impact much of the userbase.

Pull Request Pull Request #276: Negative indexes, closes #274

Run Details

2708 of 3795 relevant lines covered (71.36%)

42.14 hits per line

abs-lang / abs / 867 / 2
72%
master: 63%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 867.2

abs-lang / abs / 867 / 2 72% master: 63%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 867.2

abs-lang / abs / 867 / 2
72%
master: 63%

README BADGES
x