Skip to content
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

doc: fix XOnlyPubKey::IsFullyValid comment reference #23400

Merged

Conversation

theStack
Copy link
Contributor

The method name CreatePayToContract doesn't exist, very likely it was a (local) working title that was renamed to CreateTapTweak later.

@DrahtBot DrahtBot added the Docs label Oct 31, 2021
@fanquake fanquake requested a review from sipa November 8, 2021 22:45
src/pubkey.h Outdated Show resolved Hide resolved
The method name `CreatePayToContract` doesn't exist, very likely it was
a (local) working title that was renamed to `CreateTapTweak` later.
Also mention `CheckTapTweak`.
@theStack theStack force-pushed the 202110-doc-fix_xonlypubkey_isvalid_comment branch from 1836a9b to 037c9ee Compare November 9, 2021 11:40
@michaelfolkson
Copy link
Contributor

ACK 037c9ee

This is the only reference to CreatePayToContract in the codebase.

@sipa
Copy link
Member

sipa commented Nov 9, 2021

ACK 037c9ee

@maflcko maflcko merged commit 8ae4ba4 into bitcoin:master Nov 9, 2021
@theStack theStack deleted the 202110-doc-fix_xonlypubkey_isvalid_comment branch November 9, 2021 17:30
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Nov 10, 2021
…erence

037c9ee fix `XOnlyPubKey::IsFullyValid` comment reference (Sebastian Falbesoner)

Pull request description:

  The method name `CreatePayToContract` doesn't exist, very likely it was a (local) working title that was renamed to `CreateTapTweak` later.

ACKs for top commit:
  michaelfolkson:
    ACK 037c9ee
  sipa:
    ACK 037c9ee

Tree-SHA512: ab2b6ca93b66aba83c725b62f39e9f9316f3bea6f75ef35f66a2ac18b22a0a69ff2069cadab0365b28b0af2d30ad5ee3d3022194ac2024a1cdbb81d106fca0bb
@bitcoin bitcoin locked and limited conversation to collaborators Nov 9, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants