Re: [PATCH v5 6/7] rust: alloc: add Vec::remove'

Alice Ryhl
2025-05-07T06:32:08Z

On Wed, May 07, 2025 at 02:32:10PM +0900, Alexandre Courbot wrote:
> On Wed May 7, 2025 at 2:30 PM JST, Alexandre Courbot wrote:
> > On Fri May 2, 2025 at 10:19 PM JST, Alice Ryhl wrote:
> >> + /// Removes the element at the given index.
> >> + ///
> >> + /// # Panics
> >> + ///
> >> + /// Panics if the index is out of bounds.
> >
> > According to the commit log (and the code of the method) I think this
> > panic section is not valid anymore?
>
> Oops never mind, I didn't notice Danilo already pointed this out. >_<

Thanks for taking a look!

Alice