Skip to content

Commit

Permalink
Bug 1874215 - Don't set the Rust max log level r=markh
Browse files Browse the repository at this point in the history
Increasing the level to DEBUG is causing slowdowns for the necko code
and it doesn't seem like we're actually getting the Rust records into
the sync logs anyways.

Differential Revision: https://phabricator.services.mozilla.com/D199259
  • Loading branch information
bendk committed Jan 22, 2024
1 parent 67be8a7 commit ffe02b6
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions services/common/app_services_logger/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -46,11 +46,6 @@ impl AppServicesLogger {
xpcom_method!(register => Register(target: *const nsAString, logger: *const mozIServicesLogSink));
fn register(&self, target: &nsAString, logger: &mozIServicesLogSink) -> Result<(), nsresult> {
let log_sink_logger = LogSink::with_logger(Some(logger))?;
let max_level = cmp::max(log::max_level(), log_sink_logger.max_level);

// Note: This will only work if the max_level is lower than the compile-time
// max_level_* filter.
log::set_max_level(max_level);

ensure_observing_shutdown();

Expand Down

0 comments on commit ffe02b6

Please sign in to comment.