use unsigned to avoid runtime overflow detection

Bug: skia:
Change-Id: I487930955f75048ea27a1bcc61f7e0849c63759b
Reviewed-on: https://skia-review.googlesource.com/32681
Commit-Queue: Mike Reed <reed@google.com>
Commit-Queue: Mike Klein <mtklein@google.com>
Reviewed-by: Mike Klein <mtklein@google.com>

1 files changed, 3 insertions, 2 deletions

diff --git a/bench/MathBench.cpp b/bench/MathBench.cpp
index 1ae683aab8..3b0fd099ab 100644
--- a/bench/MathBench.cpp
+++ b/bench/MathBench.cpp
@@ -630,12 +630,13 @@ public:
 
     // These exist to try to stop the compiler from detecting what we doing, and throwing
     // parts away (or knowing exactly how big the loop counts are).
-    virtual void process(int) {}
+    virtual void process(unsigned) {}
     virtual int count() { return ARRAY; }
 
 protected:
     void onDraw(int loops, SkCanvas*) override {
-        int accum = 0;
+        // used unsigned to avoid undefined behavior if/when the += might overflow
+        unsigned accum = 0;
 
         for (int j = 0; j < loops; ++j) {
             int n = this->count();