diff options
author | 2012-03-29 15:18:04 +0000 | |
---|---|---|
committer | 2012-03-29 15:18:04 +0000 | |
commit | 54924243c1b65b3ee6d8fa064b50a9b1bb2a19a5 (patch) | |
tree | e27f07f67a7d5ecd2149ced9d389c1a078ebb656 /src/effects/SkPixelXorXfermode.cpp | |
parent | 5a64e58ebb19228828eaa800822afa4d9510ccf2 (diff) |
Cleanup code related to SkFlattenable::flatten()
The following changes were made by this CL:
1. Make flatten() a protected method as callers should use
SkFlattenableWriteBuffer to flatten an object
2. Make flatten a const method (including subclasses)
3. Mark subclass implementation of flatten with SK_OVERRIDE
4. Ensure overridden flatten impls call their parent
5. Remove no-op implementations of flatten from subclasses
Additionally, if necessary the unflattening constructor was
also moved to the protected section of the subclasses
header if it was not already there.
git-svn-id: http://skia.googlecode.com/svn/trunk@3540 2bbb7eff-a529-9590-31e7-b0007b416f81
Diffstat (limited to 'src/effects/SkPixelXorXfermode.cpp')
-rw-r--r-- | src/effects/SkPixelXorXfermode.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/effects/SkPixelXorXfermode.cpp b/src/effects/SkPixelXorXfermode.cpp index b4bbaf970c..7b58ed76a8 100644 --- a/src/effects/SkPixelXorXfermode.cpp +++ b/src/effects/SkPixelXorXfermode.cpp @@ -18,13 +18,13 @@ SkPMColor SkPixelXorXfermode::xferColor(SkPMColor src, SkPMColor dst) { return res; } -void SkPixelXorXfermode::flatten(SkFlattenableWriteBuffer& wb) { +void SkPixelXorXfermode::flatten(SkFlattenableWriteBuffer& wb) const { this->INHERITED::flatten(wb); wb.write32(fOpColor); } SkPixelXorXfermode::SkPixelXorXfermode(SkFlattenableReadBuffer& rb) - : SkXfermode(rb) { + : INHERITED(rb) { fOpColor = rb.readU32(); } |