commit | 96299d70edba60e5eb88a1efca384bc82392d977 | [log] [tgz] |
---|---|---|
author | Nico Weber <thakis@chromium.org> | Tue Oct 12 10:42:43 2021 -0400 |
committer | Behdad Esfahbod <behdad@behdad.org> | Tue Oct 12 08:56:54 2021 -0600 |
tree | df4237d77619219791f0aa31354fa998e66e297f | |
parent | b621c4fc29162c69098dce29fed887eac903d516 [diff] |
Fix Wbitwise-instead-of-logical warnings `a || b` only evaluates b if a is false. `a | b` always evaluates both a and b. If a and b are of type bool, || is usually what you want, so clang now warns on `|` where both arguments are of type bool. This warning fires twice in harfbuzz. In both cases, `|` is used intentionally, with a comment explaining this. Slightly reorder the code a bit to make the compiler happy, and to make it obvious even without a comment that both calls should be evaluated. No intended behavior change.
This is HarfBuzz, a text shaping library.
For bug reports, mailing list, and other information please visit:
For license information, see COPYING.
For build information, see BUILD.md.
For custom configurations, see CONFIG.md.
For test execution, see TESTING.md.
Documentation: https://harfbuzz.github.io