shadowfacts/fonttools
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #1780 from fonttools/varlib-add_avar-legible-error-message

Browse Source 
Legible error message for axis mapping duplicate check
This commit is contained in:
Nikolaus Waxweiler 2019-12-12 12:09:31 +00:00 committed by GitHub
commit 980f7f54c8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 11 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
Lib/fontTools/varLib/__init__.py
View File

@ -141,8 +141,14 @@ def _add_avar(font, axes):
assert axis.maximum == max(keys) assert axis.maximum == max(keys)
assert axis.default in keys assert axis.default in keys
# No duplicates # No duplicates
assert len(set(keys)) == len(keys) assert len(set(keys)) == len(keys), (
assert len(set(vals)) == len(vals) f"{axis.tag} axis: All axis mapping input='...' "
"values must be unique, but we found duplicates."
)
assert len(set(vals)) == len(vals), (
f"{axis.tag} axis: All axis mapping output='...' "
"values must be unique, but we found duplicates."
)
# Ascending values # Ascending values
assert sorted(vals) == vals assert sorted(vals) == vals

3
NEWS.rst
View File

@ -1,3 +1,6 @@
- [varLib] Added legible error messages to the ``_add_avar()`` axis mapping duplicate
check.
4.2.1 (released 2019-12-06) 4.2.1 (released 2019-12-06)
--------------------------- ---------------------------