Commit 62aa0719 authored by Ian Jackson's avatar Ian Jackson
Browse files

logging: Drop a pointless .within() call 

Since "logfiles: Introduce LogfileListConfigBuilder", this code is in
LogfileListConfigBuilder::build(), which is called by derive_builder's
generated LoggingConfig::build(), and which will add a file context
itself due to the `sub_builder` feature.

So this is otiose.  And, we are about to replace this whole thing with
macro_rules-generated code (which won't do this).
parent 0d4c53bb

crates/arti/src/logging.rs

+1 −2
Original line number Diff line number Diff line
@@ -108,8 +108,7 @@ impl LogfileListConfigBuilder { 
        let files = files

            .iter()

            .map(|item| item.build())

            .collect::<Result<_, _>>()

            .map_err(|e| e.within("files"))?;

            .collect::<Result<_, _>>()?;

        Ok(files)

    }

}