-
Notifications
You must be signed in to change notification settings - Fork 163
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: Renamed io/dynamic_io_new.hpp to io/detail/dynamic.hpp #653
refactor: Renamed io/dynamic_io_new.hpp to io/detail/dynamic.hpp #653
Conversation
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.
LGTM, thank you!
I guess I am a bit late, but this commit seems to be the one breaking the build for me. In all of the |
|
The error was due to missing dynamic.hpp (renamed in #653) header that contained dynamic_io_fnobj. Adding the header back in to write.hpp headers of io extensions fixed the problem.
s/construct_method/construct_matched/ boostorg/gil#653
That typo was already fixed for the Boost website in commit <boostorg/website@be79716>, but the change has not made it back to GIL - yet. [ci skip]
That typo was already fixed for the Boost website in commit <boostorg/website@be79716>, but the change has not made it back to GIL - yet. [ci skip]
Description
Renamed
io/dynamic_io_new.hpp
toio/detail/dynamic.hpp
.The function
construct_matched
was only used by other detail headers, thus I moved the function into the detail namespace as already suggested in the issue. Additionally I moved the whole file into the detail directory, because it only contains details now.References
Fixes #189
Tasklist