diff options
-rw-r--r-- | src/core/SkScan.h | 10 | ||||
-rw-r--r-- | src/core/SkScanPriv.h | 128 | ||||
-rw-r--r-- | src/core/SkScan_AAAPath.cpp | 75 | ||||
-rw-r--r-- | src/core/SkScan_AntiPath.cpp | 160 | ||||
-rw-r--r-- | src/core/SkScan_DAAPath.cpp | 74 |
5 files changed, 217 insertions, 230 deletions
diff --git a/src/core/SkScan.h b/src/core/SkScan.h index 8bb9d1f86d..2212d0de5b 100644 --- a/src/core/SkScan.h +++ b/src/core/SkScan.h @@ -89,10 +89,12 @@ private: const SkRegion*, SkBlitter*); static void HairLineRgn(const SkPoint[], int count, const SkRegion*, SkBlitter*); static void AntiHairLineRgn(const SkPoint[], int count, const SkRegion*, SkBlitter*); - static void AAAFillPath(const SkPath& path, const SkRegion& origClip, SkBlitter* blitter, - bool forceRLE = false); // SkAAClip uses forceRLE - static void DAAFillPath(const SkPath& path, const SkRegion& origClip, SkBlitter* blitter, - bool forceRLE = false); + static void AAAFillPath(const SkPath& path, SkBlitter* blitter, const SkIRect& pathIR, + const SkIRect& clipBounds, bool containedInClip, bool forceRLE); + static void DAAFillPath(const SkPath& path, SkBlitter* blitter, const SkIRect& pathIR, + const SkIRect& clipBounds, bool containedInClip, bool forceRLE); + static void SAAFillPath(const SkPath& path, SkBlitter* blitter, const SkIRect& pathIR, + const SkIRect& clipBounds, bool containedInClip, bool forceRLE); }; /** Assign an SkXRect from a SkIRect, by promoting the src rect's coordinates diff --git a/src/core/SkScanPriv.h b/src/core/SkScanPriv.h index 96ee695165..544f5f5dd6 100644 --- a/src/core/SkScanPriv.h +++ b/src/core/SkScanPriv.h @@ -78,45 +78,6 @@ static EdgeType* backward_insert_start(EdgeType* prev, SkFixed x) { return prev; } -static bool fitsInsideLimit(const SkRect& r, SkScalar max) { - const SkScalar min = -max; - return r.fLeft > min && r.fTop > min && - r.fRight < max && r.fBottom < max; -} - -static int overflows_short_shift(int value, int shift) { - const int s = 16 + shift; - return (SkLeftShift(value, s) >> s) - value; -} - -/** - Would any of the coordinates of this rectangle not fit in a short, - when left-shifted by shift? -*/ -static int rect_overflows_short_shift(SkIRect rect, int shift) { - SkASSERT(!overflows_short_shift(8191, shift)); - SkASSERT(overflows_short_shift(8192, shift)); - SkASSERT(!overflows_short_shift(32767, 0)); - SkASSERT(overflows_short_shift(32768, 0)); - - // Since we expect these to succeed, we bit-or together - // for a tiny extra bit of speed. - return overflows_short_shift(rect.fLeft, shift) | - overflows_short_shift(rect.fRight, shift) | - overflows_short_shift(rect.fTop, shift) | - overflows_short_shift(rect.fBottom, shift); -} - -static bool safeRoundOut(const SkRect& src, SkIRect* dst, int32_t maxInt) { - const SkScalar maxScalar = SkIntToScalar(maxInt); - - if (fitsInsideLimit(src, maxScalar)) { - src.roundOut(dst); - return true; - } - return false; -} - // Check if the path is a rect and fat enough after clipping; if so, blit it. static inline bool TryBlitFatAntiRect(SkBlitter* blitter, const SkPath& path, const SkIRect& clip) { SkRect rect; @@ -134,93 +95,4 @@ static inline bool TryBlitFatAntiRect(SkBlitter* blitter, const SkPath& path, co return true; } -using FillPathFunc = std::function<void(const SkPath& path, SkBlitter* blitter, bool isInverse, - const SkIRect& ir, const SkIRect& clipBounds, bool containedInClip, bool forceRLE)>; - -static inline void do_fill_path(const SkPath& path, const SkRegion& origClip, SkBlitter* blitter, - bool forceRLE, const int SHIFT, FillPathFunc fillPathFunc) { - if (origClip.isEmpty()) { - return; - } - - const bool isInverse = path.isInverseFillType(); - SkIRect ir; - - if (!safeRoundOut(path.getBounds(), &ir, SK_MaxS32 >> SHIFT)) { - // Bounds can't fit in SkIRect; we'll return without drawing - return; - } - if (ir.isEmpty()) { - if (isInverse) { - blitter->blitRegion(origClip); - } - return; - } - - // If the intersection of the path bounds and the clip bounds - // will overflow 32767 when << by SHIFT, we can't supersample, - // so draw without antialiasing. - SkIRect clippedIR; - if (isInverse) { - // If the path is an inverse fill, it's going to fill the entire - // clip, and we care whether the entire clip exceeds our limits. - clippedIR = origClip.getBounds(); - } else { - if (!clippedIR.intersect(ir, origClip.getBounds())) { - return; - } - } - if (rect_overflows_short_shift(clippedIR, SHIFT)) { - SkScan::FillPath(path, origClip, blitter); - return; - } - - // Our antialiasing can't handle a clip larger than 32767, so we restrict - // the clip to that limit here. (the runs[] uses int16_t for its index). - // - // A more general solution (one that could also eliminate the need to - // disable aa based on ir bounds (see overflows_short_shift) would be - // to tile the clip/target... - SkRegion tmpClipStorage; - const SkRegion* clipRgn = &origClip; - { - static const int32_t kMaxClipCoord = 32767; - const SkIRect& bounds = origClip.getBounds(); - if (bounds.fRight > kMaxClipCoord || bounds.fBottom > kMaxClipCoord) { - SkIRect limit = { 0, 0, kMaxClipCoord, kMaxClipCoord }; - tmpClipStorage.op(origClip, limit, SkRegion::kIntersect_Op); - clipRgn = &tmpClipStorage; - } - } - // for here down, use clipRgn, not origClip - - SkScanClipper clipper(blitter, clipRgn, ir); - const SkIRect* clipRect = clipper.getClipRect(); - - if (clipper.getBlitter() == nullptr) { // clipped out - if (isInverse) { - blitter->blitRegion(*clipRgn); - } - return; - } - - SkASSERT(clipper.getClipRect() == nullptr || - *clipper.getClipRect() == clipRgn->getBounds()); - - // now use the (possibly wrapped) blitter - blitter = clipper.getBlitter(); - - if (isInverse) { - sk_blit_above(blitter, ir, *clipRgn); - } - - SkASSERT(SkIntToScalar(ir.fTop) <= path.getBounds().fTop); - - fillPathFunc(path, blitter, isInverse, ir, clipRgn->getBounds(), clipRect == nullptr, forceRLE); - - if (isInverse) { - sk_blit_below(blitter, ir, *clipRgn); - } -} - #endif diff --git a/src/core/SkScan_AAAPath.cpp b/src/core/SkScan_AAAPath.cpp index d62b151e59..f9445d24f2 100644 --- a/src/core/SkScan_AAAPath.cpp +++ b/src/core/SkScan_AAAPath.cpp @@ -1672,48 +1672,45 @@ static SK_ALWAYS_INLINE void aaa_fill_path(const SkPath& path, const SkIRect& cl /////////////////////////////////////////////////////////////////////////////// -void SkScan::AAAFillPath(const SkPath& path, const SkRegion& origClip, SkBlitter* blitter, - bool forceRLE) { - FillPathFunc fillPathFunc = [](const SkPath& path, SkBlitter* blitter, bool isInverse, - const SkIRect& ir, const SkIRect& clipBounds, bool containedInClip, bool forceRLE){ - // The mask blitter (where we store intermediate alpha values directly in a mask, and then - // call the real blitter once in the end to blit the whole mask) is faster than the RLE - // blitter when the blit region is small enough (i.e., canHandleRect(ir)). - // When isInverse is true, the blit region is no longer ir so we won't use the mask blitter. - // The caller may also use the forceRLE flag to force not using the mask blitter. - // Also, when the path is a simple rect, preparing a mask and blitting it might have too - // much overhead. Hence we'll use blitFatAntiRect to avoid the mask and its overhead. - if (MaskAdditiveBlitter::canHandleRect(ir) && !isInverse && !forceRLE) { +void SkScan::AAAFillPath(const SkPath& path, SkBlitter* blitter, const SkIRect& ir, + const SkIRect& clipBounds, bool containedInClip, bool forceRLE) { + bool isInverse = path.isInverseFillType(); + + // The mask blitter (where we store intermediate alpha values directly in a mask, and then call + // the real blitter once in the end to blit the whole mask) is faster than the RLE blitter when + // the blit region is small enough (i.e., canHandleRect(ir)). When isInverse is true, the blit + // region is no longer the rectangle ir so we won't use the mask blitter. The caller may also + // use the forceRLE flag to force not using the mask blitter. Also, when the path is a simple + // rect, preparing a mask and blitting it might have too much overhead. Hence we'll use + // blitFatAntiRect to avoid the mask and its overhead. + if (MaskAdditiveBlitter::canHandleRect(ir) && !isInverse && !forceRLE) { #ifdef SK_SUPPORT_LEGACY_SMALLRECT_AA + MaskAdditiveBlitter additiveBlitter(blitter, ir, clipBounds, isInverse); + aaa_fill_path(path, clipBounds, &additiveBlitter, ir.fTop, ir.fBottom, + containedInClip, true, forceRLE); +#else + // blitFatAntiRect is slower than the normal AAA flow without MaskAdditiveBlitter. + // Hence only tryBlitFatAntiRect when MaskAdditiveBlitter would have been used. + if (!TryBlitFatAntiRect(blitter, path, clipBounds)) { MaskAdditiveBlitter additiveBlitter(blitter, ir, clipBounds, isInverse); aaa_fill_path(path, clipBounds, &additiveBlitter, ir.fTop, ir.fBottom, containedInClip, true, forceRLE); -#else - // blitFatAntiRect is slower than the normal AAA flow without MaskAdditiveBlitter. - // Hence only tryBlitFatAntiRect when MaskAdditiveBlitter would have been used. - if (!TryBlitFatAntiRect(blitter, path, clipBounds)) { - MaskAdditiveBlitter additiveBlitter(blitter, ir, clipBounds, isInverse); - aaa_fill_path(path, clipBounds, &additiveBlitter, ir.fTop, ir.fBottom, - containedInClip, true, forceRLE); - } -#endif - } else if (!isInverse && path.isConvex()) { - // If the filling area is convex (i.e., path.isConvex && !isInverse), our simpler - // aaa_walk_convex_edges won't generate alphas above 255. Hence we don't need - // SafeRLEAdditiveBlitter (which is slow due to clamping). The basic RLE blitter - // RunBasedAdditiveBlitter would suffice. - RunBasedAdditiveBlitter additiveBlitter(blitter, ir, clipBounds, isInverse); - aaa_fill_path(path, clipBounds, &additiveBlitter, ir.fTop, ir.fBottom, - containedInClip, false, forceRLE); - } else { - // If the filling area might not be convex, the more involved aaa_walk_edges would - // be called and we have to clamp the alpha downto 255. The SafeRLEAdditiveBlitter - // does that at a cost of performance. - SafeRLEAdditiveBlitter additiveBlitter(blitter, ir, clipBounds, isInverse); - aaa_fill_path(path, clipBounds, &additiveBlitter, ir.fTop, ir.fBottom, - containedInClip, false, forceRLE); } - }; - - do_fill_path(path, origClip, blitter, forceRLE, 2, std::move(fillPathFunc)); +#endif + } else if (!isInverse && path.isConvex()) { + // If the filling area is convex (i.e., path.isConvex && !isInverse), our simpler + // aaa_walk_convex_edges won't generate alphas above 255. Hence we don't need + // SafeRLEAdditiveBlitter (which is slow due to clamping). The basic RLE blitter + // RunBasedAdditiveBlitter would suffice. + RunBasedAdditiveBlitter additiveBlitter(blitter, ir, clipBounds, isInverse); + aaa_fill_path(path, clipBounds, &additiveBlitter, ir.fTop, ir.fBottom, + containedInClip, false, forceRLE); + } else { + // If the filling area might not be convex, the more involved aaa_walk_edges would + // be called and we have to clamp the alpha downto 255. The SafeRLEAdditiveBlitter + // does that at a cost of performance. + SafeRLEAdditiveBlitter additiveBlitter(blitter, ir, clipBounds, isInverse); + aaa_fill_path(path, clipBounds, &additiveBlitter, ir.fTop, ir.fBottom, + containedInClip, false, forceRLE); + } } diff --git a/src/core/SkScan_AntiPath.cpp b/src/core/SkScan_AntiPath.cpp index ba42a72d17..1178c9c541 100644 --- a/src/core/SkScan_AntiPath.cpp +++ b/src/core/SkScan_AntiPath.cpp @@ -613,33 +613,155 @@ static bool ShouldUseAAA(const SkPath& path) { return path.countPoints() < SkTMax(bounds.width(), bounds.height()) / 2 - 10; } +void SkScan::SAAFillPath(const SkPath& path, SkBlitter* blitter, const SkIRect& ir, + const SkIRect& clipBounds, bool containedInClip, bool forceRLE) { + bool isInverse = path.isInverseFillType(); + + // MaskSuperBlitter can't handle drawing outside of ir, so we can't use it + // if we're an inverse filltype + if (!isInverse && MaskSuperBlitter::CanHandleRect(ir) && !forceRLE) { + MaskSuperBlitter superBlit(blitter, ir, clipBounds, isInverse); + SkASSERT(SkIntToScalar(ir.fTop) <= path.getBounds().fTop); + sk_fill_path(path, clipBounds, &superBlit, ir.fTop, ir.fBottom, SHIFT, containedInClip); + } else { + SuperBlitter superBlit(blitter, ir, clipBounds, isInverse); + sk_fill_path(path, clipBounds, &superBlit, ir.fTop, ir.fBottom, SHIFT, containedInClip); + } +} + +static bool fitsInsideLimit(const SkRect& r, SkScalar max) { + const SkScalar min = -max; + return r.fLeft > min && r.fTop > min && + r.fRight < max && r.fBottom < max; +} + +static int overflows_short_shift(int value, int shift) { + const int s = 16 + shift; + return (SkLeftShift(value, s) >> s) - value; +} + +/** + Would any of the coordinates of this rectangle not fit in a short, + when left-shifted by shift? +*/ +static int rect_overflows_short_shift(SkIRect rect, int shift) { + SkASSERT(!overflows_short_shift(8191, shift)); + SkASSERT(overflows_short_shift(8192, shift)); + SkASSERT(!overflows_short_shift(32767, 0)); + SkASSERT(overflows_short_shift(32768, 0)); + + // Since we expect these to succeed, we bit-or together + // for a tiny extra bit of speed. + return overflows_short_shift(rect.fLeft, shift) | + overflows_short_shift(rect.fRight, shift) | + overflows_short_shift(rect.fTop, shift) | + overflows_short_shift(rect.fBottom, shift); +} + +static bool safeRoundOut(const SkRect& src, SkIRect* dst, int32_t maxInt) { + const SkScalar maxScalar = SkIntToScalar(maxInt); + + if (fitsInsideLimit(src, maxScalar)) { + src.roundOut(dst); + return true; + } + return false; +} + void SkScan::AntiFillPath(const SkPath& path, const SkRegion& origClip, SkBlitter* blitter, bool forceRLE) { - if (ShouldUseDAA(path)) { - SkScan::DAAFillPath(path, origClip, blitter, forceRLE); + if (origClip.isEmpty()) { return; - } else if (ShouldUseAAA(path)) { - // Do not use AAA if path is too complicated: - // there won't be any speedup or significant visual improvement. - SkScan::AAAFillPath(path, origClip, blitter, forceRLE); + } + + const bool isInverse = path.isInverseFillType(); + SkIRect ir; + + if (!safeRoundOut(path.getBounds(), &ir, SK_MaxS32 >> SHIFT)) { + // Bounds can't fit in SkIRect; we'll return without drawing + return; + } + if (ir.isEmpty()) { + if (isInverse) { + blitter->blitRegion(origClip); + } return; } - FillPathFunc fillPathFunc = [](const SkPath& path, SkBlitter* blitter, bool isInverse, - const SkIRect& ir, const SkIRect& clipBounds, bool containedInClip, bool forceRLE){ - // MaskSuperBlitter can't handle drawing outside of ir, so we can't use it - // if we're an inverse filltype - if (!isInverse && MaskSuperBlitter::CanHandleRect(ir) && !forceRLE) { - MaskSuperBlitter superBlit(blitter, ir, clipBounds, isInverse); - SkASSERT(SkIntToScalar(ir.fTop) <= path.getBounds().fTop); - sk_fill_path(path, clipBounds, &superBlit, ir.fTop, ir.fBottom, SHIFT, containedInClip); - } else { - SuperBlitter superBlit(blitter, ir, clipBounds, isInverse); - sk_fill_path(path, clipBounds, &superBlit, ir.fTop, ir.fBottom, SHIFT, containedInClip); + // If the intersection of the path bounds and the clip bounds + // will overflow 32767 when << by SHIFT, we can't supersample, + // so draw without antialiasing. + SkIRect clippedIR; + if (isInverse) { + // If the path is an inverse fill, it's going to fill the entire + // clip, and we care whether the entire clip exceeds our limits. + clippedIR = origClip.getBounds(); + } else { + if (!clippedIR.intersect(ir, origClip.getBounds())) { + return; + } + } + if (rect_overflows_short_shift(clippedIR, SHIFT)) { + SkScan::FillPath(path, origClip, blitter); + return; + } + + // Our antialiasing can't handle a clip larger than 32767, so we restrict + // the clip to that limit here. (the runs[] uses int16_t for its index). + // + // A more general solution (one that could also eliminate the need to + // disable aa based on ir bounds (see overflows_short_shift) would be + // to tile the clip/target... + SkRegion tmpClipStorage; + const SkRegion* clipRgn = &origClip; + { + static const int32_t kMaxClipCoord = 32767; + const SkIRect& bounds = origClip.getBounds(); + if (bounds.fRight > kMaxClipCoord || bounds.fBottom > kMaxClipCoord) { + SkIRect limit = { 0, 0, kMaxClipCoord, kMaxClipCoord }; + tmpClipStorage.op(origClip, limit, SkRegion::kIntersect_Op); + clipRgn = &tmpClipStorage; } - }; + } + // for here down, use clipRgn, not origClip + + SkScanClipper clipper(blitter, clipRgn, ir); + + if (clipper.getBlitter() == nullptr) { // clipped out + if (isInverse) { + blitter->blitRegion(*clipRgn); + } + return; + } + + SkASSERT(clipper.getClipRect() == nullptr || + *clipper.getClipRect() == clipRgn->getBounds()); - do_fill_path(path, origClip, blitter, forceRLE, SHIFT, std::move(fillPathFunc)); + // now use the (possibly wrapped) blitter + blitter = clipper.getBlitter(); + + if (isInverse) { + sk_blit_above(blitter, ir, *clipRgn); + } + + SkASSERT(SkIntToScalar(ir.fTop) <= path.getBounds().fTop); + + if (ShouldUseDAA(path)) { + SkScan::DAAFillPath(path, blitter, ir, clipRgn->getBounds(), + clipper.getClipRect() == nullptr, forceRLE); + } else if (ShouldUseAAA(path)) { + // Do not use AAA if path is too complicated: + // there won't be any speedup or significant visual improvement. + SkScan::AAAFillPath(path, blitter, ir, clipRgn->getBounds(), + clipper.getClipRect() == nullptr, forceRLE); + } else { + SkScan::SAAFillPath(path, blitter, ir, clipRgn->getBounds(), + clipper.getClipRect() == nullptr, forceRLE); + } + + if (isInverse) { + sk_blit_below(blitter, ir, *clipRgn); + } } /////////////////////////////////////////////////////////////////////////////// diff --git a/src/core/SkScan_DAAPath.cpp b/src/core/SkScan_DAAPath.cpp index e213e34fa6..cd1717fd2d 100644 --- a/src/core/SkScan_DAAPath.cpp +++ b/src/core/SkScan_DAAPath.cpp @@ -315,48 +315,42 @@ void gen_alpha_deltas(const SkPath& path, const SkIRect& clipBounds, Deltas& res } } -void SkScan::DAAFillPath(const SkPath& path, const SkRegion& origClip, SkBlitter* blitter, - bool forceRLE) { - - FillPathFunc fillPathFunc = [](const SkPath& path, SkBlitter* blitter, bool isInverse, - const SkIRect& ir, const SkIRect& clipBounds, bool containedInClip, bool forceRLE){ - bool isEvenOdd = path.getFillType() & 1; - bool isConvex = path.isConvex(); - bool skipRect = isConvex && !isInverse; - - SkIRect clippedIR = ir; - clippedIR.intersect(clipBounds); - - // The overhead of even constructing SkCoverageDeltaList/Mask is too big. - // So TryBlitFatAntiRect and return if it's successful. - if (!isInverse && TryBlitFatAntiRect(blitter, path, clipBounds)) { - return; - } +// For threaded backend with out-of-order init-once, we probably have to take care of the +// blitRegion, sk_blit_above, sk_blit_below in SkScan::AntiFillPath to maintain the draw order. If +// we do that, be caureful that blitRect may throw exception if the rect is empty. +void SkScan::DAAFillPath(const SkPath& path, SkBlitter* blitter, const SkIRect& ir, + const SkIRect& clipBounds, bool containedInClip, bool forceRLE) { + bool isEvenOdd = path.getFillType() & 1; + bool isConvex = path.isConvex(); + bool isInverse = path.isInverseFillType(); + bool skipRect = isConvex && !isInverse; + + SkIRect clippedIR = ir; + clippedIR.intersect(clipBounds); + + // The overhead of even constructing SkCoverageDeltaList/Mask is too big. + // So TryBlitFatAntiRect and return if it's successful. + if (!isInverse && TryBlitFatAntiRect(blitter, path, clipBounds)) { + return; + } #ifdef GOOGLE3 - constexpr int STACK_SIZE = 12 << 10; // 12K stack size alloc; Google3 has 16K limit. + constexpr int STACK_SIZE = 12 << 10; // 12K stack size alloc; Google3 has 16K limit. #else - constexpr int STACK_SIZE = 64 << 10; // 64k stack size to avoid heap allocation + constexpr int STACK_SIZE = 64 << 10; // 64k stack size to avoid heap allocation #endif - SkSTArenaAlloc<STACK_SIZE> alloc; // avoid heap allocation with SkSTArenaAlloc - - // Only blitter->blitXXX need to be done in order in the threaded backend. - // Everything before can be done out of order in the threaded backend. - if (!forceRLE && !isInverse && SkCoverageDeltaMask::Suitable(clippedIR)) { - SkCoverageDeltaMask deltaMask(&alloc, clippedIR); - gen_alpha_deltas(path, clipBounds, deltaMask, blitter, skipRect, containedInClip); - deltaMask.convertCoverageToAlpha(isEvenOdd, isInverse, isConvex); - blitter->blitMask(deltaMask.prepareSkMask(), clippedIR); - } else { - SkCoverageDeltaList deltaList(&alloc, clippedIR.fTop, clippedIR.fBottom, forceRLE); - gen_alpha_deltas(path, clipBounds, deltaList, blitter, skipRect, containedInClip); - blitter->blitCoverageDeltas(&deltaList, clipBounds, isEvenOdd, isInverse, isConvex); - } - }; - - // For threaded backend with out-of-order init-once (and therefore out-of-order do_fill_path), - // we probably have to take care of the blitRegion, sk_blit_above, sk_blit_below in do_fill_path - // to maintain the draw order. If we do that, be caureful that blitRect may throw exception is - // the rect is empty. - do_fill_path(path, origClip, blitter, forceRLE, 2, std::move(fillPathFunc)); + SkSTArenaAlloc<STACK_SIZE> alloc; // avoid heap allocation with SkSTArenaAlloc + + // Only blitter->blitXXX needs to be done in order in the threaded backend. + // Everything before can be done out of order in the threaded backend. + if (!forceRLE && !isInverse && SkCoverageDeltaMask::Suitable(clippedIR)) { + SkCoverageDeltaMask deltaMask(&alloc, clippedIR); + gen_alpha_deltas(path, clipBounds, deltaMask, blitter, skipRect, containedInClip); + deltaMask.convertCoverageToAlpha(isEvenOdd, isInverse, isConvex); + blitter->blitMask(deltaMask.prepareSkMask(), clippedIR); + } else { + SkCoverageDeltaList deltaList(&alloc, clippedIR.fTop, clippedIR.fBottom, forceRLE); + gen_alpha_deltas(path, clipBounds, deltaList, blitter, skipRect, containedInClip); + blitter->blitCoverageDeltas(&deltaList, clipBounds, isEvenOdd, isInverse, isConvex); + } } |