Comments (5)
I think I would like to leverage more dot notation here.
mstruct Something( someVar: uint256 )
Something.someVar.get(memPtr)
We can make a very small change to Yul+ to add this clarity here.
As for other selector
methods, yes, I'm fine with changing it to getXThing
vs selectXThing
The mstruct
change along with the selector
change should make getters more consistent across the board.
Can you expand on what you mean by stateElement here?
Fine with adding distinct Ptr suffix until we get mem pointer types and types in Yul+.
from fuel-v1-contracts.
Can you expand on what you mean by stateElement here?
State element as in, a UTXO, a deposit, or a root [UTXO]. These aren't part of an inclusion proof, and so should be named distinctly.
from fuel-v1-contracts.
- token -> tokenId where tokenId is used in all structs / events (e.e. Deposit) etc.
from fuel-v1-contracts.
- amount -> value or a consistent use of either term
value
oramount
.
Suggestion: amount
for proofs (i.e. UTXO / Deposit), value
for everything else.
from fuel-v1-contracts.
Closing in favor of rewriting contracts for future versions in Solidity.
from fuel-v1-contracts.
Related Issues (20)
- Timelock should use Ethereum block number HOT 1
- Add event for new witness registration HOT 1
- Potentially change witness registry key and value HOT 1
- Separate block header verifier HOT 1
- Simplify block header verify to just compare hashes HOT 1
- Bitflip Merkle proof index HOT 1
- Fix fee calculation HOT 1
- Commiting new block should check IDs HOT 2
- CI doesn't catch errors HOT 3
- Check invalidInput reverts
- Document token funnel contracts HOT 1
- Patch: Fix funnel self-destruct
- Patch: add totalsupply check in deposit
- Patch: up test coverage
- Benchmarks: Reformat
- Verification Script
- Update comment on deposit logic
- Document and refactor contract verification
- Housekeeping: rename repo
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from fuel-v1-contracts.