diff options
author | Tom Cherry <tomcherry@google.com> | 2019-12-06 13:33:11 -0800 |
---|---|---|
committer | Tom Cherry <tomcherry@google.com> | 2019-12-06 13:55:08 -0800 |
commit | e17b4f62df498c7e2f41ef55510995c317f3aaec (patch) | |
tree | cb82d66d1de5af93a76678f1372601fe0f65a6fc /trusty/coverage/coverage.cpp | |
parent | ed860ff4bfc37f36190d0707f2ec623c56413f29 (diff) |
Cleanup package_string() and its users
package_string() isn't readable in its current form and a loop is
unnecessary, so let's replace that with the direct calculations. The
new and old functions are identical in results except an edge case
where the old function incorrectly believes it needs to round up to
'10' for the size prefix, when '9' would be ok, specifically:
10\naaaaa\n\f vs 9\naaaaa\n\f. This is true for all powers of 10.
Clean up the calling side in logcat as well.
Test: printing log statistics and prune list works
Change-Id: Ib62ab2badab59040215b130ec9e3efbc7c95af3f
Diffstat (limited to 'trusty/coverage/coverage.cpp')
0 files changed, 0 insertions, 0 deletions