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;