diff options
| author | Bernie Innocenti <codewiz@google.com> | 2020-12-20 07:51:19 +0000 |
|---|---|---|
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2020-12-20 07:51:19 +0000 |
| commit | 32ffd0d4ca3bb31652b9b5d0128e0b6cd678b6fe (patch) | |
| tree | 068a5dce6c6a9aa08b35a0955013095a6ae62855 /trusty/coverage/coverage.cpp | |
| parent | dbf85defc0b893e45446c69cc2fea3475f331fa4 (diff) | |
| parent | 989efb67e3d1770f55da6080876025785d9e4640 (diff) | |
Merge "Add explicit Result::has_value() checks where needed" am: 989efb67e3
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1533183
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: Ib6bd423a7ececfa7fc076542206ee0d0bf385c84
Diffstat (limited to 'trusty/coverage/coverage.cpp')
| -rw-r--r-- | trusty/coverage/coverage.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/trusty/coverage/coverage.cpp b/trusty/coverage/coverage.cpp index 9413ce385..f383dd14a 100644 --- a/trusty/coverage/coverage.cpp +++ b/trusty/coverage/coverage.cpp @@ -142,7 +142,7 @@ Result<void> CoverageRecord::Open() { void CoverageRecord::ResetFullRecord() { auto header_region = GetRegionBounds(COV_START); - if (!header_region) { + if (!header_region.ok()) { // If the header cannot be parsed, we can't reset the proper region yet. return; } @@ -202,7 +202,7 @@ void CoverageRecord::GetRawData(volatile void** begin, volatile void** end) { void CoverageRecord::GetRawCounts(volatile uint8_t** begin, volatile uint8_t** end) { auto region = GetRegionBounds(COV_8BIT_COUNTERS); - if (!region) { + if (!region.ok()) { *begin = 0; *end = 0; return; @@ -216,7 +216,7 @@ void CoverageRecord::GetRawCounts(volatile uint8_t** begin, volatile uint8_t** e void CoverageRecord::GetRawPCs(volatile uintptr_t** begin, volatile uintptr_t** end) { auto region = GetRegionBounds(COV_INSTR_PCS); - if (!region) { + if (!region.ok()) { *begin = 0; *end = 0; return; |