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): 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 """count maps (locality, street name) to a count. Returns, per locality, rows for the per_locality names counted
name, each row holding its count under column.""" most, ties broken by name; each row holds its count under column."""
of = collections.defaultdict(list) of = collections.defaultdict(list)
for (locality, name), n in count.items(): for (locality, name), n in count.items():
of[locality].append((n, name)) of[locality].append((n, name))
+3 -2
View File
@@ -13,8 +13,9 @@ const reservedInIdentifier = ".|({}/[]\""
var reservedList = strings.Join(strings.Split(reservedInIdentifier, ""), " ") var reservedList = strings.Join(strings.Split(reservedInIdentifier, ""), " ")
// An identifier names a folder, category, field, column or bound name. CheckIdentifier rejects one // CheckIdentifier rejects an identifier that the dot path, {token} and JSON grammars cannot spell,
// that the dot path, {token} and JSON grammars cannot spell, or that a struct tag cannot read. // or that a struct tag cannot read. An identifier names a folder, category, field, column or bound
// name.
func CheckIdentifier(name string) error { func CheckIdentifier(name string) error {
if name == "" { if name == "" {
return fmt.Errorf("%q is empty, which is not a path segment, so List never offers it", name) return fmt.Errorf("%q is empty, which is not a path segment, so List never offers it", name)