aave-v3-core icon indicating copy to clipboard operation
aave-v3-core copied to clipboard

Published 20 hours ago • verified publisher icon aave

Fix event emission of AToken and DebtToken events

Open miguelmtzinf opened this issue 2 years ago • 1 comments

  • Fix emission of scaled balance in BalanceTransfer event
  • Fix emission of whole amounts in Transfer event
  • Add emission of Mint events in the transfer function to reflect interest accruals
  • Clarify natspec docs of events and functions
  • Add multiple test cases for AToken and DebtToken events

miguelmtzinf avatar Jun 02 '22 08:06 miguelmtzinf

Closes #681

miguelmtzinf avatar Jun 02 '22 09:06 miguelmtzinf

How is the gas impact of this @miguelmtzinf ?

eboadom avatar Nov 15 '22 13:11 eboadom

How is the gas impact of this @miguelmtzinf ?

There is a slight increase up to 5k gas units. Here you can see the diff: https://editor.mergely.com/i0TdBTPn/

miguelmtzinf avatar Nov 15 '22 16:11 miguelmtzinf