abs-lang / abs

72%

master: 76%

LAST BUILD ON BRANCH negative-indexes

branch: negative-indexes

CHANGE BRANCH

x

Reset

• negative-indexes
• 1.1.0
• 1.1.x
• 1.10.0
• 1.10.1
• 1.10.x
• 1.11.0
• 1.11.1
• 1.11.2
• 1.11.3
• 1.11.4
• 1.11.x
• 1.12.0
• 1.12.1
• 1.12.x
• 1.13.x
• 1.2.0
• 1.2.x
• 1.3.0
• 1.3.1
• 1.3.2
• 1.3.x
• 1.4.0
• 1.4.1
• 1.4.x
• 1.5.0
• 1.5.1
• 1.5.x
• 1.6.0
• 1.6.1
• 1.6.2
• 1.6.x
• 1.7.0
• 1.7.x
• 1.8.0
• 1.8.1
• 1.8.2
• 1.8.3
• 1.8.x
• 1.9.x
• 2.0.0
• 2.0.x
• 2.1.0
• 2.1.x
• 2.2.0
• 2.2.1
• 2.2.2
• 2.2.x
• 2.3.0
• 2.3.1
• 2.3.x
• 2.4.0
• 2.4.x
• 2.5.x
• 2.6.x
• abs-installer-fixes
• absrc-source-file
• add-code-of-conduct-1
• allow-digits-in-vars
• arg-panic
• array-diff
• array-flatten
• array-flatten-deep
• array-intersect
• array-max
• array-min
• array-partition
• array-reduce
• array-shuffle
• array-tsv
• array-union
• background-commands
• badges
• bash-style-interpolation-in-strings
• bats-testing
• benchmark-for-lazy-evaluation
• better-illegal-tokens-when-parsing-numbers
• break-continue
• builtin-functions-tests
• bye-bye-darwin-386
• bye-bye-travis-ci
• cli-repl
• contributing
• decorators
• default-args-for-functions
• defer
• dependabot/bundler/docs/nokogiri-1.10.8
• dependabot/bundler/docs/rubyzip-2.2.0
• dependabot/go_modules/golang.org/x/crypto-0.1.0
• dependabot/npm_and_yarn/docs/async-2.6.4
• dependabot/npm_and_yarn/docs/browserslist-4.16.6
• dependabot/npm_and_yarn/docs/browserslist-4.16.8
• dependabot/npm_and_yarn/docs/browserslist-4.20.2
• dependabot/npm_and_yarn/docs/decode-uri-component-0.2.2
• dependabot/npm_and_yarn/docs/dns-packet-1.3.4
• dependabot/npm_and_yarn/docs/eventsource-1.1.1
• dependabot/npm_and_yarn/docs/express-4.18.2
• dependabot/npm_and_yarn/docs/follow-redirects-1.14.8
• dependabot/npm_and_yarn/docs/follow-redirects-1.14.9
• dependabot/npm_and_yarn/docs/http-cache-semantics-4.1.1
• dependabot/npm_and_yarn/docs/minimatch-3.1.2
• dependabot/npm_and_yarn/docs/minimist-1.2.6
• dependabot/npm_and_yarn/docs/path-parse-1.0.7
• dependabot/npm_and_yarn/docs/postcss-7.0.36
• dependabot/npm_and_yarn/docs/postcss-7.0.39
• dependabot/npm_and_yarn/docs/prismjs-1.24.0
• dependabot/npm_and_yarn/docs/prismjs-1.24.1
• dependabot/npm_and_yarn/docs/prismjs-1.27.0
• dependabot/npm_and_yarn/docs/qs-6.5.3
• dependabot/npm_and_yarn/docs/ssri-6.0.2
• dependabot/npm_and_yarn/docs/terser-4.8.1
• dependabot/npm_and_yarn/docs/url-parse-1.5.10
• dependabot/npm_and_yarn/docs/url-parse-1.5.3
• dependabot/npm_and_yarn/docs/url-parse-1.5.7
• dependabot/npm_and_yarn/docs/ws-6.2.2
• deprecate-legacy-command-syntax
• deprecations
• descending-int-range
• destructuring-hashes
• doc-command-executor
• echo-dying-message
• edge-case-silent-return
• embedded-interpolation
• eval
• find-shortcut
• fix-324
• fix-for-parsing-prefixes
• fix-hash-to-json
• fix-mixing-property-and-index-expression
• fix-negative-precedence
• fix-tests
• fix-win-tests
• fn-between
• fn-call
• fn-chunk
• fn-clamp
• funny-cases
• go-1.12
• go-1.13
• go-1.15
• go-116
• go-mod
• go-prompt-master
• hash-str-should-convert-to-valid-json
• higher-file-writers
• implicit-return-value
• imported-moduled-cache
• index-range-panics
• index-ranges
• index-ranges-bug
• issue-144-cd-only
• issue-144-v2
• issue-164
• issue-164-v2
• issue-166
• issue-171
• issue-171-alex
• issue-190
• issue-199-exec
• issue-284
• issue-303
• issue-315
• key-in-hash
• lazy-evaluation-in-1.3.x
• make-hashes-iterable
• master
• memo-fix
• memoize
• mount-abs-history-in-dev
• multiple-decorators
• named-functions
• negate-in
• new-docs
• number-abbr
• number-math-rounding
• numeric-bash-arguments-support
• optional-chaining
• parse-exponential-notation
• playground
• refs/tags/2.4.2
• refs/tags/2.5.0
• refs/tags/2.5.1
• refs/tags/2.6.0
• release-1.1.0
• repl-without-terminal-no-panic
• replace-array
• replace-short-form
• return-semicolon
• reverse-copy
• reverse-search
• reverse-search-2
• share-snippets-playground
• simplified-installer
• sleep
• source-vs-require
• specify-command-executor
• stack-overflow
• std-library
• symmetric-diff
• underscores-in-numeric-literals
• unicode
• unique
• upgrade-guide
• upgraded-deps
• write-outer-scope

Build # 871

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

22 of 22 new or added lines in 1 file covered. (100.0%)

2739 of 3800 relevant lines covered (72.08%)

84.27 hits per line