-
Notifications
You must be signed in to change notification settings - Fork 717
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
[Refactoring] Lightweight abstraction of boost::filesystem #1630
[Refactoring] Lightweight abstraction of boost::filesystem #1630
Conversation
f8ab778
to
b4df9ae
Compare
backports bitcoin/bitcoin@19e36bb
This is step one in abstracting the use of boost::filesystem. backports bitcoin/bitcoin@7d5172d
Step two in abstracting away boost::filesystem. To repeat this, simply run: ``` git ls-files \*.cpp \*.h | xargs sed -i 's/boost::filesystem/fs/g' ``` backports bitcoin/bitcoin@bac5c9c
Abstracts away how a path is opened to a `FILE*`. Reduces the number of places where path is converted to a string for anything else but printing. backports bitcoin/bitcoin@2a5f574
Having these inside functions is silly and redundant now. backports bitcoin/bitcoin@f110272
b4df9ae
to
286aa99
Compare
Rebased after #1629 merge. Ready for review/QA. |
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.
Code review ACK 286aa99.
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.
ACK 286aa99.
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.
ACK 286aa99
Based on top of:
Backports bitcoin#9902