Skip to content

InternalLogger - LogLevel is now LogLevel.Off by default#4547

Merged
snakefoot merged 1 commit intoNLog:devfrom
snakefoot:internallogger-checklevel
Sep 4, 2021
Merged

InternalLogger - LogLevel is now LogLevel.Off by default#4547
snakefoot merged 1 commit intoNLog:devfrom
snakefoot:internallogger-checklevel

Conversation

@snakefoot
Copy link
Contributor

@snakefoot snakefoot commented Sep 4, 2021

Instead of null. This makes IsLogLevelEnabled-check faster, as compiler inlining is activated.

@snakefoot snakefoot added this to the 5.0 (new) milestone Sep 4, 2021
@snakefoot snakefoot added the breaking behavior change Same API, different result label Sep 4, 2021
@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 4, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@snakefoot snakefoot enabled auto-merge (squash) September 4, 2021 22:06
@snakefoot snakefoot merged commit 9f548ba into NLog:dev Sep 4, 2021
@snakefoot snakefoot deleted the internallogger-checklevel branch July 30, 2022 10:36
@snakefoot snakefoot added the new default (breaking) Kind of Breaking behavior change label Jan 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking behavior change Same API, different result new default (breaking) Kind of Breaking behavior change performance size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant