Skip to content

Conversation

@maflcko
Copy link
Member

@maflcko maflcko commented Aug 15, 2022

Not sure why this was put there, so remove it

Copy link
Contributor

@aureleoules aureleoules left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK fac09f4.
Verified that std::string() + is not needed.

@fanquake fanquake merged commit d9cd8b4 into bitcoin:master Aug 15, 2022
@maflcko maflcko deleted the 2208-doc-str-🤑 branch August 15, 2022 12:35
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Aug 15, 2022
…tring

fac09f4 refactor: Remove not needed empty RPC doc std::string (MacroFake)

Pull request description:

  Not sure why this was put there, so remove it

ACKs for top commit:
  aureleoules:
    ACK fac09f4.

Tree-SHA512: 7fae9c612dbb82db39e3dc6e20cc7d0c8128f4abf27e34f4854f908dc1bf3ad48ebee2dbbb864a9ce95e98c5d57119bf976793945a2084a43d137c2d95a25ea1
@bitcoin bitcoin locked and limited conversation to collaborators Aug 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants