Re: [boost] [modularization] Recommendations

Subject: Re: [boost] [modularization] Recommendations
From: Antony Polukhin (antoshkka_at_[hidden])
Date: 2014-06-09 05:31:41

2014-06-08 15:44 GMT+04:00 Peter Dimov <lists_at_[hidden]>:

> Thanks, Stephen, for the updated recommendations.
>
>
> Stephen Kelly wrote:
>
>  Recommendation 2) Move implicit_cast.hpp and cast.hpp from conversion to
>> core
>>
>> This also requires moving select_type.hpp from detail to core.
>>
>
> Actually, select_type.hpp is not being used in cast.hpp at all, as far as
> I can see. The include can just be removed.
>
> cast.hpp probably also needs to die, the polymorphic casts going into
> their own header. This will be post-release because it will break code,
> though.
>
> Inclusion report for <boost/cast.hpp> (in module conversion):
>
>    from gil:
>        <boost/gil/extension/toolbox/color_spaces/hsv.hpp>
>        <boost/gil/extension/toolbox/color_spaces/lab.hpp>
>        <boost/gil/extension/toolbox/color_spaces/xyz.hpp>
>        <boost/gil/extension/toolbox/color_spaces/ycbcr.hpp>
>
>    from range:
>        <boost/range/adaptor/type_erased.hpp>
>        <boost/range/any_range.hpp>
>        <boost/range/detail/any_iterator.hpp>
>        <boost/range/detail/any_iterator_wrapper.hpp>
>
>    from statechart:
>        <boost/statechart/custom_reaction.hpp>
>        <boost/statechart/detail/reaction_dispatcher.hpp>
>        <boost/statechart/event.hpp>
>        <boost/statechart/simple_state.hpp>
>        <boost/statechart/state_machine.hpp>
>

Made pull requests for those libraries to use polymorphic_cast.hpp instead
of cast.hpp:
https://github.com/boostorg/gil/pull/9
https://github.com/boostorg/statechart/pull/1
https://github.com/boostorg/range/pull/10

Moved cast.hpp to the NumericCast library:
https://github.com/boostorg/numeric_conversion/pull/2

-- 
Best regards,
Antony Polukhin