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

Add support for transient keyword #1670

Open
zerosnacks opened this issue Oct 11, 2024 · 5 comments
Open

Add support for transient keyword #1670

zerosnacks opened this issue Oct 11, 2024 · 5 comments
Assignees

Comments

@zerosnacks
Copy link

zerosnacks commented Oct 11, 2024

Request

0.8.28 introduced the transient keyword in Solidity: https://soliditylang.org/blog/2024/10/09/solidity-0.8.28-release-announcement/

Context

Foundry uses Solang's parser for its formatter, see foundry-rs/foundry#9088

@zerosnacks
Copy link
Author

ping @ryjones / @seanyoung, would be great to get support for this

@ryjones
Copy link
Contributor

ryjones commented Oct 16, 2024

@zerosnacks I do backend stuff, not code, but I've added the right people

@zerosnacks
Copy link
Author

@zerosnacks I do backend stuff, not code, but I've added the right people

Thanks! I appreciate it

@seanyoung
Copy link
Contributor

I just need to find some time to add this feature to the parser - should be straightforward.

@seanyoung
Copy link
Contributor

So I had a look at this - it's not so simple. The way this is added to solc is that there is no new keyword. For example:

contract C {
	uint transient transient;
}

is allowed. We need to figure out how to add this without adding a new keyword.

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

No branches or pull requests

5 participants