Call exit(1) if we mean to crash, rather than *(char*)0 = 0

skia/ext/canvas_paint_win.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef SKIA_EXT_CANVAS_PAINT_WIN_H_
 #define SKIA_EXT_CANVAS_PAINT_WIN_H_
 
 #include "skia/ext/canvas_paint_common.h"
 #include "skia/ext/platform_canvas.h"
 
@@ skipping 93 matching lines @@
   void init(bool opaque) {
     PlatformCanvas* canvas = GetPlatformCanvas(this);
     // FIXME(brettw) for ClearType, we probably want to expand the bounds of
     // painting by one pixel so that the boundaries will be correct (ClearType
     // text can depend on the adjacent pixel). Then we would paint just the
     // inset pixels to the screen.
     const int width = ps_.rcPaint.right - ps_.rcPaint.left;
     const int height = ps_.rcPaint.bottom - ps_.rcPaint.top;
     if (!canvas->initialize(width, height, opaque, NULL)) {
       // Cause a deliberate crash;
-      *(char*) 0 = 0;
+      NOTREACHED();

[Stephen White, 2012/09/26 14:48:27]

I've never really looked at NOTREACHED() before, but it looks like it does a
DCHECK(false), which only crashes in debug builds AFAIK.  You probably want a
CHECK(false) here instead.

Alternatively, you could do what OnNoMemory does on Windows, which is:

__debugbreak();
_exit(1);

     }
     canvas->clear(SkColorSetARGB(0, 0, 0, 0));
 
     // This will bring the canvas into the screen coordinate system for the
     // dirty rect
     canvas->translate(SkIntToScalar(-ps_.rcPaint.left),
                       SkIntToScalar(-ps_.rcPaint.top));
   }
 
   // If true, this canvas was created for a BeginPaint.
   const bool for_paint_;
 
   // Disallow copy and assign.
   CanvasPaintT(const CanvasPaintT&);
   CanvasPaintT& operator=(const CanvasPaintT&);
 };
 
 typedef CanvasPaintT<PlatformCanvas> PlatformCanvasPaint;
 
 }  // namespace skia
 
 #endif  // SKIA_EXT_CANVAS_PAINT_WIN_H_