have _brace_coordinates() consistently method return a list

it was returning a generator expression for format_version>2 or a list for format_version=2 which doesn't make any sense #971 (comment)
googlefonts · Feb 2, 2024 · a944fd3 · a944fd3
1 parent 90f10c4
commit a944fd3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/Lib/glyphsLib/builder/transformations/intermediate_layers.py b/Lib/glyphsLib/builder/transformations/intermediate_layers.py
@@ -40,7 +40,7 @@ def ensure_component_has_sparse_layer(font, component, parent_layer):
  tags = [axis.axisTag for axis in font.axes]
  master_locations = [x.axes for x in font.masters]
  _, limits = variation_model(font, master_locations)
- location = tuple(parent_layer._brace_coordinates())
+ location = parent_layer._brace_coordinates()
  default_location = get_regular_master(font).axes
  normalized_location = {
  axisTag: normalizeValue(

diff --git a/Lib/glyphsLib/classes.py b/Lib/glyphsLib/classes.py
@@ -3976,7 +3976,7 @@ def _brace_coordinates(self):
  return None
 
  if self.parent.parent.format_version > 2:
- return (float(v) for v in self.attributes["coordinates"]) # Glyphs 3
+ return [float(v) for v in self.attributes["coordinates"]] # Glyphs 3
 
  # Glyphs 2
  name = self.name