Commit 739a04bc authored by kozyatinskiy's avatar kozyatinskiy Committed by Commit bot

[inspector] removed ArrayLengthHelperFunction specialization for empty array

There is no zero length array usage in inspector codebase. We can safely remove template specialization.

It was reverted to revert another patch and is good by itself.

BUG=chromium:635948
TBR=jochen@chromium.org

Review-Url: https://codereview.chromium.org/2340193002
Cr-Commit-Position: refs/heads/master@{#39454}
parent 3195ab9b
......@@ -27,11 +27,6 @@ enum NotNullTagEnum { NotNullLiteral };
// gives an error if passed a non-array.
template <typename T, std::size_t Size>
char (&ArrayLengthHelperFunction(T (&)[Size]))[Size];
// GCC needs some help to deduce a 0 length array.
#if defined(__GNUC__)
template <typename T>
char (&ArrayLengthHelperFunction(T (&)[0]))[0];
#endif
#define V8_INSPECTOR_ARRAY_LENGTH(array) \
sizeof(::ArrayLengthHelperFunction(array))
......
......@@ -1756,7 +1756,6 @@
4996, # Deprecated function call.
],
'cflags': [
'-Wno-zero-length-array',
'-Wno-shorten-64-to-32',
'-Wno-deprecated-declarations',
],
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment