commit a913c8522a7289cf377bf7ee1c9a05b3a7b2b336
author Ali Zhang <alizhang@google.com> Fri Aug 06 12:26:34 2021 -0700
committer CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue Aug 24 18:14:30 2021 +0000
tree d3bb6abffa1b8de32f18e4724e5755e98adc74f8
parent 43f24395b10fe456a8f0cddece3bf9ead9f91b06

pw_crypto: Consolidate logging

Updates logging to be consistent with the following principles.

1. Library modules should favor PW_LOG_DEBUG() over PW_LOG_ERROR() as
   whether something is considered and "error" should a discretion
   of the caller, not the callee. PW_LOG_ERROR() also bloats code
   size. Libraries should try to communicate the result of an operation
   via status codes.

2. Use PW_LOG_DEBUG() liberally in library modules to help development
   and debugging, but use a local conditional to compile them out by
   default. In this change, this is achieved by setting the default
   debug level to PW_LOG_LEVEL_WARN for each file.

No-Docs-Update-Reason: refactor only.
Change-Id: I032c7a86ae6aaee5c6ec0c28cef5662a36bb9358
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/56360
Commit-Queue: Ali Zhang <alizhang@google.com>
Reviewed-by: Keir Mierle <keir@google.com>