casey / casey/x-serialization-format

Rename allocator?

Open
#104 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
1
Forks
1
PR merge metrics
No merged PRs in 30d

Description

There are multiple kinds of allocator:

- SliceAllocator: Write to a fixed size slice, does no allocation.
- VecAllocator: Write to a vector, actually does allocation.
- IoAllocator: Write to a seekable I/O object, doesn't do allocation.

I think I should probably rename the `Allocator` trait to something else, since two of these don't do any allocation. I'm thinking `Store` or `Storer`.

Contributor guide

No contributing guide indexed for this repository

Research direction

Review the Allocator trait and its SliceAllocator, VecAllocator, and IoAllocator implementations to understand how the name is used. Decide on terminology that accurately covers all three implementations, then update the affected references consistently. Done means the naming is clear and no implementation is incorrectly described as allocating.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
backend
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.