0xMiden / 0xMiden/protocol

Optimize input note num inputs/assets memory layout

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

説明

After #2109 is closed (to avoid merge conflicts), we should be able to make a tiny PR that moves these two constants closer together:

https://github.com/0xMiden/miden-base/blob/eee0e8450b825857e7d56ffebfaa269378a61aca/crates/miden-protocol/src/transaction/kernel/memory.rs#L388-L389

It should be possible to do:
```rust
pub const INPUT_NOTE_NUM_INPUTS_OFFSET: MemoryOffset = 36;
pub const INPUT_NOTE_NUM_ASSETS_OFFSET: MemoryOffset = 37;
pub const INPUT_NOTE_ASSETS_OFFSET: MemoryOffset = 40
```

This applies to `memory.masm` in the same way.

Context: https://github.com/0xMiden/miden-base/pull/2249#discussion_r2679280006

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

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

評価

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

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

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