Guard against 0 tolerance in curve subdivision.
git-svn-id: http://skia.googlecode.com/svn/trunk@1735 2bbb7eff-a529-9590-31e7-b0007b416f81
This commit is contained in:
parent
9ac4a89d35
commit
c10a88825d
@ -18,11 +18,17 @@
|
||||
#include "GrPoint.h"
|
||||
|
||||
static const int MAX_POINTS_PER_CURVE = 1 << 10;
|
||||
const GrScalar GrPathUtils::gMinCurveTol (GrFloatToScalar(0.0001f));
|
||||
|
||||
uint32_t GrPathUtils::quadraticPointCount(const GrPoint points[],
|
||||
GrScalar tol) {
|
||||
if (tol < gMinCurveTol) {
|
||||
tol == gMinCurveTol;
|
||||
}
|
||||
GrAssert(tol > 0);
|
||||
|
||||
GrScalar d = points[1].distanceToLineSegmentBetween(points[0], points[2]);
|
||||
if (d < tol) {
|
||||
if (d <= tol) {
|
||||
return 1;
|
||||
} else {
|
||||
// Each time we subdivide, d should be cut in 4. So we need to
|
||||
@ -68,10 +74,16 @@ uint32_t GrPathUtils::generateQuadraticPoints(const GrPoint& p0,
|
||||
|
||||
uint32_t GrPathUtils::cubicPointCount(const GrPoint points[],
|
||||
GrScalar tol) {
|
||||
GrScalar d = GrMax(points[1].distanceToLineSegmentBetweenSqd(points[0], points[3]),
|
||||
if (tol < gMinCurveTol) {
|
||||
tol == gMinCurveTol;
|
||||
}
|
||||
GrAssert(tol > 0);
|
||||
|
||||
GrScalar d = GrMax(
|
||||
points[1].distanceToLineSegmentBetweenSqd(points[0], points[3]),
|
||||
points[2].distanceToLineSegmentBetweenSqd(points[0], points[3]));
|
||||
d = SkScalarSqrt(d);
|
||||
if (d < tol) {
|
||||
if (d <= tol) {
|
||||
return 1;
|
||||
} else {
|
||||
int temp = SkScalarCeil(SkScalarSqrt(SkScalarDiv(d, tol)));
|
||||
@ -118,6 +130,11 @@ uint32_t GrPathUtils::generateCubicPoints(const GrPoint& p0,
|
||||
|
||||
int GrPathUtils::worstCasePointCount(const GrPath& path, int* subpaths,
|
||||
GrScalar tol) {
|
||||
if (tol < gMinCurveTol) {
|
||||
tol == gMinCurveTol;
|
||||
}
|
||||
GrAssert(tol > 0);
|
||||
|
||||
int pointCount = 0;
|
||||
*subpaths = 1;
|
||||
|
||||
|
@ -26,9 +26,13 @@
|
||||
*/
|
||||
class GrPathUtils : public GrNoncopyable {
|
||||
public:
|
||||
/// Since we divide by tol if we're computing exact worst-case bounds,
|
||||
/// very small tolerances will be increased to gMinCurveTol.
|
||||
static int worstCasePointCount(const GrPath&,
|
||||
int* subpaths,
|
||||
GrScalar tol);
|
||||
/// Since we divide by tol if we're computing exact worst-case bounds,
|
||||
/// very small tolerances will be increased to gMinCurveTol.
|
||||
static uint32_t quadraticPointCount(const GrPoint points[], GrScalar tol);
|
||||
static uint32_t generateQuadraticPoints(const GrPoint& p0,
|
||||
const GrPoint& p1,
|
||||
@ -36,6 +40,8 @@ public:
|
||||
GrScalar tolSqd,
|
||||
GrPoint** points,
|
||||
uint32_t pointsLeft);
|
||||
/// Since we divide by tol if we're computing exact worst-case bounds,
|
||||
/// very small tolerances will be increased to gMinCurveTol.
|
||||
static uint32_t cubicPointCount(const GrPoint points[], GrScalar tol);
|
||||
static uint32_t generateCubicPoints(const GrPoint& p0,
|
||||
const GrPoint& p1,
|
||||
@ -44,5 +50,8 @@ public:
|
||||
GrScalar tolSqd,
|
||||
GrPoint** points,
|
||||
uint32_t pointsLeft);
|
||||
|
||||
private:
|
||||
static const GrScalar gMinCurveTol;
|
||||
};
|
||||
#endif
|
||||
|
Loading…
Reference in New Issue
Block a user