Commit a036c64c authored by Amin Mesbah's avatar Amin Mesbah
Browse files

Use geohash limit defines in constraint check

Slight robustness improvement, especially if the limit values are
changed, as was suggested in antires/redis#4291 [1].

[1] https://github.com/antirez/redis/pull/4291
parent febe102b
...@@ -127,8 +127,8 @@ int geohashEncode(const GeoHashRange *long_range, const GeoHashRange *lat_range, ...@@ -127,8 +127,8 @@ int geohashEncode(const GeoHashRange *long_range, const GeoHashRange *lat_range,
/* Return an error when trying to index outside the supported /* Return an error when trying to index outside the supported
* constraints. */ * constraints. */
if (longitude > 180 || longitude < -180 || if (longitude > GEO_LONG_MAX || longitude < GEO_LONG_MIN ||
latitude > 85.05112878 || latitude < -85.05112878) return 0; latitude > GEO_LAT_MAX || latitude < GEO_LAT_MIN) return 0;
hash->bits = 0; hash->bits = 0;
hash->step = step; hash->step = step;
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment