Skip to content

Commit 903ef40

Browse files
committed
minor cleanup and documentation fixes
1 parent a8e7db7 commit 903ef40

File tree

1 file changed

+5
-8
lines changed

1 file changed

+5
-8
lines changed

goadbWriter.py

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -202,12 +202,9 @@ def get_uni_name(cp):
202202
def get_uni_override(cp_list):
203203
'''
204204
comma-separated Unicode override string
205+
(or a single string if len(cp_list) == 1)
205206
'''
206-
if len(cp_list) == 1:
207-
unicode_override = get_uni_name(cp_list[0])
208-
else:
209-
# more than one codepoint
210-
unicode_override = ','.join([get_uni_name(cp) for cp in cp_list])
207+
unicode_override = ','.join([get_uni_name(cp) for cp in cp_list])
211208
return unicode_override
212209

213210

@@ -216,7 +213,7 @@ def make_unique_final_name(gname):
216213
Since final glyph names need to be sanitized, a duplication of final glyph
217214
names is possible. This adds a 4-digit index to the glyph name.
218215
219-
If the glyph name has already a 4-digit index, the index is incremented.
216+
If the glyph name already has a 4-digit index, the index is incremented.
220217
'''
221218

222219
# glyph name already has an index
@@ -285,9 +282,9 @@ class GlyphBaptism(object):
285282
Not dealing with ligatures/alternates here.
286283
287284
Either
288-
- glyph name is in the AGD, and makeotf finds the code point that way
285+
- glyph name is in the AGD dict, makeotf associates the code point
289286
- glyph name is not in the AGD, but the glyph has attached code point(s)
290-
- glyph name implies a code point
287+
- glyph name implies a code point (uniXXXX or uXXXXX)
291288
- glyph is not encoded
292289
293290
'''

0 commit comments

Comments
 (0)