Re: [boost] [optional] Changes in Boost.Optional

Subject: Re: [boost] [optional] Changes in Boost.Optional
From: Andrzej Krzemienski (akrzemi1_at_[hidden])
Date: 2014-09-02 03:56:44

2014-09-01 17:48 GMT+02:00 Dean Michael Berris <mikhailberis_at_[hidden]>:

> On Tue Aug 26 2014 at 3:59:12 PM Dean Michael Berris <
> mikhailberis_at_[hidden]>
> wrote:
>
> > On Tue Aug 26 2014 at 9:38:37 AM Eric Niebler <eniebler_at_[hidden]>
> wrote:
> >
> >> On 08/24/2014 11:20 AM, Rob Stewart wrote:
> >> > [Corrected subject to target optional.]
> >> >
> >> > On August 24, 2014 7:55:11 AM EDT, Dean Michael Berris <
> >> mikhailberis_at_[hidden]> wrote:
> >> >> I realize that this message may have not passed the spam filters.
> >> >>
> >> >> Can someone comment on the changes in Boost.Optional in 1.56.0
> breaking
> >> >> existing code?
> >>
> >> <snip>
> >>
> >> Hi Dean,
> >>
> >> I don't know the specifics, but if you file a bug, at least it won't get
> >> lost.
> >>
> >>
> > Filed: https://svn.boost.org/trac/boost/ticket/10399
> >
> > Thanks Eric.
> >
>
> It's now been almost a week and there's been no update on the issue. I
> think this is serious enough that I'm positive it couldn't just be me
> that's experiencing this regression.
>
> Can someone with access to MSVC 2010 confirm whether this indeed is an
> issue, and whether it's actually a regression?
>
> If I wanted to get a fix in for this, whom do I send the pull request to
> (community maintenance team) so it gets fixed for 1.57?
>

Hi Dean,
I am a maintainer of Boost.Optional. Was on vacation so couldn't hear you.
I just verified that your code example works with 1.55 on VS2010, so it is
a regression. I will not be able to have a look at it for the next couple
of days. I would like to determine why it even works on 1.55. The
converting constructor from arbitrary U is also explicit in 1.55, so it
works because of some sequence of conversions that I cannot figure out
right now.

If you wish to submit a patch you can do it via GitHub at
https://github.com/boostorg/optional. I can see and act upon the pull
requests there.

Regards,
&rzej