Zilliqa / ZRC

Zilliqa Reference Contracts
MIT License
40 stars 57 forks source link

feat: multiple events for batch ops #138

Closed ghost closed 2 years ago

ghost commented 2 years ago

This PR changes BatchBurn, BatchTransferFrom to emit an event per item to include the previous token owner address in the event. It's more complete and consistent because Burn and TransferFrom events provide the previous token owner address.

N Current BatchBurn (cumulative_gas) New BatchBurn with Multiple Events (cumulative_gas)
10 766 885
50 2137 2766
100 3843 5110

This approach is more gas-efficient way, compared to the approach #137 with single event.