NightOwl888 commented on PR #838: URL: https://github.com/apache/lucenenet/pull/838#issuecomment-1518921816
> Figured out what's up with the NextInt64 call, I removed the using for extension method as my IDE showed it as not being used, but as we can see, that gets used under different .net framework versions. I will have to keep that in mind. Actually, since we consistently use J2N.Randomizer, it will call the overridden NextInt64() since it is a subclass of System.Random. The concrete method takes precedence over the extension method. But as previously mentioned, the extension method currently has a different implementation than the concrete method, so it will vary depending on whether dotnet supplies the method or not. -- 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
