Bug 140401 – Make WTF::Optional work with msvc 2013

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Template Framework

[Chris Dumez]

Make WTF::Optional work with mdvc 2013 by using std::aligned_memory instead of an unrestricted union internally. MSVC 2013 still does not support unrestricted unions.

[Chris Dumez]

Created attachment 244531 [details]
Patch

[Anders Carlsson]

Comment on attachment 244531 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=244531&action=review

> Source/WTF/wtf/Optional.h:65
> +            new (&m_value) T(*other.asPtr());

I still think you want NotNull here.

> Source/WTF/wtf/Optional.h:72
> +            new (&m_value) T(WTF::move(*other.asPtr()));

I still think you want NotNull here.

> Source/WTF/wtf/Optional.h:78
> +        new (&m_value) T(WTF::move(value));

NotNull.

> Source/WTF/wtf/Optional.h:85
> +        new (&m_value) T(std::forward<Args>(args)...);

NotNull.

> Source/WTF/wtf/Optional.h:106
> +            new (&m_value) T(*other.asPtr());

NotNull.

> Source/WTF/wtf/Optional.h:-138
> -    template<typename U>
> -    Optional& operator=(U&& u)

What happened to this overload?

> Source/WTF/wtf/Optional.h:119
> +            new (&m_value) T(WTF::move(*other.asPtr()));

NotNull.

[Chris Dumez]

Created attachment 244535 [details]
Patch

[Chris Dumez]

Comment on attachment 244531 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=244531&action=review

>> Source/WTF/wtf/Optional.h:65
>> +            new (&m_value) T(*other.asPtr());
> 
> I still think you want NotNull here.

Right, done.

>> Source/WTF/wtf/Optional.h:-138
>> -    Optional& operator=(U&& u)
> 
> What happened to this overload?

I had trouble getting it to build initially and forgot to add it back. It is back in.

[WebKit Commit Bot]

Comment on attachment 244535 [details]
Patch

Clearing flags on attachment: 244535

Committed r178372: <http://trac.webkit.org/changeset/178372>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.