Comments (2)
My thinking is to make attributes general context-sensitive. So we would have something like
#[evm_contract] // indicates that attributes in this module are related to EVM
module M {
#[callable] // interpreted in the context of evm_contract
public fun f(...) { ... }
}
Thoughts?
from move.
@wrwg sorry for the late reply. I think you made a good point that attributes are just strings interpreted by individual tools, so there isn't really that much of an inherent namespace pollution issue here.
Using less verbose (and therefore more general) attribute names will likely make the program look cleaner to people who are familiar, but may confuse newcomers or cause trouble if we want to introduce a similar set of attributes for a different target platform/language and have them present in the same module, but for now I don't think this is a real problem and I like the conciseness of the context-sensitive approach so let's just move forward with it. We can make adjustments should needs arise.
from move.
Related Issues (20)
- [Bug] installation requires rustc 1.66.0 or newer HOT 2
- Update rust-toolchain.toml to latest version HOT 1
- cargo install failed due to toml_edit version HOT 3
- [Bug]: update rust version to 1.66.0 from 1.65.0 HOT 3
- [Dwarf] New language name for Move HOT 9
- [Bug] move-analyzer --branch sui-move server crash HOT 13
- [Feature Request] HOT 1
- [Bug]
- [Bug]
- aptos-move-analyzer 0.1.0 is not sync with the last version 2.4.0 HOT 1
- The sui-move-analyzer plug-in is installed in win10 vscode. Only code highlighting, no code error prompts. How to get code error prompts? HOT 2
- [Bug] cargo test on M2(Macbook pro) fails in docgen HOT 1
- Error parsing '[package]' section of manifest, VSCode HOT 3
- Unable to parse Move package manifest
- Error while following tutorial, Error message : ' found when reading file. Only ASCII printable characters, tabs (\t), and line endings (\n) are permitted.
- [Bug] Use openssl higher than 0.10.41
- Move analyzer parsing error in VS Code
- VSCode sui-move-analyzer plugin problem HOT 1
- [Bug]
- link to <https://github.com/MystenLabs/awesome-move>
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 move.