diff --git a/Lib/ufo2ft/featureWriters/markFeatureWriter.py b/Lib/ufo2ft/featureWriters/markFeatureWriter.py index 0a2f1475..0552b86d 100644 --- a/Lib/ufo2ft/featureWriters/markFeatureWriter.py +++ b/Lib/ufo2ft/featureWriters/markFeatureWriter.py @@ -551,28 +551,6 @@ def _groupMarkClasses(self, markGlyphToMarkClasses): ), ) - def _logIfAmbiguous(self, attachments, groupedMarkClasses): - """Warn about ambiguous situations and log the current resolution. - An anchor attachment is ambiguous if for the same mark glyph, more - than one mark class can be used to attach it to the base. - """ - for attachment in attachments: - for markGlyph, markClasses in attachment.getMarkGlyphToMarkClasses(): - if len(markClasses) > 1: - self.log.info( - "The base glyph %s and mark glyph %s are ambiguously " - "connected by several anchor classes: %s. " - "The last one will prevail.", - attachment.name, - markGlyph, - ", ".join( - markClass - for group in groupedMarkClasses - for markClass in group - if markClass in markClasses - ), - ) - def _removeClassPrefix(self, markClass): assert markClass.startswith(self.markClassPrefix) return markClass[len(self.markClassPrefix) :] @@ -614,7 +592,6 @@ def _groupAttachments(self, attachments): [markClass.name] for _, markClass in sorted(self.context.markClasses.items()) ] - self._logIfAmbiguous(attachments, groupedMarkClasses) lookups = [] for markClasses in groupedMarkClasses: lookup = [] diff --git a/tests/featureWriters/markFeatureWriter_test.py b/tests/featureWriters/markFeatureWriter_test.py index 7ca0ef6c..fa9e0c9b 100644 --- a/tests/featureWriters/markFeatureWriter_test.py +++ b/tests/featureWriters/markFeatureWriter_test.py @@ -1685,12 +1685,6 @@ def test_multiple_anchor_classes_conflict_warning( generated = self.writeFeatures(ufo, groupMarkClasses=groupMarkClasses) - assert ( - "The base glyph a and mark glyph acutecomb are ambiguously " - f"connected by several anchor classes: {warning}. " - "The last one will prevail." in caplog.text - ) - assert str(generated) == expected def test_skipExportGlyphs(self, testufo):