Skip Menu |

This queue is for tickets about the GeoHash CPAN distribution.

Report information
The Basics
Id: 79933
Status: resolved
Priority: 0/
Queue: GeoHash

People
Owner: Nobody in particular
Requestors: dirkus [...] yopmail.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Don't unecessarily create new top-level namespaces
Instead of creating a new top-level namespace, you could put this under Geo::Hash::Any. There are many existing examples of similar modules, like JSON::Any, YAML::Any, Cone::Any, etc.
GeoHash official name is not Geo::Hash. It should not be separated by *::*. And *::Any name- space is not preferable. On 2012-9月-30 日 13:50:21, dirk wrote: Show quoted text
> Instead of creating a new top-level namespace, you could put this under > Geo::Hash::Any. There are many existing examples of similar modules, > like JSON::Any, YAML::Any, Cone::Any, etc.
BTW, I shipped the Geohash 0.02 to update the document. https://metacpan.org/module/YAPPO/Geohash-0.02/lib/Geohash.pm#DESCRIPTION