Answer prose round 2: top_streets says what it returns, and CheckIdentifier's doc opens with what it does

This commit is contained in:
2026-10-07 02:23:13 +02:00
committed by Lillem4n
parent c2147d2282
commit ad599675d1
2 changed files with 5 additions and 4 deletions
+2 -2
View File
@@ -64,8 +64,8 @@ class Nearest:
def top_streets(count, per_locality, column):
"""count maps (locality, street name) to a count. Per locality, the per_locality names counted most, ties broken by
name, each row holding its count under column."""
"""count maps (locality, street name) to a count. Returns, per locality, rows for the per_locality names counted
most, ties broken by name; each row holds its count under column."""
of = collections.defaultdict(list)
for (locality, name), n in count.items():
of[locality].append((n, name))
+3 -2
View File
@@ -13,8 +13,9 @@ const reservedInIdentifier = ".|({}/[]\""
var reservedList = strings.Join(strings.Split(reservedInIdentifier, ""), " ")
// An identifier names a folder, category, field, column or bound name. CheckIdentifier rejects one
// that the dot path, {token} and JSON grammars cannot spell, or that a struct tag cannot read.
// CheckIdentifier rejects an identifier that the dot path, {token} and JSON grammars cannot spell,
// or that a struct tag cannot read. An identifier names a folder, category, field, column or bound
// name.
func CheckIdentifier(name string) error {
if name == "" {
return fmt.Errorf("%q is empty, which is not a path segment, so List never offers it", name)