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
I think Match could be improved in the following way:
i and j currently indicate character positions. Instead, I think it would be better to replace them by a range that instead can be used to slice the string, i.e. changing them to byte indexes.
guesses is always set in the public API. For that reason I think the publicly exposed type should not use an Option for it.
The text was updated successfully, but these errors were encountered:
I think
Match
could be improved in the following way:i
andj
currently indicate character positions. Instead, I think it would be better to replace them by a range that instead can be used to slice the string, i.e. changing them to byte indexes.guesses
is always set in the public API. For that reason I think the publicly exposed type should not use anOption
for it.The text was updated successfully, but these errors were encountered: