Github user dsmiley commented on a diff in the pull request: https://github.com/apache/lucene-solr/pull/323#discussion_r172258894 --- Diff: lucene/core/src/java/org/apache/lucene/geo/GeoEncodingUtils.java --- @@ -127,6 +130,10 @@ public static double decodeLatitude(int encoded) { public static double decodeLatitude(byte[] src, int offset) { return decodeLatitude(NumericUtils.sortableBytesToInt(src, offset)); } + + public static double decodeLatitudeCeil(long encoded) { --- End diff -- @nknize could you please review this part? It's not clear to me why GeoEncodingUtils has dual ceil/floor for encode but not for decode.
--- --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org