Skip to content
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

Meeting 17th June #1477

Closed
jacqueswww opened this issue Jun 12, 2019 · 7 comments
Closed

Meeting 17th June #1477

jacqueswww opened this issue Jun 12, 2019 · 7 comments
Labels
meeting Meeting

Comments

@jacqueswww
Copy link
Contributor

jacqueswww commented Jun 12, 2019

Start: Monday 17th June 2019, 15:00 UTC (https://time.is/UTC)

Jitsi: https://meet.jit.si/IndifferentSquidAdaptRegularly
https://zoom.us/j/219889477

Agenda

@fubuloubu
Copy link
Member

Yeah, Jitsi sucks lol

@charles-cooper
Copy link
Member

charles-cooper commented Jun 12, 2019

Agenda item: Array indexes as int128

@fubuloubu
Copy link
Member

Proposal: create new VIP from conversation in #1468 to ensure all state-modifying external calls with returns are handled appropriately, without question.

@jacqueswww
Copy link
Contributor Author

jacqueswww commented Jun 17, 2019

Meeting Notes:
On assert transferFrom() the best resolution we came to:

  • Is adding a special return assert statement assert_modifiable().
  • Separate VIP for enforcing to always handle return values via asserts and or assigning variables.

On arrays:

  • Both uint256 and int128 should be first class citizens.
  • Will still consider adding support for negative array lookups, in it's current form negative values have to be clamped to avoid illegal access.

@charles-cooper
Copy link
Member

charles-cooper commented Jun 17, 2019

evaluate some sort of tail-call restriction. external calls can only be the last instruction in a function, or maybe no state changes are allowed after a modifiable (non-static) call.

@davesque
Copy link
Contributor

@jacqueswww @charles-cooper What ended up being discussed about YUL and WASM?

@fubuloubu
Copy link
Member

Next: #1509

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
meeting Meeting
Projects
None yet
Development

No branches or pull requests

4 participants