On Tue, 2025-12-02 at 21:59 -0800, John Hubbard wrote:
> +#[inline(always)]
> +pub(crate) const fn const_align_up<const ALIGN: usize>(value: usize) ->
> usize {
> + build_assert!(ALIGN.is_power_of_two());
> + (value + (ALIGN - 1)) & !(ALIGN - 1)
> +}
So this is just like the issue I'm having with .next_multiple_of() in my patch
#10.
Shouldn't you have a check to make sure that value + (ALIGN - 1) doesn't
overflow? Since I need to
align up to the nearest const power of two, I could use this function instead
of align_up() and
avoid testing for an error condition.
