Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2537)

Unified Diff: tests/ClipStackTest.cpp

Issue 27933002: More clang warning fixes. (Closed) Base URL: https://skia.googlecode.com/svn/trunk
Patch Set: trybot fixes Created 7 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tests/CanvasTest.cpp ('k') | tests/ColorTest.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/ClipStackTest.cpp
diff --git a/tests/ClipStackTest.cpp b/tests/ClipStackTest.cpp
index c16220e50041f66bced5144fe21a56ef152fab76..04b48b678364ee0f2bd44f532868c32a48e56cfd 100644
--- a/tests/ClipStackTest.cpp
+++ b/tests/ClipStackTest.cpp
@@ -858,29 +858,6 @@ static void add_elem_to_region(const SkClipStack::Element& element,
region->op(elemRegion, element.getOp());
}
-// This can assist with debugging the clip stack reduction code when the test below fails.
-static inline void print_clip(const SkClipStack::Element& element) {
- static const char* kOpStrs[] = {
- "DF",
- "IS",
- "UN",
- "XR",
- "RD",
- "RP",
- };
- if (SkClipStack::Element::kEmpty_Type != element.getType()) {
- const SkRect& bounds = element.getBounds();
- bool isRect = SkClipStack::Element::kRect_Type == element.getType();
- SkDebugf("%s %s %s [%f %f] x [%f %f]\n",
- kOpStrs[element.getOp()],
- (isRect ? "R" : "P"),
- (element.isInverseFilled() ? "I" : " "),
- bounds.fLeft, bounds.fRight, bounds.fTop, bounds.fBottom);
- } else {
- SkDebugf("EM\n");
- }
-}
-
static void test_reduced_clip_stack(skiatest::Reporter* reporter) {
// We construct random clip stacks, reduce them, and then rasterize both versions to verify that
// they are equal.
« no previous file with comments | « tests/CanvasTest.cpp ('k') | tests/ColorTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698