-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Solidity 0.7 VRF Consumer Contracts 7️⃣ #4131
Conversation
I see that you haven't updated any CHANGELOG files. Would it make sense to do so? |
* @dev compatibility with older versions. It can't *hurt* to mix in some of | ||
* @dev your own randomness, here, but it's not necessary because the VRF | ||
* @dev oracle will mix the hash of the block containing your request into the | ||
* @dev VRF seed it ultimately uses. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So if I don't care about it, should I just pass 0?
// This is the seed passed to VRFCoordinator. The oracle will mix this with | ||
// the hash of the block containing this request to obtain the seed/input | ||
// which is finally passed to the VRF cryptographic machinery. | ||
uint256 vRFSeed = makeVRFInputSeed(_keyHash, _seed, address(this), nonces[_keyHash]); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This comment seems outdated considering the description
No description provided.