From 6fe21da979e95b4717e190002602d81d9128521b Mon Sep 17 00:00:00 2001 From: "A. Molzer" <5550310+197g@users.noreply.github.com> Date: Thu, 27 Nov 2025 22:45:13 +0100 Subject: [PATCH] Simplify layout proxy methods --- src/decoder/image.rs | 14 -------------- src/decoder/mod.rs | 7 +++++-- 2 files changed, 5 insertions(+), 16 deletions(-) diff --git a/src/decoder/image.rs b/src/decoder/image.rs index 59046a9..113a66c 100644 --- a/src/decoder/image.rs +++ b/src/decoder/image.rs @@ -606,20 +606,6 @@ impl Image { } } - /// Get the layout for each output plane (i.e. one plane for non-planar tiff). - pub(crate) fn preferred_output_layout(&self) -> TiffResult { - let Image { width, height, .. } = *self; - self.preferred_output_layout_for(width, height) - } - - pub(crate) fn preferred_output_layout_for( - &self, - width: u32, - height: u32, - ) -> TiffResult { - self.readout_for_size(width, height)?.to_plane_layout() - } - pub(crate) fn readout_for_image(&self) -> TiffResult { let Image { width, height, .. } = *self; self.readout_for_size(width, height) diff --git a/src/decoder/mod.rs b/src/decoder/mod.rs index d7e8290..e303e01 100644 --- a/src/decoder/mod.rs +++ b/src/decoder/mod.rs @@ -1136,9 +1136,12 @@ impl Decoder { } let data_dims = self.image().chunk_data_dimensions(chunk_index)?; + let layout = self .image() - .preferred_output_layout_for(data_dims.0, data_dims.1)?; + .readout_for_size(data_dims.0, data_dims.1)? + .to_plane_layout()?; + Ok(BufferLayoutPreference::from_planes(&layout)) } @@ -1212,7 +1215,7 @@ impl Decoder { /// indicate the layout needed to read the first data plane. This will be fixed in a future /// major version of `tiff`. pub fn image_buffer_layout(&mut self) -> TiffResult { - let layout = self.image().preferred_output_layout()?; + let layout = self.image().readout_for_image()?.to_plane_layout()?; Ok(BufferLayoutPreference::from_planes(&layout)) }