Re: [PATCH v10 08/27] rust: adapt `alloc` crate to the kernel
From: Wei Liu
Date: Wed Sep 28 2022 - 11:11:14 EST
On Tue, Sep 27, 2022 at 03:14:39PM +0200, Miguel Ojeda wrote:
> + /// Tries to append an element to the back of a collection.
> + ///
> + /// # Examples
> + ///
> + /// ```
> + /// let mut vec = vec![1, 2];
> + /// vec.try_push(3).unwrap();
> + /// assert_eq!(vec, [1, 2, 3]);
> + /// ```
> + #[inline]
> + #[stable(feature = "kernel", since = "1.0.0")]
> + pub fn try_push(&mut self, value: T) -> Result<(), TryReserveError> {
> + if self.len == self.buf.capacity() {
> + self.buf.try_reserve_for_push(self.len)?;
> + }
> + unsafe {
> + let end = self.as_mut_ptr().add(self.len);
> + ptr::write(end, value);
> + self.len += 1;
> + }
Missing safety comment here?
With a safety comment added:
Reviewed-by: Wei Liu <wei.liu@xxxxxxxxxx>
> + Ok(())
> + }