diff options
author | Max Moroz <mmoroz@chromium.org> | 2020-02-06 08:44:42 -0800 |
---|---|---|
committer | Max Moroz <mmoroz@chromium.org> | 2020-02-10 06:50:59 -0800 |
commit | 5ad62d3b7f7e75df776a4524bda0c9a1a9952a4e (patch) | |
tree | 19a5fd4ea9476ad60c1cb3077e7a5e3c84ef5c93 /llvm/tools/llvm-cov/SourceCoverageViewHTML.cpp | |
parent | b1c7bfe6da2b5d171d025bf261f62773470cfc05 (diff) | |
download | llvm-5ad62d3b7f7e75df776a4524bda0c9a1a9952a4e.zip llvm-5ad62d3b7f7e75df776a4524bda0c9a1a9952a4e.tar.gz llvm-5ad62d3b7f7e75df776a4524bda0c9a1a9952a4e.tar.bz2 |
[compiler-rt] Some clean up / refactoring in sanitizer_symbolizer_libcdep.cpp.
Summary:
Nothing critical, just a few potential improvements I've noticed while reading
the code:
- return `false` when symbolizer buffer is too small to read all data
- invert some conditions to reduce indentation
- prefer `nullptr` over `0` for pointers; init some pointers on stack;
- remove minor code duplication
Reviewers: eugenis, vitalybuka
Subscribers: dberris, #sanitizers, llvm-commits, kcc
Tags: #sanitizers, #llvm
Differential Revision: https://reviews.llvm.org/D74137
Diffstat (limited to 'llvm/tools/llvm-cov/SourceCoverageViewHTML.cpp')
0 files changed, 0 insertions, 0 deletions