Skip to content

Commit 89dff13

Browse files
committed
Follow-up build fix for r231223
https://bugs.webkit.org/show_bug.cgi?id=185159 <rdar://problem/39891074> Unreviewed build fix. We should be consistent about when we include <optional> vs define our own. * wtf/Optional.h: Canonical link: https://commits.webkit.org/200695@main git-svn-id: https://svn.webkit.org/repository/webkit/trunk@231251 268f45cc-cd09-0410-ab3c-d52691b4dbfc
1 parent 0b6074b commit 89dff13

2 files changed

Lines changed: 13 additions & 5 deletions

File tree

Source/WTF/ChangeLog

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,15 @@
1+
2018-05-02 Jonathan Bedard <jbedard@apple.com>
2+
3+
Follow-up build fix for r231223
4+
https://bugs.webkit.org/show_bug.cgi?id=185159
5+
<rdar://problem/39891074>
6+
7+
Unreviewed build fix.
8+
9+
We should be consistent about when we include <optional> vs define our own.
10+
11+
* wtf/Optional.h:
12+
113
2018-05-02 Jer Noble <jer.noble@apple.com>
214

315
Enable HAVE_AVKIT for the MINIMAL_SIMULATOR

Source/WTF/wtf/Optional.h

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -47,13 +47,9 @@
4747
# include <wtf/Compiler.h>
4848
# include <wtf/StdLibExtras.h>
4949

50-
#if !COMPILER(MSVC) && __has_include(<optional>)
51-
# include <optional>
52-
#endif
53-
5450
#if !COMPILER(MSVC) && defined(__cpp_lib_optional) && __cpp_lib_optional >= 201603
5551

56-
// Use default std::optional.
52+
# include <optional>
5753

5854
#else
5955

0 commit comments

Comments
 (0)