[MERGED] osmo-gsm-tester[master]: log.py: LogTarget: return self for chaining modifiers

[Neels Hofmeyr]

Neels Hofmeyr has submitted this change and it was merged.

Change subject: log.py: LogTarget: return self for chaining modifiers
......................................................................


log.py: LogTarget: return self for chaining modifiers

For example this allows
  tgt = LogTarget(foo).set_all_levels(bar).style_change(baz)

Change-Id: If168cc06f644bde6183f3bc51e394c7705386b3e
---
M src/osmo_gsm_tester/log.py
1 file changed, 4 insertions(+), 0 deletions(-)

Approvals:
  Pau Espin Pedrol: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/osmo_gsm_tester/log.py b/src/osmo_gsm_tester/log.py
index 0e9b1f6..d3f2ea0 100644
--- a/src/osmo_gsm_tester/log.py
+++ b/src/osmo_gsm_tester/log.py
@@ -105,6 +105,7 @@
         self.origin_fmt = '{:>%ds}' % self.origin_width
         self.do_log_src = src
         self.do_log_traceback = trace
+        return self
 
     def style_change(self, time=None, time_fmt=None, category=None, level=None, origin=None, origin_width=None, src=None, trace=None):
         'modify only the given aspects of the logging format'
@@ -118,13 +119,16 @@
             src=(src if src is not None else self.do_log_src),
             trace=(trace if trace is not None else self.do_log_traceback),
             )
+        return self
 
     def set_level(self, category, level):
         'set global logging log.L_* level for a given log.C_* category'
         self.category_levels[category] = level
+        return self
 
     def set_all_levels(self, level):
         self.all_levels = level
+        return self
 
     def is_enabled(self, category, level):
         if level == L_TRACEBACK:

-- 
To view, visit https://gerrit.osmocom.org/2507
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If168cc06f644bde6183f3bc51e394c7705386b3e
Gerrit-PatchSet: 2
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Pau Espin Pedrol <pespin at sysmocom.de>