diff options
author | Shawn Willden <swillden@google.com> | 2015-05-22 19:58:06 -0600 |
---|---|---|
committer | Shawn Willden <swillden@google.com> | 2015-05-26 10:35:48 -0600 |
commit | de7e66c3692073eb967f01cc8281441709701e2d (patch) | |
tree | 8f73074c8ddc4f9c323ec8b0bed6685953bc8042 /integrity_assured_key_blob.cpp | |
parent | 58427c44b9261035351d2eee604a299c0b46dbb4 (diff) | |
download | android_system_keymaster-de7e66c3692073eb967f01cc8281441709701e2d.tar.gz android_system_keymaster-de7e66c3692073eb967f01cc8281441709701e2d.tar.bz2 android_system_keymaster-de7e66c3692073eb967f01cc8281441709701e2d.zip |
Change handling of debug log statments in non-debug builds.
The previous way had a problem when used in statments like:
if (foo)
LOG_D(...);
When built without debugging, this became:
if (foo)
;
Which is sort of okay, but the compiler complains. The new way also has
the advantage that the compiler always sees and checks the log
arguments. Given that it ends up compiling something like:
do {
if (0)
Logger::Debug(...);
} while (0);
It should optimize the entire block out, and should even discard the
literal string used for the format. So it's better all around.
Change-Id: I895141077f627a2d08dcb0d7d2d0799067a2c957
Diffstat (limited to 'integrity_assured_key_blob.cpp')
0 files changed, 0 insertions, 0 deletions