search

aionnetwork / AVM

Enabling Java code to run in a blockchain environment
https://theoan.com/
MIT License
49 stars 25 forks source link

Complete support for "default" invocations on value transfer #316

Closed aionbot closed 5 years ago

aionbot commented 5 years ago

Issue created by jeff-aion (on Thursday Nov 22, 2018 at 19:39 GMT)

In #260, we started handling the value transfers directly in the AVM but we explicitly restricted the ability to invoke any "default" entry-point for handling data-less transactions, which we do need to support.

We need to determine what the BlockchainRuntime.getData() should return in this case (null or byte[0] - I suspect byte[0] is the right answer since there is no other way of passing a null through here) and then re-add the support, with more tests.

nancyaion commented 5 years ago

Also, commit in integration_kernel f0661e0f08fff9e53d4349429dd5fa511918f25b "Support default invocations on balance transfers"