alamb commented on a change in pull request #9496:
URL: https://github.com/apache/arrow/pull/9496#discussion_r576405717
##########
File path: rust/arrow/src/bytes.rs
##########
@@ -17,20 +17,20 @@
//! This module contains an implementation of a contiguous immutable memory
region that knows
//! how to de-allocate itself, [`Bytes`].
-//! Note that this is a low-level functionality of this crate.
use core::slice;
-use std::ptr::NonNull;
-use std::sync::Arc;
use std::{fmt::Debug, fmt::Formatter};
+use std::{ptr::NonNull, sync::Arc};
-use crate::{alloc, ffi};
+use crate::ffi;
+
+use super::{alloc, alloc::NativeType};
/// Mode of deallocating memory regions
pub enum Deallocation {
/// Native deallocation, using Rust deallocator with Arrow-specific memory
aligment
Native(usize),
- /// Foreign interface, via a callback
+ // Foreign interface, via a callback
Review comment:
Is there a reason to change from `//` to `///`?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
