Skip to content
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

chore!: deprecate any_color_converted_view #678

Merged

Conversation

striezel
Copy link
Contributor

Description

color_converted_view has the same implementation and can be used as a replacement, providing the same functionality.

References

This builds on #660 by @marco-langer.
Since there needs to be a deprecation notice first before removing a feature, here it is.

Due to the fact that the switch to C++14 has been announced in #677, we can now officially use the [[deprecated]] attribute which was standardized in C++14.

Tasklist

  • Ensure all CI builds pass
  • Review and approve

color_converted_view has the same implementation and can be used
as a replacement, providing the same functionality.
Copy link
Member

@mloskot mloskot left a comment

Choose a reason for hiding this comment

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

Looks good!

This is initial part of deprecating the any_color_converted_view and as @marco-langer pointed in #660 (comment) there are more changes to follow.

@mloskot mloskot merged commit 6d312af into boostorg:develop May 26, 2022
@mloskot mloskot added this to the Boost 1.80 milestone May 26, 2022
@mloskot mloskot added cat/deprecation Deprecate a feature or functionality ext/dynamic_image boost/gil/extension/dynamic_image labels May 26, 2022
@striezel striezel deleted the deprecate-any_color_converted_view branch May 26, 2022 22:14
@mloskot mloskot mentioned this pull request Jul 5, 2022
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cat/deprecation Deprecate a feature or functionality ext/dynamic_image boost/gil/extension/dynamic_image
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants