diff options
| author | Mike Frysinger <vapier@chromium.org> | 2016-03-18 15:48:45 -0400 |
|---|---|---|
| committer | Mike Frysinger <vapier@chromium.org> | 2016-04-11 13:05:44 -0400 |
| commit | 2f1b7adcd3387bb2e5077c07f64ff71b2a399798 (patch) | |
| tree | e79f425b05fe63aaeb6d09601bc05f41acf8c872 /src/common/linux | |
| parent | Added an option (-i) to have dump_syms output header information only. (diff) | |
| download | breakpad-2f1b7adcd3387bb2e5077c07f64ff71b2a399798.tar.xz | |
tweak IGNORE_RET behavior a bit
GCC will still warn about unused return value with the form:
if (write(...));
Instead, change the semi-colon to an empty set of braces.
BUG=chromium:428478
TEST=build+test still works
Diffstat (limited to 'src/common/linux')
| -rw-r--r-- | src/common/linux/ignore_ret.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/common/linux/ignore_ret.h b/src/common/linux/ignore_ret.h index f60384bb..efd274c2 100644 --- a/src/common/linux/ignore_ret.h +++ b/src/common/linux/ignore_ret.h @@ -35,6 +35,6 @@ // the call fails, IGNORE_RET() can be used to mark the return code as ignored. // This avoids spurious compiler warnings. -#define IGNORE_RET(x) do { if (x); } while (0) +#define IGNORE_RET(x) do { if (x) {} } while (0) #endif // COMMON_LINUX_IGNORE_RET_H_ |