solhint icon indicating copy to clipboard operation
solhint copied to clipboard

Upgrade @solidity-parser/parser

Open patrickd- opened this issue 2 years ago • 3 comments

This will add support for memory-safe assembly inline blocks:

-        assembly {
+        assembly ("memory-safe") {

as per https://docs.soliditylang.org/en/v0.8.13/assembly.html#memory-safety

Currently parsing errors due to the outdated version:

  195:17  error  Parse error: extraneous input '(' expecting {'{', StringLiteralFragment}
  195:31  error  Parse error: extraneous input ')' expecting '{'

The upgrade was successfully tested in the https://github.com/patrickd-/cheatsheet.sol repository

patrickd- avatar Jun 16 '22 17:06 patrickd-

Need this!

k06a avatar Jun 23 '22 08:06 k06a

Hey guys, is anyone still maintaining this repository? Sorry to tag you all but PRs like this are becoming a necessity in the latest versions of Solidity.

@fvictorio, @fernandomg, @idrabenia

PaulRBerg avatar Jul 17 '22 08:07 PaulRBerg

please merge this ASAP, ("memory-safe") is a somewhat common pattern

bretth18 avatar Sep 21 '22 18:09 bretth18