Answer prose round 2: top_streets says what it returns, and CheckIdentifier's doc opens with what it does
This commit is contained in:
+2
-2
@@ -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))
|
||||||
|
|||||||
@@ -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)
|
||||||
|
|||||||
Reference in New Issue
Block a user