NightOwl888 commented on code in PR #897: URL: https://github.com/apache/lucenenet/pull/897#discussion_r1457957008
########## src/Lucene.Net.Tests/Util/Fst/Test2BFST.cs: ########## @@ -1,11 +1,14 @@ using Lucene.Net.Support; using NUnit.Framework; -using RandomizedTesting.Generators; using System; using Assert = Lucene.Net.TestFramework.Assert; using Console = Lucene.Net.Util.SystemConsole; using Int64 = J2N.Numerics.Int64; +#if !NET6_0_OR_GREATER Review Comment: I was wrong. Since we have our own subclass of `System.Random`, our implementation is what is being used. So, this is just about importing the namespace. Let's add a new feature `FEATURE_RANDOM_NEXTINT64_NEXTSINGLE` to cover this so we don't have to resort to `!NET6_0_OR_GREATER`. `NextInt64()` and `NextSingle()` were added at the same time in .NET 6.0, so we can group them. -- 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
