aboutsummaryrefslogtreecommitdiffhomepage
path: root/include/effects/SkLayerDrawLooper.h
diff options
context:
space:
mode:
authorGravatar mdempsky <mdempsky@chromium.org>2015-09-24 15:04:45 -0700
committerGravatar Commit bot <commit-bot@chromium.org>2015-09-24 15:04:45 -0700
commit00d6e515e5835f7df6163ceb5f5ceb1770552bf7 (patch)
tree33e26b1cb307b54eae83a87c1bdaafd0691011c1 /include/effects/SkLayerDrawLooper.h
parent8db65a6d0c60d0e31eadc64d427f6e7f050e1140 (diff)
Eliminate some clutter in SkFlattenable
The Registrar class is unnecessary, as SkFlattenable factory registration is now handled via initialization routines that can just call the Register function directly. Also, no need to lazily initialize gCount to 0, as initializing an int to a constant value does not require dynamic initialization. (C++ actually guarantees zero initialization of global ints anyway, but existing practice in Skia appears to favor the explicit "= 0"). Relatedly, this requires removing the unused/unimplemented SkLayerDrawLooper::MyRegistrar class. And removing that allows Clang to realize that SkLayerDrawLooper::fTopRec is unneeded too, so remove that too to squelch the compiler warning/error. This doesn't change any public API. TBR=reed@google.com Review URL: https://codereview.chromium.org/1361323002
Diffstat (limited to 'include/effects/SkLayerDrawLooper.h')
-rw-r--r--include/effects/SkLayerDrawLooper.h6
1 files changed, 0 insertions, 6 deletions
diff --git a/include/effects/SkLayerDrawLooper.h b/include/effects/SkLayerDrawLooper.h
index c59b8b006a..76172dfd4b 100644
--- a/include/effects/SkLayerDrawLooper.h
+++ b/include/effects/SkLayerDrawLooper.h
@@ -94,7 +94,6 @@ private:
LayerInfo fInfo;
};
Rec* fRecs;
- Rec* fTopRec;
int fCount;
// state-machine during the init/next cycle
@@ -111,11 +110,6 @@ private:
static void ApplyInfo(SkPaint* dst, const SkPaint& src, const LayerInfo&);
};
- class MyRegistrar : public SkFlattenable::Registrar {
- public:
- MyRegistrar();
- };
-
typedef SkDrawLooper INHERITED;
public: