8000 Fix more pylint recommendations · bozhou94/fluent-logger-python@405d4dd · GitHub
[go: up one dir, main page]

Skip to content

Commit 405d4dd

Browse files
committed
Fix more pylint recommendations
1 parent 0895338 commit 405d4dd

File tree

2 files changed

+9
-7
lines changed

2 files changed

+9
-7
lines changed

fluent/event.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,6 @@
44
class Event(object):
55
def __init__(self, label, data, **kwargs):
66
assert isinstance(data, dict), 'data must be a dict'
7-
s = kwargs.get('sender', sender.get_global_sender())
7+
sender_ = kwargs.get('sender', sender.get_global_sender())
88
timestamp = kwargs.get('time', int(time.time()))
9-
s.emit_with_time(label, timestamp, data)
9+
sender_.emit_with_time(label, timestamp, data)

fluent/handler.py

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -39,10 +39,11 @@ def _structuring(self, data, msg):
3939
except (ValueError, json.JSONDecodeError):
4040
pass
4141

42-
def _add_dic(self, data, dic):
43-
for k, v in dic.items():
44-
if isinstance(k, basestring):
45-
data[str(k)] = v
42+
@staticmethod
43+
def _add_dic(data, dic):
44+
for key, value in dic.items():
45+
if isinstance(key, basestring):
46+
data[str(key)] = value
4647

4748
class FluentHandler(logging.Handler):
4849
'''
@@ -63,7 +64,8 @@ def __init__(self,
6364
logging.Handler.__init__(self)
6465

6566
def emit(self, record):
66-
if record.levelno < self.level: return
67+
if record.levelno < self.level:
68+
return
6769
data = self.fmt.format(record)
6870
self.sender.emit(None, data)
6971

0 commit comments

Comments
 (0)
0