pwasm-abi icon indicating copy to clipboard operation
pwasm-abi copied to clipboard

Published 20 hours ago • verified publisher icon openethereum

Alternate multiresult implementation

Open cheme opened this issue 6 years ago • 2 comments

Hello, I was updating a branch from months ago and I have seen that I did an alternate implementation for multiresult at the time. I extract it from other changes (partial support for u8 u16 and support for Byte array) to see if it works, seems to pass the test locally . I just PR for information (to communicate this other version), please don't merge this PR (unless the support for tuple of bigger size than 6 is wanted).

cheme avatar Aug 14 '18 16:08 cheme

why closing?

i think it might be better implementation then current ;)

NikVolf avatar Aug 15 '18 14:08 NikVolf

I can reopen (I got the button). I was just thinking of the work to review in comparison to the benefits.

cheme avatar Aug 15 '18 14:08 cheme