Clean up Skia for Clang's -Wtautological-undefined-compare

This is blocking Chrome from rolling a new Clang, and me too for our TSAN bots.

BUG=chromium:381910

Review URL: https://codereview.chromium.org/325603002

diff --git a/include/core/SkTLazy.h b/include/core/SkTLazy.h
index 518beec..a291e22 100644 (file)
--- a/include/core/SkTLazy.h
+++ b/include/core/SkTLazy.h
@@ -66,11 +66,6 @@ public:
      *  contents.
      */
     T* set(const T& src) {
-        // Diagnoistic. May remove later. See crbug.com/364224
-        if (NULL == &src) {
-            sk_throw();
-        }
-
         if (this->isValid()) {
             *fPtr = src;
         } else {

diff --git a/src/core/SkPaint.cpp b/src/core/SkPaint.cpp
index f3bf01b..78d9470 100644 (file)
--- a/src/core/SkPaint.cpp
+++ b/src/core/SkPaint.cpp
@@ -106,11 +106,6 @@ SkPaint::SkPaint() {
 }
 
 SkPaint::SkPaint(const SkPaint& src) {
-    // Diagnoistic. May remove later. See crbug.com/364224
-    if (NULL == &src) {
-        sk_throw();
-    }
-
 #define COPY(field) field = src.field
 #define REF_COPY(field) field = SkSafeRef(src.field)
 

diff --git a/src/core/SkRegion.cpp b/src/core/SkRegion.cpp
index 98670b6..24d7248 100644 (file)
--- a/src/core/SkRegion.cpp
+++ b/src/core/SkRegion.cpp
@@ -212,12 +212,6 @@ char* SkRegion::toString() {
 ///////////////////////////////////////////////////////////////////////////////
 
 int SkRegion::count_runtype_values(int* itop, int* ibot) const {
-    if (this == NULL) {
-        *itop = SK_MinS32;
-        *ibot = SK_MaxS32;
-        return 0;
-    }
-
     int maxT;
 
     if (this->isRect()) {