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
5-10% of performance drop is a bit too much for noise even with unpinned frequency considering that criterion does a proper CPU warm-up. I will try to take a closer look at the RNG implementation later.
Should we mark it as deprecated? Certainly not now (no final release of chacha20 yet).
Later? I don't see much point since it won't be compatible with the next rand_core version. Also, I don't see any need to force people to migrate (no known security issues or other major problems).
I think we can move rand_chacha to the rngs repo and release v0.10 with the new rand_core. Next, after chacha20 is released and assuming there are no problems with it, we can release empty rand_chacha v0.11 with suggestion to migrate to chacha20, thus effectively deprecating rand_chacha. Those who track outdated dependencies will migrate, those who don't care will stay on v0.10.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Source moved: rust-random/rngs#68