Don't clear old storage data past stale block lag.

delvtech / council

Flexible DAO governance smart contracts, developed by Delv.

Apache License 2.0

93 stars 19 forks source link

Don't clear old storage data past stale block lag. #87

Closed sentilesdal closed 6 months ago

sentilesdal commented 1 year ago

Removes the _clear() part of findAndClear() from History.sol. Clearing storage used to grant a gas savings cost, but that is no longer the case and so should be removed to prevent Out of Gas errors if there are many changes to History.

sentilesdal commented 6 months ago

Closing this for now since its stale. If this feature is requested later we can add again.