After some back and forth in this thread, IRC, and in bugzilla I've been 
convinced that adding a Verbose level will make this transition smoother.

The new enum will look something like:

enum class LogLevel {
  Disabled = 0, // Logging is disabled for this module
  Error,
  Warning,
  Info,
  Debug,
  Verbose
};

This also has the benefit of using the same numeric values for Debug 
(PR_LOG_DEBUG = 4) and Verbose (PR_LOG_DEBUG + 1 = 5) as we currently do. So 
for example setting "MediaManager:5" will output the same levels both before 
and after this change.

Thanks for all the feedback!
_______________________________________________
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform

Reply via email to