Bug 227166 – [LFC][WPE] Build failure with GCC 8.x

• Component: WPE WebKit
• Status: RESOLVED FIXED

[Carlos Alberto Lopez Perez]

When building the WPE port with GCC 8.x (shipped by default on distros like Debian 10 or RHEL 8) the following build failure happens if LAYOUT_FORMATTING_CONTEXT is enabled:

Building CXX object Source/WebCore/CMakeFiles/WebCore.dir/__/__/WebCore/DerivedSources/unified-sources/UnifiedSource-207b877e-6.cpp.o
[...]
In file included from WebCore/DerivedSources/unified-sources/UnifiedSource-207b877e-6.cpp:6:
../../Source/WebCore/layout/integration/LayoutIntegrationLineLayout.cpp: In member function ‘void WebCore::LayoutIntegration::LineLayout::prepareLayoutState()’:
../../Source/WebCore/layout/integration/LayoutIntegrationLineLayout.cpp:253:36: error: converting to ‘std::optional<WebCore::Layout::Edges>’ from initializer list would use explicit constructor ‘constexpr std::optional<_Tp>::optional(std::in_place_t, _Args&& ...) [with _Args = {}; typename std::enable_if<is_constructible_v<_Tp, _Args&& ...>, bool>::type <anonymous> = 0; _Tp = WebCore::Layout::Edges]’
     rootGeometry.setPadding({ { } });
                                    ^
../../Source/WebCore/layout/integration/LayoutIntegrationLineLayout.cpp:253:36: error: converting to ‘std::in_place_t’ from initializer list would use explicit constructor ‘constexpr std::in_place_t::in_place_t()’
ninja: build stopped: subcommand failed.

[Carlos Alberto Lopez Perez]

Created attachment 431769 [details]
Patch

[Carlos Alberto Lopez Perez]

Created attachment 431770 [details]
Patch

[alan]

Comment on attachment 431770 [details]
Patch

Thanks for fixing it.

[EWS]

Committed r279036 (238956@main): <https://commits.webkit.org/238956@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 431770 [details].