Bug 192583 – [Win][Clang] Fix compilation warnings of WTF

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

[Fujii Hironori]

[Win][Clang] Fix compilation warnings of WTF

clang-cl reports the following warnings.

> [92/206] Building CXX object Source\WTF\wtf\CMakeFiles\WTF.dir\StackBounds.cpp.obj
> ..\..\Source\WTF\wtf\StackBounds.cpp(163,48):  warning: missing field 'AllocationBase' initializer [-Wmissing-field-initializers]
>     MEMORY_BASIC_INFORMATION stackOrigin = { 0 };
>                                                ^
> 1 warning generated.
> [160/206] Building CXX object Source\WTF\wtf\CMakeFiles\WTF.dir\win\RunLoopWin.cpp.obj
> ..\..\Source\WTF\wtf\win\RunLoopWin.cpp(34,54):  warning: ISO C++11 does not allow conversion from string literal to 'const LPWSTR' (aka 'wchar_t *const') [-Wwritable-strings]
> static const LPWSTR kRunLoopMessageWindowClassName = L"RunLoopMessageWindow";
>                                                      ^
> ..\..\Source\WTF\wtf\win\RunLoopWin.cpp(86,32):  warning: missing field 'lpfnWndProc' initializer [-Wmissing-field-initializers]
>     WNDCLASS windowClass = { 0 };
>                                ^
> 2 warnings generated.
> [175/206] Building CXX object Source\WTF\wtf\CMakeFiles\WTF.dir\DateMath.cpp.obj
> ..\..\Source\WTF\wtf\DateMath.cpp(125,20):  warning: unused function 'getLocalTime' [-Wunused-function]
> static inline void getLocalTime(const time_t* localTime, struct tm* localTM)
>                    ^
> 1 warning generated.
> [180/206] Building CXX object Source\WTF\wtf\CMakeFiles\WTF.dir\Assertions.cpp.obj
> ..\..\Source\WTF\wtf\Assertions.cpp(263,1):  warning: function declared 'noreturn' should not return [-Winvalid-noreturn]
> }
> ^
> 1 warning generated.

[Fujii Hironori]

Created attachment 357041 [details]
Patch

[Ross Kirsling]

Comment on attachment 357041 [details]
Patch

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

> Source/WTF/wtf/StackBounds.cpp:163
> -    MEMORY_BASIC_INFORMATION stackOrigin = { 0 };
> +    MEMORY_BASIC_INFORMATION stackOrigin = { };

We should probably remove the = here.
(I guess if the default constructor works then we don't necessarily need the { } either, but it doesn't hurt to keep it.)

> Source/WTF/wtf/win/RunLoopWin.cpp:86
> -    WNDCLASS windowClass = { 0 };
> +    WNDCLASS windowClass = { };

Ditto.

[Fujii Hironori]

Comment on attachment 357041 [details]
Patch

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

>> Source/WTF/wtf/StackBounds.cpp:163
>> +    MEMORY_BASIC_INFORMATION stackOrigin = { };
> 
> We should probably remove the = here.
> (I guess if the default constructor works then we don't necessarily need the { } either, but it doesn't hurt to keep it.)

Good catch. Thanks. Will fix.
Unfortunately, Windows API is implemented in C. It doesn't have ctor.
https://docs.microsoft.com/en-us/windows/desktop/api/winnt/ns-winnt-_memory_basic_information

[Fujii Hironori]

Created attachment 357090 [details]
Patch for landing

[Fujii Hironori]

Committed r239093: <https://trac.webkit.org/changeset/239093>

[Radar WebKit Bug Importer]

<rdar://problem/46647928>