-
Notifications
You must be signed in to change notification settings - Fork 36.2k
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
Refactor: Stop using namespace std (bitcoin-cli/-tx). #9220
Conversation
…ppropriate places.
Yes, this has to be done at some point. |
Concept ACK |
ACK 2f2625a Good candidate for same-binary check. |
same-binaries ACK 2f2625a |
utACK 2f2625a |
2f2625a Removed using namespace std from bitcoin-cli/-tx and added std:: in appropriate places. (Karl-Johan Alm)
@MarcoFalke I can't find any resources on same-binaries checking. I would like to make a more encompassing commit that deals with all these, but the diff is rather big so I'd like to do the same-binaries check beforehand myself. |
@kallewoof Have a look at this repo https://github.com/laanwj/bitcoin-maintainer-tools |
Ahh, thanks a lot! |
…tx). 2f2625a Removed using namespace std from bitcoin-cli/-tx and added std:: in appropriate places. (Karl-Johan Alm)
CLI binary improvements Cherry-picked from the following upstream PRs: - bitcoin/bitcoin#5936 - bitcoin/bitcoin#7550 - bitcoin/bitcoin#7989 - bitcoin/bitcoin#7957 - bitcoin/bitcoin#9067 - bitcoin/bitcoin#9220 Excludes any changes that affected the QT code.
…tx). 2f2625a Removed using namespace std from bitcoin-cli/-tx and added std:: in appropriate places. (Karl-Johan Alm)
……tx). 2f2625a Removed using namespace std from bitcoin-cli/-tx and added std:: in appropriate places. (Karl-Johan Alm)
According to the "Source code organization" section in Developer notes,
using namespace
should be avoided. This commit removesusing namespace std
from bitcoin-cli/-tx and adds thestd::
prefix in appropriate places.