Fix variable name, add and remove some comments

src/fontra_glyphs/backend.py

@@ -293,6 +293,7 @@ async def getGlyph(self, glyphName: str) -> VariableGlyph | None:
         gsLayers = sorted(
             gsLayers, key=lambda i_gsLayer: masterOrder[i_gsLayer[1].associatedMasterId]
         )
+
         seenLocations = []
         for i, gsLayer in gsLayers:
             braceLocation = self._getBraceLayerLocation(gsLayer)
@@ -810,9 +811,7 @@ def fontraLayerToGSLayer(layer, gsLayer):
     gsLayer.components = [
         fontraComponentToGSComponent(component) for component in layer.glyph.components
     ]
-    gsLayer.anchors = [
-        fontraAnchorsToGSAnchor(anchor) for anchor in layer.glyph.anchors
-    ]
+    gsLayer.anchors = [fontraAnchorToGSAnchor(anchor) for anchor in layer.glyph.anchors]
 
 
 def fontraComponentToGSComponent(component):
@@ -824,7 +823,7 @@ def fontraComponentToGSComponent(component):
     return gsComponent
 
 
-def fontraAnchorsToGSAnchor(anchor):
+def fontraAnchorToGSAnchor(anchor):
     gsAnchor = glyphsLib.classes.GSAnchor()
     gsAnchor.name = anchor.name
     gsAnchor.position.x = anchor.x

src/fontra_glyphs/utils.py

@@ -40,11 +40,10 @@ def getAssociatedMasterId(gsGlyph, gsLocation):
 
 
 def gsFormatting(content):
-    # openstep_plist.dump changes the whole formatting, therefore
-    # it's very diffucute to see what has changed.
-    # This function is a very bad try to get close to how the formatting
-    # looks like for a .glyphs file.
-    # There must be a better solution, but this is better than nothing.
+    # TODO: This need a different solution.
+    # Should be solved in the raw data not via regular expressions.
+    # The raw data is made out of list. We need to convert some part into tuples.
+    # For more please see: https://github.com/fonttools/openstep-plist/issues/33
 
     patterns = [
         (

tests/test_utils.py

@@ -36,6 +36,8 @@ async def test_getLocationFromSources(testFont):
 
 
 def test_getAssociatedMasterId(testGSFont):
+    # TODO: need more complex test with at least two axes,
+    # then improvement getAssociatedMasterId
     gsGlyph = testGSFont.glyphs["a"]
     associatedMasterId = getAssociatedMasterId(gsGlyph, [155])
     associatedMaster = gsGlyph.layers[associatedMasterId]