From 02770eff60a6f7c45dd342fd5959f0f392c21de0 Mon Sep 17 00:00:00 2001 From: Nicholas Bishop Date: Sat, 2 Dec 2023 16:59:52 -0500 Subject: [PATCH] fs: Remove a couple debug logs The trace log in `open` can be a bit misleading since at first glance it looks like an error, but in some cases it's expected for open to fail, like when calling `try_exists`. The debug log in `create_dir_all` is showing internal details so probably not all that useful to end users. --- uefi/src/fs/file_system/fs.rs | 3 --- 1 file changed, 3 deletions(-) diff --git a/uefi/src/fs/file_system/fs.rs b/uefi/src/fs/file_system/fs.rs index d08e1b892..8eadc6710 100644 --- a/uefi/src/fs/file_system/fs.rs +++ b/uefi/src/fs/file_system/fs.rs @@ -10,7 +10,6 @@ use alloc::vec::Vec; use core::fmt; use core::fmt::{Debug, Formatter}; use core::ops::Deref; -use log::debug; /// Return type for public [`FileSystem`] operations. pub type FileSystemResult = Result; @@ -156,7 +155,6 @@ impl<'a> FileSystem<'a> { // Collect all relevant sub paths in a vector. let mut dirs_to_create = vec![path.to_path_buf()]; while let Some(parent) = dirs_to_create.last().unwrap().parent() { - debug!("parent={parent}"); dirs_to_create.push(parent) } // Now reverse, so that we have something like this: @@ -421,7 +419,6 @@ impl<'a> FileSystem<'a> { self.open_root()? .open(path.to_cstr16(), mode, attr) .map_err(|err| { - log::trace!("Can't open file {path}: {err:?}"); Error::Io(IoError { path: path.to_path_buf(), context: IoErrorContext::OpenError,