Replace SkFAIL with SK_ABORT.
SkFAIL is a legacy macro which is just SK_ABORT. This CL mechanically
changes uses of SkFAIL to SK_ABORT in preparation for its removal. The
related sk_throw macro will be changed independently, due to needing to
actually clean up its users.
Change-Id: Id70b5c111a02d2458dc60c8933f444df27d9cebb
Reviewed-on: https://skia-review.googlesource.com/35284
Reviewed-by: Derek Sollenberger <djsollen@google.com>
Commit-Queue: Ben Wagner <bungeman@google.com>
diff --git a/src/gpu/glsl/GrGLSL.cpp b/src/gpu/glsl/GrGLSL.cpp
index 76ffb82..b580f0b 100644
--- a/src/gpu/glsl/GrGLSL.cpp
+++ b/src/gpu/glsl/GrGLSL.cpp
@@ -41,7 +41,7 @@
out->append("precision lowp float;\n");
break;
default:
- SkFAIL("Unknown precision value.");
+ SK_ABORT("Unknown precision value.");
}
}
}
diff --git a/src/gpu/glsl/GrGLSLBlend.cpp b/src/gpu/glsl/GrGLSLBlend.cpp
index 5afb984..34df0df 100644
--- a/src/gpu/glsl/GrGLSLBlend.cpp
+++ b/src/gpu/glsl/GrGLSLBlend.cpp
@@ -380,7 +380,7 @@
break;
}
default:
- SkFAIL("Unknown Custom Xfer mode.");
+ SK_ABORT("Unknown Custom Xfer mode.");
break;
}
}
@@ -427,7 +427,7 @@
fsBuilder->codeAppendf(" * (1.0 - %s.a)", dstColorName);
break;
default:
- SkFAIL("Unsupported Blend Coeff");
+ SK_ABORT("Unsupported Blend Coeff");
}
return true;
}
@@ -494,7 +494,7 @@
dstCoeff = SkBlendModeCoeff::kZero;
break;
default:
- SkFAIL("Unsupported Op");
+ SK_ABORT("Unsupported Op");
// We should never get here but to make compiler happy
srcCoeff = SkBlendModeCoeff::kZero;
dstCoeff = SkBlendModeCoeff::kZero;
diff --git a/src/gpu/glsl/GrGLSLFragmentShaderBuilder.cpp b/src/gpu/glsl/GrGLSLFragmentShaderBuilder.cpp
index 7f16136..afbfa09 100644
--- a/src/gpu/glsl/GrGLSLFragmentShaderBuilder.cpp
+++ b/src/gpu/glsl/GrGLSLFragmentShaderBuilder.cpp
@@ -106,7 +106,7 @@
}
return true;
default:
- SkFAIL("Unexpected GLSLFeature requested.");
+ SK_ABORT("Unexpected GLSLFeature requested.");
return false;
}
}
diff --git a/src/gpu/glsl/GrGLSLGeometryShaderBuilder.cpp b/src/gpu/glsl/GrGLSLGeometryShaderBuilder.cpp
index ea8e73f..d9f34c2 100644
--- a/src/gpu/glsl/GrGLSLGeometryShaderBuilder.cpp
+++ b/src/gpu/glsl/GrGLSLGeometryShaderBuilder.cpp
@@ -18,7 +18,7 @@
case InputType::kTriangles: return "triangles";
case InputType::kTrianglesAdjacency: return "triangles_adjacency";
}
- SkFAIL("invalid input type");
+ SK_ABORT("invalid input type");
return "unknown_input";
}
@@ -29,7 +29,7 @@
case OutputType::kLineStrip: return "line_strip";
case OutputType::kTriangleStrip: return "triangle_strip";
}
- SkFAIL("invalid output type");
+ SK_ABORT("invalid output type");
return "unknown_output";
}
diff --git a/src/gpu/glsl/GrGLSLXferProcessor.h b/src/gpu/glsl/GrGLSLXferProcessor.h
index 791bb06..763fcfa 100644
--- a/src/gpu/glsl/GrGLSLXferProcessor.h
+++ b/src/gpu/glsl/GrGLSLXferProcessor.h
@@ -89,7 +89,7 @@
* it can construct a GrXferProcessor that will not read the dst color.
*/
virtual void emitOutputsForBlendState(const EmitArgs&) {
- SkFAIL("emitOutputsForBlendState not implemented.");
+ SK_ABORT("emitOutputsForBlendState not implemented.");
}
/**
@@ -105,7 +105,7 @@
const char* outColor,
const char* outColorSecondary,
const GrXferProcessor&) {
- SkFAIL("emitBlendCodeForDstRead not implemented.");
+ SK_ABORT("emitBlendCodeForDstRead not implemented.");
}
virtual void onSetData(const GrGLSLProgramDataManager&, const GrXferProcessor&) = 0;