paulirwin commented on PR #1055: URL: https://github.com/apache/lucenenet/pull/1055#issuecomment-2514892341
> I wonder, could [Spatial4n/Spatial4n#1](https://github.com/Spatial4n/Spatial4n/issues/1) be a similar issue as this (overflow)? I don't think so, since it's not an addition that overflows Int32. In that case, I think it could be an out of range character in the geohash. Without trying out the code, just from thinking through it, it seems like characters above the ASCII alphanumeric set could cause that issue, such as `'Ä'` (int value 196) which would result in a calculated index of `(int)'Ä' - (int)'0'` == 148 which is out of range. Just a thought there, if the caller is not validating that the input is a valid geohash. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@lucenenet.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
