diff options
author | Don Lewis <truckman@apache.org> | 2016-08-05 05:49:46 +0000 |
---|---|---|
committer | Don Lewis <truckman@apache.org> | 2016-08-05 05:49:46 +0000 |
commit | 5b6c4bddf6134f377a61446d8bcf1be2f096aad1 (patch) | |
tree | 87a89482e51af8bb24d2f5a019d7603eb6da7b71 /postprocess | |
parent | 67579b01b47d10f5b835dcb7f8bcce415ff9e221 (diff) |
Fix --enable-debug build with clang as the compiler.
Clang warns about this expression in graphite:
if (!m_srAttachTo == 0)
Operator precedence rules require this to be evaluated as:
if ((!m_srAttachTo) == 0)
but clang worries that the author might have intended this to mean:
if (!(m_srAttachTo == 0))
Fortunately these two are equivalent, as is:
if (m_srAttachTo != 0)
and the latter is much easier for humans to understand.
For some reason, --enable-debug adds -Werror to the graphite CFLAGS even
though its makefile.mk specifies:
EXTERNAL_WARNINGS_NOT_ERRORS := TRUE
Notes
Notes:
ignore: obsolete
Diffstat (limited to 'postprocess')
0 files changed, 0 insertions, 0 deletions