wallet: Simplify logic in CCryptoKeyStore::AddKeyPubKey(...) #11682

pull practicalswift wants to merge 1 commits into bitcoin:master from practicalswift:CCryptoKeyStore-AddKeyPubKey changing 1 files +1 −4
  1. practicalswift commented at 9:11 AM on November 14, 2017: contributor

    Simplify logic in CCryptoKeyStore::AddKeyPubKey(...).

  2. wallet: Simplify logic in CCryptoKeyStore::AddKeyPubKey(...) d4761db2dd
  3. eklitzke commented at 9:14 AM on November 14, 2017: contributor

    drive by review but, lgtm

  4. promag commented at 9:58 AM on November 14, 2017: member

    ACK d4761db.

  5. fanquake added the label Wallet on Nov 14, 2017
  6. MarcoFalke commented at 4:07 PM on November 14, 2017: member

    This pattern is used throughout the code base. Fixing a single instance does not really help. Also, not everyone might agree that this "simplifies the logic".

    Closing the refactoring pull according to the developer notes.

  7. MarcoFalke closed this on Nov 14, 2017
  8. ryanofsky commented at 4:11 PM on November 14, 2017: member

    ACK d4761db2dd77f5924e6b83b6206164892f98465b

  9. practicalswift deleted the branch on Apr 10, 2021
  10. DrahtBot locked this on Aug 16, 2022
Contributors
practicalswifteklitzkepromagMarcoFalkeryanofsky
Labels
Wallet
github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-16 15:15 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me