tree 8f73074c8ddc4f9c323ec8b0bed6685953bc8042
parent 58427c44b9261035351d2eee604a299c0b46dbb4
author Shawn Willden <swillden@google.com> 1432346286 -0600
committer Shawn Willden <swillden@google.com> 1432658148 -0600

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