Add a location for error reporting

fonttools · Aug 18, 2022 · 2302b77 · 2302b77
1 parent 201142a
commit 2302b77
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/Lib/fontTools/feaLib/ast.py b/Lib/fontTools/feaLib/ast.py
@@ -2289,7 +2289,7 @@ def __init__(
         self.conditions = conditions
 
     def build(self, builder) -> None:
-        builder.add_conditionset(self.name, self.conditions)
+        builder.add_conditionset(self.location, self.name, self.conditions)
 
     def asFea(self, indent="") -> str:
         res = indent + f"conditionset {self.name} " + "{\n"

diff --git a/Lib/fontTools/feaLib/builder.py b/Lib/fontTools/feaLib/builder.py
@@ -1765,10 +1765,16 @@ def add_hhea_field(self, key, value):
     def add_vhea_field(self, key, value):
         self.vhea_[key] = value
 
-    def add_conditionset(self, key: str, value: Dict[str, Tuple[float, float]]):
+    def add_conditionset(
+        self,
+        location: FeatureLibLocation,
+        key: str,
+        value: Dict[str, Tuple[float, float]],
+    ):
         if not "fvar" in self.font:
             raise FeatureLibError(
-                "Cannot add feature variations to a font without an 'fvar' table"
+                "Cannot add feature variations to a font without an 'fvar' table",
+                location,
             )
 
         # Normalize