Reviewers: jochen,
Description:
Fix some clang warnings with -Wmissing-braces in v8.
Clang warns if there are missing braces around a subobject
initializer. The most common idiom that triggers this is:
STRUCT s = {0};
if the first field of STRUCT is itself a struct. This can
be more simply written as:
STRUCT s = {};
which also prevents the warning from firing.
BUG=505297
Please review this at https://codereview.chromium.org/1219233003/
Base URL: https://chromium.googlesource.com/v8/v8.git@master
Affected files (+1, -1 lines):
M src/base/sys-info.cc
Index: src/base/sys-info.cc
diff --git a/src/base/sys-info.cc b/src/base/sys-info.cc
index
617a88ea818c969865082d779be86fcbbef06153..a2dc6ab27fc7e8f649dfd2dcc005092ee404de09
100644
--- a/src/base/sys-info.cc
+++ b/src/base/sys-info.cc
@@ -44,7 +44,7 @@ int SysInfo::NumberOfProcessors() {
}
return static_cast<int>(result);
#elif V8_OS_WIN
- SYSTEM_INFO system_info = {0};
+ SYSTEM_INFO system_info = {};
::GetNativeSystemInfo(&system_info);
return static_cast<int>(system_info.dwNumberOfProcessors);
#endif
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
