Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(170)

Issue 2691243004: Add ANALYSIS_ASSUME_TRUE annotations to DCHECK_OP impl bodies. (Closed)

Created:
3 years, 10 months ago by Kevin M
Modified:
3 years, 10 months ago
Reviewers:
danakj, Wez
CC:
chromium-reviews, vmpstr+watch_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Add ANALYSIS_ASSUME_TRUE annotations to DCHECK_OP impl bodies. The annotations stop static analysis if a comparison-based DCHECK fails, which is a signal to the static analyzer that the asserted condition will always be true. R=danakj@chromium.org,wez@chromium.org BUG=327707 Review-Url: https://codereview.chromium.org/2691243004 Cr-Commit-Position: refs/heads/master@{#450532} Committed: https://chromium.googlesource.com/chromium/src/+/9db26fbab2a06135a5519f21324d1a01851cef3d

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -9 lines) Patch
M base/logging.h View 1 chunk +13 lines, -9 lines 0 comments Download

Messages

Total messages: 7 (3 generated)
Kevin M
3 years, 10 months ago (2017-02-14 21:04:52 UTC) #1
danakj
LGTM
3 years, 10 months ago (2017-02-14 21:11:28 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2691243004/1
3 years, 10 months ago (2017-02-14 21:55:55 UTC) #4
commit-bot: I haz the power
3 years, 10 months ago (2017-02-15 01:07:29 UTC) #7
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/9db26fbab2a06135a5519f21324d...

Powered by Google App Engine
This is Rietveld 408576698