You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@iggy.apache.org by gk...@apache.org on 2026/08/18 08:20:35 UTC
(iggy) 03/03: tempv3
This is an automated email from the ASF dual-hosted git repository.
numinnex pushed a commit to branch fix_cleanup_issus
in repository https://gitbox.apache.org/repos/asf/iggy.git
commit 13294df8c259190be64962c23f44c0c6e1dc480c
Author: Grzegorz Koszyk <g....@gmail.com>
AuthorDate: Mon Aug 17 15:33:44 2026 +0200
tempv3
---
core/server_common/src/iobuf.rs | 10 ++++------
core/server_common/src/lib.rs | 13 -------------
2 files changed, 4 insertions(+), 19 deletions(-)
diff --git a/core/server_common/src/iobuf.rs b/core/server_common/src/iobuf.rs
index 7175ef1f6..3c793ec98 100644
--- a/core/server_common/src/iobuf.rs
+++ b/core/server_common/src/iobuf.rs
@@ -465,13 +465,11 @@ impl<const ALIGN: usize> Clone for Extent<ALIGN> {
}
}
-pub(crate) fn freeze_extent<const ALIGN: usize>(extent: Extent<ALIGN>) -> Frozen<ALIGN> {
+fn freeze_extent<const ALIGN: usize>(extent: Extent<ALIGN>) -> Frozen<ALIGN> {
Frozen { inner: extent }
}
-pub(crate) fn extent_from_aligned_vec<const ALIGN: usize>(
- vec: AVec<u8, ConstAlign<ALIGN>>,
-) -> Extent<ALIGN> {
+fn extent_from_aligned_vec<const ALIGN: usize>(vec: AVec<u8, ConstAlign<ALIGN>>) -> Extent<ALIGN> {
let (ptr, _, len, capacity) = vec.into_raw_parts();
let guard = AVecRawGuard::<ALIGN>::new(ptr, len, capacity);
let data = unsafe { NonNull::new_unchecked(ptr) };
@@ -485,7 +483,7 @@ pub(crate) fn extent_from_aligned_vec<const ALIGN: usize>(
}
}
-pub(crate) fn extent_offset_from_base<const ALIGN: usize>(extent: &Extent<ALIGN>) -> usize {
+fn extent_offset_from_base<const ALIGN: usize>(extent: &Extent<ALIGN>) -> usize {
// SAFETY: the extent points into the control block allocation while it is alive.
let offset = unsafe {
extent
@@ -598,7 +596,7 @@ unsafe fn try_merge_extents<const ALIGN: usize>(
}
}
-pub(crate) fn try_coalesce_prefix_with_tail<const ALIGN: usize>(
+fn try_coalesce_prefix_with_tail<const ALIGN: usize>(
prefix: Extent<ALIGN>,
tail: Frozen<ALIGN>,
) -> Result<Frozen<ALIGN>, (Extent<ALIGN>, Frozen<ALIGN>)> {
diff --git a/core/server_common/src/lib.rs b/core/server_common/src/lib.rs
index 4c0fffdec..c8fa01d42 100644
--- a/core/server_common/src/lib.rs
+++ b/core/server_common/src/lib.rs
@@ -23,19 +23,6 @@ pub mod crypto;
pub mod diagnostics;
pub mod executor;
pub mod fs_utils;
-// TODO(hubcio): iobuf was relocated verbatim from `core/binary_protocol/src/consensus/iobuf.rs`
-// during the sans-io split. Its implementation is intentionally untouched; the
-// lints below predate the move and are tracked as tech debt for a 0.9.x cleanup pass.
-#[allow(
- private_interfaces,
- clippy::missing_errors_doc,
- clippy::missing_panics_doc,
- clippy::must_use_candidate,
- clippy::return_self_not_must_use,
- clippy::missing_const_for_fn,
- clippy::non_send_fields_in_send_ty,
- clippy::use_self
-)]
pub mod iobuf;
pub mod log;
mod memory_pool;