From cfa69f245ff5a39da38b3284578e5296e830227f Mon Sep 17 00:00:00 2001
From: David Corbett <corbett.dav@northeastern.edu>
Date: Sat, 6 Nov 2021 13:32:43 -0400
Subject: [PATCH] [otlLib] Remove duplicates when building coverage

---
 Lib/fontTools/otlLib/builder.py | 2 +-
 Tests/otlLib/builder_test.py    | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/Lib/fontTools/otlLib/builder.py b/Lib/fontTools/otlLib/builder.py
index db4cb6def..6e98e4b21 100644
--- a/Lib/fontTools/otlLib/builder.py
+++ b/Lib/fontTools/otlLib/builder.py
@@ -52,7 +52,7 @@ def buildCoverage(glyphs, glyphMap):
     if not glyphs:
         return None
     self = ot.Coverage()
-    self.glyphs = sorted(glyphs, key=glyphMap.__getitem__)
+    self.glyphs = sorted(set(glyphs), key=glyphMap.__getitem__)
     return self
 
 
diff --git a/Tests/otlLib/builder_test.py b/Tests/otlLib/builder_test.py
index 1c2c324d0..245b62e60 100644
--- a/Tests/otlLib/builder_test.py
+++ b/Tests/otlLib/builder_test.py
@@ -204,7 +204,7 @@ class BuilderTest(object):
         assert builder.buildComponentRecord(None) is None
 
     def test_buildCoverage(self):
-        cov = builder.buildCoverage({"two", "four"}, {"two": 2, "four": 4})
+        cov = builder.buildCoverage(("two", "four", "two"), {"two": 2, "four": 4})
         assert getXML(cov.toXML) == [
             "<Coverage>",
             '  <Glyph value="two"/>',