Storage.Put() overcharging GAS fees

[king-smith]

Storage.Put() currently charges a minimum GAS fee of 1, no matter how much data is actually written. I'm unsure if this is intended but it seems incorrect based on the documented system fees which indicate 1 GAS per KB: http://docs.neo.org/en-us/sc/systemfees.html

This line: https://github.com/neo-project/neo/blob/master/neo/SmartContract/ApplicationEngine.cs#L388

If unintended, the formula could be updated to something similar to: ((l1 + l2 - 1) * 1000) / 1024. If this was intended then the documents should be updated for clarity on the cost.

[erikzhang]

You are right, the document should be updated.

[king-smith]

Alright, thank you for clarifying 👍

[metachris]

What about leaving this issue open to track it and the progress, and only close it after the document has been updated?

[king-smith]

Document has been updated by neo-project/docs#258, closing this.