Skip to content
This repository has been archived by the owner on Nov 15, 2021. It is now read-only.

completed implementation of take() builtin #29

Merged
merged 1 commit into from Oct 13, 2017
Merged

completed implementation of take() builtin #29

merged 1 commit into from Oct 13, 2017

Conversation

hal0x2328
Copy link
Member

First foray into trying to implement builtins into compiler, started with take() since it was already fleshed out in builtins.py. Test case seems to work:
`
neo> build boa/tests/src/TakeTest.py test 05 05 False
Saved output to boa/tests/src/TakeTest.avm


Calling boa/tests/src/TakeTest.py with arguments []
Test deploy invoke successful
Used total of 38 operations
Result b'hello'
Invoke TX gas cost: 0.001
`

@coveralls
Copy link

Coverage Status

Coverage remained the same at 50.954% when pulling d8d4206 on hal0x2328:feature-boa into 2bcdcca on CityOfZion:feature-boa.

@localhuman
Copy link
Collaborator

Looks good, though I'll have to test is out when I get home from work.

The same functionality can be done with slice notation, so currently str[:5] should return the same thing.

@localhuman localhuman merged commit f44a52a into CityOfZion:feature-boa Oct 13, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants