From 70d005fde5998cf749c5418a9e8a2a440e28d45c Mon Sep 17 00:00:00 2001 From: Agus Zubiaga Date: Sat, 18 May 2024 02:25:05 +0200 Subject: [PATCH] cli/tarball read_header return tuple directly --- crates/packaging/src/tarball.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/crates/packaging/src/tarball.rs b/crates/packaging/src/tarball.rs index f50125bd5a..0197ba8449 100644 --- a/crates/packaging/src/tarball.rs +++ b/crates/packaging/src/tarball.rs @@ -274,11 +274,11 @@ fn read_header<'a>( // (We can't use that for the parser state and still return Module<'a> unfortunately.) let arena_buf = bumpalo::collections::Vec::from_iter_in(buf.iter().copied(), arena); let parse_state = State::new(arena_buf.into_bump_slice()); - let (module, state) = parse_header(arena, parse_state).unwrap_or_else(|_err| { + let result = parse_header(arena, parse_state).unwrap_or_else(|_err| { todo!(); // TODO report a nice error and exit 1 - or maybe just return Err, for better testability? }); - Ok((module, state)) + Ok(result) } fn add_ingested_files(