Mir

Merge lp:~alan-griffiths/mir/fix-set_logger into lp:mir

  1. fix-set_logger
  2. Merge into development-branch
Proposed by Alan Griffiths
Status: Merged
Approved by: Alan Griffiths
Approved revision: no longer in the source branch.
Merged at revision: 2082
Proposed branch: lp:~alan-griffiths/mir/fix-set_logger
Merge into: lp:mir
Diff against target: 12 lines (+1/-1)
1 file modified
src/common/logging/logger.cpp (+1/-1)
To merge this branch: bzr merge lp:~alan-griffiths/mir/fix-set_logger
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Cemil Azizoglu (community) Approve
Review via email: mp+242392@code.launchpad.net

Commit message

logging: enable global logger to be set

Description of the change

logging: enable global logger to be set

To post a comment you must log in.
Revision history for this message
Cemil Azizoglu (cemil-azizoglu) wrote :

LGTM

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/common/logging/logger.cpp'
2--- src/common/logging/logger.cpp 2014-11-17 18:26:11 +0000
3+++ src/common/logging/logger.cpp 2014-11-20 17:30:55 +0000
4@@ -60,7 +60,7 @@
5
6 void ml::set_logger(std::shared_ptr<Logger> const& new_logger)
7 {
8- if (!new_logger)
9+ if (new_logger)
10 {
11 std::lock_guard<decltype(log_mutex)> lock{log_mutex};
12 the_logger = new_logger;

Subscribers

People subscribed via source and target branches