0xMiden / 0xMiden/protocol

Split `$kernel::account` into sub modules

オープン
#2,146 コメント 4 件 リアクション 1 件 担当者 0 名 GitHub で見る
good first issue kernels
主要言語
Rust
スター
132
フォーク
167
平均マージ
1日 23時間
マージ済み PR(30日)
110

説明

We should refactor `$kernel::account` module into sub modules.

> this file has become pretty huge - we should probably split this up into multiple files - e.g.,:
>
> - `asm/kernel/transaction/lib/account/mod.masm`
> - `asm/kernel/transaction/lib/account/code.masm`
> - `asm/kernel/transaction/lib/account/storage.masm`
> - `asm/kernel/transaction/lib/account/vault.masm`
> - maybe something else?
>
_Originally posted by @bobbinth in https://github.com/0xMiden/miden-base/pull/2025#discussion_r2596181154_

All storage related procedures can go into `storage.masm`, vault-related ones in `vault.masm` and code-related ones in `code.masm`. The other procedures working with account and storage commitment, nonce, data loading during prologue can probably stay in `mod.masm`.

This makes sense to do after #2025 at the earliest, though even better after the account procedure info follow-up was done, since this will again touch a bunch of account internals.

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。