diff options
| author |  caryclark@google.com <caryclark@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2012-06-06 12:11:45 +0000 |
|---|---|---|
| committer | caryclark@google.com <caryclark@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2012-06-06 12:11:45 +0000 |
| commit | 19069a28225356bc24a5df7ad03c4d4ac5c532cb (patch) | |
| tree | aa3bf0dc543ad71eb69370d0aee6e0d9f453b2c1 /bench/InterpBench.cpp | |
| parent | 9da0cea3a32e274ad8b40174a25e5f8c532fbd9b (diff) | |
fix warnings on Mac in bench
Fix these class of warnings:
- unused functions
- unused locals
- sign mismatch
- missing function prototypes
- missing newline at end of file
- 64 to 32 bit truncation
The changes prefer to link in dead code in the debug build
with 'if (false)' than to comment it out, but trivial cases
are commented out or sometimes deleted if it appears to be
a copy/paste error.
Review URL: https://codereview.appspot.com/6302044
git-svn-id: http://skia.googlecode.com/svn/trunk@4188 2bbb7eff-a529-9590-31e7-b0007b416f81
Diffstat (limited to 'bench/InterpBench.cpp')
| -rw-r--r-- | bench/InterpBench.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/bench/InterpBench.cpp b/bench/InterpBench.cpp index 89841f8a69..85fada3f0e 100644 --- a/bench/InterpBench.cpp +++ b/bench/InterpBench.cpp @@ -102,10 +102,10 @@ protected: int64_t step = (int64_t)(dx * 65536 * 655536 * 65536); SkFixed tmp; for (int i = 0; i < count; i += 4) { - tmp = curr >> 32; dst[i + 0] = TILE(tmp, count); curr += step; - tmp = curr >> 32; dst[i + 1] = TILE(tmp, count); curr += step; - tmp = curr >> 32; dst[i + 2] = TILE(tmp, count); curr += step; - tmp = curr >> 32; dst[i + 3] = TILE(tmp, count); curr += step; + tmp = (SkFixed) (curr >> 32); dst[i + 0] = TILE(tmp, count); curr += step; + tmp = (SkFixed) (curr >> 32); dst[i + 1] = TILE(tmp, count); curr += step; + tmp = (SkFixed) (curr >> 32); dst[i + 2] = TILE(tmp, count); curr += step; + tmp = (SkFixed) (curr >> 32); dst[i + 3] = TILE(tmp, count); curr += step; } } private: |