Commit 7928f578 authored by Amin Mesbah's avatar Amin Mesbah Committed by antirez
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 bb2bed78
...@@ -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