You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This cast from uint to int is necessary because of the Take operation we use inside the shuffler. This could be a - albeit very theoretical - future problem when we introduce custom minefields since one could define a minecount that's greater than an int.
For now maybe we should throw an ArgumentException in the constructor of the Minefield in case someone tries to generate it with a minecount greater than int.MaxValue?
This cast from uint to int is necessary because of the
Take
operation we use inside the shuffler. This could be a - albeit very theoretical - future problem when we introduce custom minefields since one could define a minecount that's greater than an int.For now maybe we should throw an
ArgumentException
in the constructor of theMinefield
in case someone tries to generate it with a minecount greater thanint.MaxValue
?Originally posted by @dev-in-disguise in #56 (comment)
The text was updated successfully, but these errors were encountered: