From 9f7041f65e5cd345fa894bfbbb6e9235260d9c33 Mon Sep 17 00:00:00 2001 From: mathieui Date: Thu, 28 Sep 2017 00:32:10 +0200 Subject: Use match groups instead of re.split because we don't want that --- poezio/logger.py | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) (limited to 'poezio/logger.py') diff --git a/poezio/logger.py b/poezio/logger.py index b5c81f82..1d98e059 100644 --- a/poezio/logger.py +++ b/poezio/logger.py @@ -52,10 +52,13 @@ class LogMessage(LogItem): self.nick = nick def parse_message_line(msg): - if re.match(MESSAGE_LOG_RE, msg): - return LogMessage(*[i for i in re.split(MESSAGE_LOG_RE, msg) if i]) - if re.match(INFO_LOG_RE, msg): - return LogInfo(*[i for i in re.split(INFO_LOG_RE, msg) if i]) + match = re.match(MESSAGE_LOG_RE, msg) + if match: + return LogMessage(*match.groups()) + match = re.match(INFO_LOG_RE, msg) + if match: + return LogInfo(*match.groups()) + log.debug('Error while parsing "%s"', msg) return None class Logger(object): -- cgit v1.2.3