diff options
| author | Primiano Tucci <primiano@chromium.org> | 2016-05-03 16:56:06 +0200 |
|---|---|---|
| committer | Primiano Tucci <primiano@chromium.org> | 2016-05-03 16:56:06 +0200 |
| commit | fcb844ee32171e2340162f1ced0e7fdcde7b2b24 (patch) | |
| tree | b4e7d3b862f533becb32c9277cade4818d693c5c /src/client/solaris/handler/exception_handler_test.cc | |
| parent | Revert of Extend mapping merge to include reserved but unused mappings. (http... (diff) | |
| download | breakpad-fcb844ee32171e2340162f1ced0e7fdcde7b2b24.tar.xz | |
Add parentheses to silence clang warning
crrev.com/1887033002 introuced a clang warning (see below).
This fixes it, so that breakpad can be rolled in chrome, where
warnings are always fatal.
From: https://build.chromium.org/p/tryserver.chromium.android/builders/android_chromium_gn_compile_dbg/builds/59031/steps/compile%20%28with%20patch%29/logs/stdio
FAILED: clang_x64/obj/breakpad/dump_syms/dwarf_cu_to_module.o
../../breakpad/src/common/dwarf_cu_to_module.cc:420:20: error: '&&' within '||' [-Werror,-Wlogical-op-parentheses]
if (declaration_ && qualified_name || (unqualified_name && enclosing_name)) {
~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~ ~~
../../breakpad/src/common/dwarf_cu_to_module.cc:420:20: note: place parentheses around the '&&' expression to silence this warning
if (declaration_ && qualified_name || (unqualified_name && enclosing_name)) {
^
( )
R=mark@chromium.org, petrcermak@chromium.org
Review URL: https://codereview.chromium.org/1928363002 .
Diffstat (limited to 'src/client/solaris/handler/exception_handler_test.cc')
0 files changed, 0 insertions, 0 deletions