-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Default ddLogLevel set to DDLogLevelVerbose #473
Conversation
…o start using CocoaLumberjack is clumsy.", we can define the global log level to Verbose unless it's already defined.
Good idea! |
…o_verbose Default ddLogLevel set to DDLogLevelVerbose
For Swift the global log level ist still set to .Warning instead of .Verbose public var defaultDebugLevel = DDLogLevel.Warning
public func resetDefaultDebugLevel() {
defaultDebugLevel = DDLogLevel.Warning
} |
Should we set it to Verbose, to make it consistent with the Objective-C default one? |
I think so. |
@@ -24,7 +24,11 @@ | |||
* The constant/variable/method responsible for controlling the current log level. | |||
**/ | |||
#ifndef LOG_LEVEL_DEF | |||
#define LOG_LEVEL_DEF ddLogLevel | |||
#ifdef ddLogLevel |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also noticed that this check never works as usually ddLogLevel
is defined as a variable instead of a preprocessor #define
.
…it created more issues than solved.
… #ifdef ddLogLevel was a bad idea and it created more issues than solved." This reverts commit 932a65e.
To avoid
we can define the global log level to Verbose unless it's already defined.