alloy-rs / core

High-performance, well-tested & documented core libraries for Ethereum, in Rust
https://alloy.rs
Apache License 2.0
792 stars 155 forks source link

fix(sol-macro): expand all getter return types #812

Closed DaniPopes closed 3 days ago

DaniPopes commented 3 days ago

Fixes https://github.com/alloy-rs/core/issues/811

Logic was implemented slightly incorrectly from https://github.com/ethereum/solidity/blob/9d7cc42bc1c12bb43e9dccf8c6c36833fdfcbbca/libsolidity/ast/Types.cpp#L2852

This is a breaking change for the generated return type structs of some mapping getters, but it should be fine since it was incorrect before too