From 4e9c54e2850593879e3745aabca526376a229dc7 Mon Sep 17 00:00:00 2001 From: Filippo Costa Date: Wed, 14 Jul 2021 14:47:48 +0200 Subject: [PATCH] Fixed gas integration (#265) * Replace `get_` prefix in many getters * Make imports ordering more consistent and use //! docs * Improve documentation for RuntimePtr * Unify impl blocks of Outcome * Simplify Config initialization patterns * Remove duplicated Call API (getters and pub attributes) * Fix wrongly run doctest * Move module documentation to the top * Automatically derive Default for Program * Remove blank lines between struct fields * Replace utility macro with a function * Draft the integration of gas metering * Reorder some imports * Remove too simple utility testing function * Remove redundant Option for cycle examples * Move module docs to top of files within runtime * WIP * Switch to thiserror * Make TemplateHash a typedef, not a struct * Add documentation * WIP * Check for svm_alloc when deploying a template * Small change to comment * Integrate svm_alloc checks before deployment * Improve doc commenting style in svm-query * deny some lints in svm-query * Fix crate name in doc comment * Reduce macro usage in runtime-ffi * Add rustdoc links in context.rs * Change some allow's into warn's and fix lint warnings * WIP * Fix typo in comment * Unify address and app locator traits * Rename 'compue_address.rs' file * Refactor the new_store utility function * Add price resolving draft to spawn_app * Check ctor price before spawning an App * Change the error type on invalid CTORs * Add a pricing mechanism to the SVM runtime * Fix failing test due to price resolver issue * Add a global dependency to `thiserror` * Make svm_alloc error detection more generic * Wrap gas calculation utils in a module * Refactor the opcode validation logic with an .all() * Fix typo in comment * Use a better name for struct field * Reorder import- and export-related items in svm-gas * Rename gas metering -related argument * Rename the AddressLocator trait * Remove unnecessary cast * Use more consistent # Safety notices * More descriptive test name * Better local variable name * Add intra-doc links in a doc comment * Rename StandardPriceResolrver to V0PriceResolver * Remove Clone bound from PriceResolver * Better docs for Outcome * Move local variable out of loop