diff options
| author | danakj <danakj@chromium.org> | 2015-03-09 15:27:25 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-03-09 22:28:34 +0000 |
| commit | 94219a212c616eac1914ca851c55256c4bbd9e36 (patch) | |
| tree | b9721bd6f2d10f117355fdfa1a0391262b745628 /base/barrier_closure.cc | |
| parent | 6be122fb693fe8f2a5b150a12bddd5c85160b7a6 (diff) | |
| download | chromium_src-94219a212c616eac1914ca851c55256c4bbd9e36.zip chromium_src-94219a212c616eac1914ca851c55256c4bbd9e36.tar.gz chromium_src-94219a212c616eac1914ca851c55256c4bbd9e36.tar.bz2 | |
base: Use more specific CHECK macros for comparisons.
Prefer DCHECK_EQ(a, b) over DCHECK(a == b) when possible as this
provides more data on a failure. Similar for other operators such as
>=, <, !=, etc.
This also applies to CHECK, EXPECT, and ASSERT macros.
R=Nico
BUG=464816
Review URL: https://codereview.chromium.org/985723003
Cr-Commit-Position: refs/heads/master@{#319751}
Diffstat (limited to 'base/barrier_closure.cc')
| -rw-r--r-- | base/barrier_closure.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/base/barrier_closure.cc b/base/barrier_closure.cc index 5abe2bf..1b77429 100644 --- a/base/barrier_closure.cc +++ b/base/barrier_closure.cc @@ -39,7 +39,7 @@ namespace base { base::Closure BarrierClosure(int num_callbacks_left, const base::Closure& done_closure) { - DCHECK(num_callbacks_left >= 0); + DCHECK_GE(num_callbacks_left, 0); if (num_callbacks_left == 0) done_closure.Run(); |