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

otConverters: parse 'VarIndexBase + {offset}' from converters' description

Browse Source
This commit is contained in:
Cosimo Lupo 2022-06-14 18:14:17 +01:00
parent 871ee3d527
commit 396353e8e3
1 changed files with 30 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

40
Lib/fontTools/ttLib/tables/otConverters.py
View File

@ -18,7 +18,9 @@ from .otTables import (lookupTypes, AATStateTable, AATState, AATAction,
CompositeMode as _CompositeMode) CompositeMode as _CompositeMode)
from itertools import zip_longest from itertools import zip_longest
from functools import partial from functools import partial
import re
import struct import struct
from typing import Optional
import logging import logging
@ -60,7 +62,7 @@ def buildConverters(tableSpec, tableNamespace):
else: else:
converterClass = eval(tp, tableNamespace, converterMapping) converterClass = eval(tp, tableNamespace, converterMapping)
conv = converterClass(name, repeat, aux) conv = converterClass(name, repeat, aux, description=descr)
if conv.tableClass: if conv.tableClass:
# A "template" such as OffsetTo(AType) knowss the table class already # A "template" such as OffsetTo(AType) knowss the table class already
@ -136,7 +138,7 @@ class BaseConverter(object):
"""Base class for converter objects. Apart from the constructor, this """Base class for converter objects. Apart from the constructor, this
is an abstract class.""" is an abstract class."""
def __init__(self, name, repeat, aux, tableClass=None): def __init__(self, name, repeat, aux, tableClass=None, *, description=""):
self.name = name self.name = name
self.repeat = repeat self.repeat = repeat
self.aux = aux self.aux = aux
@ -159,6 +161,7 @@ class BaseConverter(object):
"BaseGlyphRecordCount", "BaseGlyphRecordCount",
"LayerRecordCount", "LayerRecordCount",
] ]
self.description = description
def readArray(self, reader, font, tableDict, count): def readArray(self, reader, font, tableDict, count):
"""Read an array of values from the reader.""" """Read an array of values from the reader."""
@ -211,6 +214,15 @@ class BaseConverter(object):
"""Write a value to XML.""" """Write a value to XML."""
raise NotImplementedError(self) raise NotImplementedError(self)
varIndexBasePlusOffsetRE = re.compile(r"VarIndexBase\s*\+\s*(\d+)")
def getVarIndexOffset(self) -> Optional[int]:
"""If description has `VarIndexBase + {offset}`, return the offset else None."""
m = self.varIndexBasePlusOffsetRE.search(self.description)
if not m:
return None
return int(m.group(1))
class SimpleValue(BaseConverter): class SimpleValue(BaseConverter):
@staticmethod @staticmethod
@ -696,8 +708,10 @@ class FeatureParams(Table):
class ValueFormat(IntValue): class ValueFormat(IntValue):
staticSize = 2 staticSize = 2
def __init__(self, name, repeat, aux, tableClass=None): def __init__(self, name, repeat, aux, tableClass=None, *, description=""):
BaseConverter.__init__(self, name, repeat, aux, tableClass) BaseConverter.__init__(
self, name, repeat, aux, tableClass, description=description
)
self.which = "ValueFormat" + ("2" if name[-1] == "2" else "1") self.which = "ValueFormat" + ("2" if name[-1] == "2" else "1")
def read(self, reader, font, tableDict): def read(self, reader, font, tableDict):
format = reader.readUShort() format = reader.readUShort()
@ -730,8 +744,10 @@ class ValueRecord(ValueFormat):
class AATLookup(BaseConverter): class AATLookup(BaseConverter):
BIN_SEARCH_HEADER_SIZE = 10 BIN_SEARCH_HEADER_SIZE = 10
def __init__(self, name, repeat, aux, tableClass): def __init__(self, name, repeat, aux, tableClass, *, description=""):
BaseConverter.__init__(self, name, repeat, aux, tableClass) BaseConverter.__init__(
self, name, repeat, aux, tableClass, description=description
)
if issubclass(self.tableClass, SimpleValue): if issubclass(self.tableClass, SimpleValue):
self.converter = self.tableClass(name='Value', repeat=None, aux=None) self.converter = self.tableClass(name='Value', repeat=None, aux=None)
else: else:
@ -1029,8 +1045,10 @@ class MorxSubtableConverter(BaseConverter):
val: key for key, val in _PROCESSING_ORDERS.items() val: key for key, val in _PROCESSING_ORDERS.items()
} }
def __init__(self, name, repeat, aux): def __init__(self, name, repeat, aux, tableClass=None, *, description=""):
BaseConverter.__init__(self, name, repeat, aux) BaseConverter.__init__(
self, name, repeat, aux, tableClass, description=description
)
def _setTextDirectionFromCoverageFlags(self, flags, subtable): def _setTextDirectionFromCoverageFlags(self, flags, subtable):
if (flags & 0x20) != 0: if (flags & 0x20) != 0:
@ -1150,8 +1168,10 @@ class MorxSubtableConverter(BaseConverter):
# https://developer.apple.com/fonts/TrueType-Reference-Manual/RM06/Chap6Tables.html#ExtendedStateHeader # https://developer.apple.com/fonts/TrueType-Reference-Manual/RM06/Chap6Tables.html#ExtendedStateHeader
# TODO: Untangle the implementation of the various lookup-specific formats. # TODO: Untangle the implementation of the various lookup-specific formats.
class STXHeader(BaseConverter): class STXHeader(BaseConverter):
def __init__(self, name, repeat, aux, tableClass): def __init__(self, name, repeat, aux, tableClass, *, description=""):
BaseConverter.__init__(self, name, repeat, aux, tableClass) BaseConverter.__init__(
self, name, repeat, aux, tableClass, description=description
)
assert issubclass(self.tableClass, AATAction) assert issubclass(self.tableClass, AATAction)
self.classLookup = AATLookup("GlyphClasses", None, None, UShort) self.classLookup = AATLookup("GlyphClasses", None, None, UShort)
if issubclass(self.tableClass, ContextualMorphAction): if issubclass(self.tableClass, ContextualMorphAction):