Skip to content

Add rolling tape and refactor - #5

Open
Boog900 wants to merge 6 commits into
mainfrom
rolling-tapes
Open

Add rolling tape and refactor#5
Boog900 wants to merge 6 commits into
mainfrom
rolling-tapes

Conversation

@Boog900

@Boog900 Boog900 commented Sep 8, 2026

Copy link
Copy Markdown
Member

The tapes now use a trait to abstract their behaviour and the fixed sized tape and the cache are now wrappers.

Here is an example:

    pub(crate) rct_outputs: FixedSizedTape<RctOutput, CachedBlobTape<WholeBlobTape>>,

That type is the same as the old:

    pub(crate) rct_outputs: FixedSizedTape<RctOutput>,

This PR also adds a new RollingBlobTape, which allows removing data from the front and back.

@Boog900

Boog900 commented Sep 8, 2026

Copy link
Copy Markdown
Member Author

This has been reviewed by an LLM and an LLM wrote some tests. I wrote all the other code excluding that.

@Boog900
Boog900 marked this pull request as ready for review September 8, 2026 23:11
Comment thread src/tapes/rolling_tape.rs Outdated
Co-authored-by: redsh4de <25299353+redsh4de@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants