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

Issue 14180004: Use ILLEGAL_MODE macro to silence clang error. (Closed)

Created:
7 years, 8 months ago by tfarina
Modified:
7 years, 8 months ago
CC:
skia-review_googlegroups.com
Visibility:
Public.

Description

Use ILLEGAL_MODE macro to silence clang error. This fixes the following clang error: ../../tests/XfermodeTest.cpp:43:44: error: comparison of constant -1 with expression of type 'SkXfermode::Mode' is always false [-Werror,-Wtautological-constant-out-of-range-compare] REPORTER_ASSERT(reporter, reportedMode == -1); ~~~~~~~~~~~~ ^ ~~ R=vandebo@chromium.org,reed@google.com Committed: https://code.google.com/p/skia/source/detail?r=8692

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -3 lines) Patch
M tests/XfermodeTest.cpp View 1 chunk +3 lines, -3 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
tfarina
7 years, 8 months ago (2013-04-12 02:44:55 UTC) #1
vandebo (ex-Chrome)
If I understand correctly, the test is right and the clang error is wrong. Can ...
7 years, 8 months ago (2013-04-12 16:34:47 UTC) #2
tfarina
On 2013/04/12 16:34:47, vandebo wrote: > If I understand correctly, the test is right and ...
7 years, 8 months ago (2013-04-12 16:47:35 UTC) #3
vandebo (ex-Chrome)
I understand now. LGTM
7 years, 8 months ago (2013-04-12 16:48:30 UTC) #4
tfarina
7 years, 8 months ago (2013-04-15 22:12:55 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 manually as r8692 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698