Skip to content

Commit

Permalink
Removing toOrderedDict, not necessary anymore
Browse files Browse the repository at this point in the history
because it's done by Just's openstep-plist
  • Loading branch information
ollimeier committed Jan 23, 2025
1 parent d41b0b2 commit 2161e6d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 15 deletions.
5 changes: 2 additions & 3 deletions src/fontra_glyphs/backend.py
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,6 @@
getAssociatedMasterId,
getLocationFromSources,
matchTreeFont,
toOrderedDict,
)

rootInfoNames = [
Expand Down Expand Up @@ -115,8 +114,8 @@ def _setupFromPath(self, path: PathLike) -> None:
self.gsFont.glyphs = [
glyphsLib.classes.GSGlyph() for i in range(len(rawGlyphsData))
]
self.rawFontData = toOrderedDict(rawFontData)
self.rawGlyphsData = toOrderedDict(rawGlyphsData)
self.rawFontData = rawFontData
self.rawGlyphsData = rawGlyphsData

self.glyphNameToIndex = {
glyphData["glyphname"]: i for i, glyphData in enumerate(rawGlyphsData)
Expand Down
12 changes: 0 additions & 12 deletions src/fontra_glyphs/utils.py
Original file line number Diff line number Diff line change
@@ -1,16 +1,4 @@
import re
from collections import OrderedDict


# The following is obsolete once this is merged:
# https://github.com/fonttools/openstep-plist/pull/35
def toOrderedDict(obj):
if isinstance(obj, dict):
return OrderedDict({k: toOrderedDict(v) for k, v in obj.items()})
elif isinstance(obj, list):
return [toOrderedDict(item) for item in obj]
else:
return obj


def getLocationFromSources(sources, layerName):
Expand Down

0 comments on commit 2161e6d

Please sign in to comment.