From b858f7bb3ec0211c0a5ae7b8c6720075d0a96346 Mon Sep 17 00:00:00 2001 From: Jens Kutilek Date: Wed, 18 Dec 2019 10:03:08 +0100 Subject: [PATCH] Change log message, remove comments --- Lib/fontTools/feaLib/builder.py | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/Lib/fontTools/feaLib/builder.py b/Lib/fontTools/feaLib/builder.py index 44ca5a04b..eefb334fb 100644 --- a/Lib/fontTools/feaLib/builder.py +++ b/Lib/fontTools/feaLib/builder.py @@ -915,13 +915,11 @@ class Builder(object): for (from_glyph, to_glyph) in mapping.items(): if from_glyph in lookup.mapping: if to_glyph == lookup.mapping[from_glyph]: - # log warning? - # FDK logs "[NOTE] Removing duplicate single substitution in 'xxxx' feature: from_glyph, to_glyph" in this case - log.info(FeatureLibError( - 'Removing duplicate single substitution for glyph "%s" by "%s"' % - (from_glyph, to_glyph), - location - )) + log.info( + 'Removing duplicate single substitution from glyph' + ' "%s" to "%s" at %s:%i:%i' % + (from_glyph, to_glyph, *location), + ) else: raise FeatureLibError( 'Already defined rule for replacing glyph "%s" by "%s"' %