Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Replace AlignedBuffer with more general Align #80

Merged
merged 1 commit into from
Oct 19, 2022
Merged

Replace AlignedBuffer with more general Align #80

merged 1 commit into from
Oct 19, 2022

Conversation

joshlf
Copy link
Member

@joshlf joshlf commented Oct 19, 2022

Rename AlignedBuffer<T, B> to Align<T, A>. The former is a B aligned to at least align_of::<T>(), while the latter is a T aligned to at least align_of::<A>(). They're functionally equivalent, but the latter isn't documented in such a way that implies that it can only be used for buffers.

Rename `AlignedBuffer<T, B>` to `Align<T, A>`. The former is a `B`
aligned to at least `align_of::<T>()`, while the latter is a `T` aligned
to at least `align_of::<A>()`. They're functionally equivalent, but the
latter isn't documented in such a way that implies that it can only be
used for buffers.
@joshlf joshlf merged commit 810cef9 into main Oct 19, 2022
@joshlf joshlf deleted the align branch October 19, 2022 22:48
joshlf added a commit that referenced this pull request Aug 3, 2023
Rename `AlignedBuffer<T, B>` to `Align<T, A>`. The former is a `B`
aligned to at least `align_of::<T>()`, while the latter is a `T` aligned
to at least `align_of::<A>()`. They're functionally equivalent, but the
latter isn't documented in such a way that implies that it can only be
used for buffers.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant