unwanted_tables: Remove checking for 'prop'

because it is an AAT table. (issue #4989)
fonttools · Feb 13, 2025 · 4cc71b3 · 4cc71b3
1 parent 27044ce
commit 4cc71b3
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 6 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -10,6 +10,9 @@ A more detailed list of changes is available in the corresponding milestones for
 ### On the OpenType profile
   - **[opentype/unwanted_aat_tables]:** EBSC is not an AAT table. (issue #4992)
 
+### On the Universal profile
+  - **[unwanted_tables]:** Remove checking for 'prop' because it is an AAT table. (issue #4989)
+
 
 ##  0.13.2 (2025-Feb-03)
 ### Changes to existing checks

diff --git a/Lib/fontbakery/checks/unwanted_tables.py b/Lib/fontbakery/checks/unwanted_tables.py
@@ -33,11 +33,6 @@ def check_unwanted_tables(ttFont):
         "TSID": "Table contains data only used in VOLT",
         "TSIJ": "Table contains data only used in VOLT",
         "TSIB": "Table contains data only used in VOLT",
-        "prop": (
-            "Table used on AAT, Apple's OS X specific technology."
-            " Although Harfbuzz now has optional AAT support,"
-            " new fonts should not be using that."
-        ),
         "Debg": "FontTools debugging table",
     }
     unwanted_tables_found = []

diff --git a/tests/test_checks_unwanted_tables.py b/tests/test_checks_unwanted_tables.py
@@ -38,7 +38,6 @@ def test_check_unwanted_tables(check):
         "TSIB",
         "TSI3",
         "TSI5",
-        "prop",  # FIXME: Why is this one unwanted?
         "Debg",  # Tables such as `Debg` are useful in the pre-production stages of
         #          font development, but add unnecessary bloat to a production font
         #          and should be removed before release.