solang icon indicating copy to clipboard operation
solang copied to clipboard

Add support for `transient` keyword

Open zerosnacks opened this issue 1 year ago • 9 comments

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 https://github.com/foundry-rs/foundry/issues/9088

zerosnacks avatar Oct 11 '24 10:10 zerosnacks

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

zerosnacks avatar Oct 16 '24 17:10 zerosnacks

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

ryjones avatar Oct 16 '24 17:10 ryjones

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

Thanks! I appreciate it

zerosnacks avatar Oct 16 '24 17:10 zerosnacks

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

seanyoung avatar Oct 17 '24 08:10 seanyoung

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.

seanyoung avatar Oct 18 '24 17:10 seanyoung

Hey hey, any update on this?

nikkaroraa avatar Dec 02 '24 11:12 nikkaroraa

Hi @seanyoung, any updates on this? Would be great to get this resolved

zerosnacks avatar Dec 19 '24 15:12 zerosnacks

Hi all, any updates on this?

It is currently causing issues for our users preventing them from using transient because forge fmt currently uses Solang.

zerosnacks avatar Feb 24 '25 12:02 zerosnacks

Hi @LucasSte / @seanyoung / @xermicus, any updates or blockers on your side?

zerosnacks avatar Mar 03 '25 14:03 zerosnacks