Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Define all cost related const values in one place #18993

Closed
tao-stones opened this issue Jul 30, 2021 · 1 comment · Fixed by #18994
Closed

Define all cost related const values in one place #18993

tao-stones opened this issue Jul 30, 2021 · 1 comment · Fixed by #18994

Comments

@tao-stones
Copy link
Contributor

Problem

there are several const variables defined few places, eg cost_model.rs, block_cost_limits.rs, maybe other places. Should put them under single roof.

Proposed Solution

Propose to move all const into block_cost_limits.rs, which is currently under ledger/src, since this file will be shared cross multiple crates (core, ledger, etc).

@github-actions
Copy link
Contributor

This issue has been automatically locked since there has not been any activity in past 7 days after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 30, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants