-
Notifications
You must be signed in to change notification settings - Fork 132
Opcodes stores + loads and Pass-By-Reference Semantics
#183
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closed
Closed
Changes from all commits
Commits
Show all changes
55 commits
Select commit
Hold shift + click to select a range
7ac246b
first cut
d0a9e98
Update pyteal/ast/subroutine_test.py
tzaffi 84eb656
lint
32855a7
unused import
1be9ffd
better unit test
63ee75f
pass on circle too
d4c03bf
async issue on circle???
6d4e00c
pass partial annotations case
707c67c
lint
e585c70
handle ScratchVar and Expr subtypes for the return
c284da2
mypy
c42f6d2
error massaging
5b4d9e0
better error message
c5fbbdc
Merge remote-tracking branch 'origin' into stores-loads
7143e43
tuple not list
fdd5040
pass tests on travis (enums treated differently between python versions)
77ea79e
travis be nice
236c837
partial skeleton
780f68e
loadSlotidFromStack --> slodIdFromStack
d4448a8
another test case for init ScratchSlot
113b215
make mypy happy, but I'm not sure I am...
dded740
fromStack --> slotIdFromStack
fd2dcbf
per CR
70968f4
Update pyteal/ast/scratchvar.py
tzaffi 072ad1e
Merge branch 'stores-loads' of github.com:algorand/pyteal into stores…
9205cf3
DynamicSlot
32a4fce
doc wording
421e841
unit tests passing, but need lots more
9b23e43
close, but still need stack variables before subroutine call
13b238f
looks like it's working. Next: cleanup + tests
c05437a
nope before - but now can pass-by-ref non-duynamic as well
4a96878
clean up
ed673ce
revert withDynamic param
5fc470a
less wrong comment
86a3771
uncomment clear
d3e00b7
TDD: 'ScratchVar' object has no attribute 'Index'
5fb32e8
start cleanup
e3876cb
spacing/reorder imports
ae7c811
unused imports
3c81ad1
remove dead branch
ef0250b
as name suggests, subroutineInternal should be internal
2e6ba74
test cases
4a5a68c
unnneeded variable
92598b0
easier diff
c0dbc52
update error messages
ab38218
comment
1399e09
question comments
351d3c9
Merge branch 'master' of github.com:algorand/pyteal into stores-loads
2fda6c2
ugly commit, but saving my work at a good spot
1c0498a
it works... but major cleanup necessary and need to support implicit …
e48be41
helpers in UnaryExpr
b17fa15
allow more general types
39ae893
don't need the extra swap anymore
20e3117
easier API
eb8e743
better comment
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unrelated doc changes.