My clang now doesn't complain about !"foo".

src/utils/SkJSON.cpp

 /*
  * Copyright 2011 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkJSON.h"
 #include "SkString.h"
 
@@ skipping 396 matching lines @@
             case kInt:
                 SkDebugf("%d", slot->fValue.fInt);
                 break;
             case kFloat:
                 SkDebugf("%g", slot->fValue.fFloat);
                 break;
             case kBool:
                 SkDebugf("%s", slot->fValue.fBool ? "true" : "false");
                 break;
             default:
-                SkASSERT(!"how did I get here");
+                SkDEBUGFAIL("how did I get here");
                 break;
         }
         if (slot->fNext) {
             SkDebugf(",");
         }
         SkDebugf("\n");
     }
 }
 
 void SkJSON::Array::dumpLevel(int level) const {
@@ skipping 55 matching lines @@
             }
             SkDebugf(" %g ", fArray.fFloats[last]);
         } break;
         case kBool: {
             for (int i = 0; i < last; ++i) {
                 SkDebugf(" %s,", fArray.fBools[i] ? "true" : "false");
             }
             SkDebugf(" %s ", fArray.fInts[last] ? "true" : "false");
         } break;
         default:
-            SkASSERT(!"unsupported array type");
+            SkDEBUGFAIL("unsupported array type");
             break;
     }
 }
 
 ///////////////////////////////////////////////////////////////////////////////
 
 static const uint8_t gBytesPerType[] = {
     sizeof(SkJSON::Object*),
     sizeof(SkJSON::Array*),
     sizeof(char*),
@@ skipping 121 matching lines @@
 }
 
 void SkJSON::Array::setString(int index, const char str[]) {
     SkASSERT((unsigned)index < (unsigned)fCount);
     char*& prev = fArray.fStrings[index];
     if (prev != str) {
         free_string(prev);
         prev = dup_string(str);
     }
 }