[subset] FeatureVariations subsetting is wrong
Fixes https://github.com/fonttools/fonttools/issues/1881
This commit is contained in:
parent
dfd2ffe751
commit
cab7d13dc0
@ -1321,8 +1321,14 @@ def subset_features(self, feature_indices):
|
|||||||
@_add_method(otTables.FeatureVariations)
|
@_add_method(otTables.FeatureVariations)
|
||||||
def subset_features(self, feature_indices):
|
def subset_features(self, feature_indices):
|
||||||
self.ensureDecompiled()
|
self.ensureDecompiled()
|
||||||
self.FeaturVariationRecord = [r for r in self.FeatureVariationRecord
|
for r in self.FeatureVariationRecord:
|
||||||
if r.FeatureTableSubstitution.subset_features(feature_indices)]
|
r.FeatureTableSubstitution.subset_features(feature_indices)
|
||||||
|
# Prune empty records at the end only
|
||||||
|
# https://github.com/fonttools/fonttools/issues/1881
|
||||||
|
while (self.FeatureVariationRecord and
|
||||||
|
not self.FeatureVariationRecord[-1]
|
||||||
|
.FeatureTableSubstitution.SubstitutionCount):
|
||||||
|
self.FeatureVariationRecord.pop()
|
||||||
self.FeatureVariationCount = len(self.FeatureVariationRecord)
|
self.FeatureVariationCount = len(self.FeatureVariationRecord)
|
||||||
return bool(self.FeatureVariationCount)
|
return bool(self.FeatureVariationCount)
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user