From 921fadaad9284709adad7a8c51de32da611247ce Mon Sep 17 00:00:00 2001 From: Fini Jastrow Date: Thu, 20 Apr 2023 16:42:15 +0200 Subject: [PATCH] name-parser: Unify name related error message format [why] It is hard to grep for all naming related errors, because there is no common format. [how] Make them also visually similar and error cases easily spotted. Signed-off-by: Fini Jastrow --- bin/scripts/name_parser/FontnameParser.py | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/bin/scripts/name_parser/FontnameParser.py b/bin/scripts/name_parser/FontnameParser.py index 9fdc0e896..d8f70da24 100644 --- a/bin/scripts/name_parser/FontnameParser.py +++ b/bin/scripts/name_parser/FontnameParser.py @@ -36,10 +36,10 @@ class FontnameParser: q = limit - len(r.groups()[1]) if q < 1: q = 1 - self.logger.error('Shortening too long PS {}name: Garbage warning'. format(fam)) + self.logger.error('====-< Shortening too long PS {}name: Garbage warning'. format(fam)) new_n = r.groups()[0][:q] + r.groups()[1] if new_n != n: - self.logger.error('Shortening too long PS {}name: {} -> {}'.format(fam, n, new_n)) + self.logger.error('====-< Shortening too long PS {}name: {} -> {}'.format(fam, n, new_n)) return new_n def _shortened_name(self): @@ -238,8 +238,9 @@ class FontnameParser: def checklen(self, max_len, entry_id, name): """Check the length of a name string and report violations""" if len(name) <= max_len: - return name - self.logger.error('{} too long ({} > {}): {}'.format(entry_id, len(name), max_len, name)) + self.logger.debug('=====> {:18} ok ({:2} <={:2}): {}'.format(entry_id, len(name), max_len, name)) + else: + self.logger.error('====-< {:18} too long ({:2} > {:2}): {}'.format(entry_id, len(name), max_len, name)) return name def rename_font(self, font):