diff options
author | 2016-02-03 12:25:40 -0800 | |
---|---|---|
committer | 2016-02-03 12:25:40 -0800 | |
commit | 9cdb9920fcad286ecf7875ea19902022b644fbdc (patch) | |
tree | d3bd4068495f5b18530889c60c73f6ce27489323 /src/gpu/effects/GrConvexPolyEffect.cpp | |
parent | d32d19a2299d03fa9e9da9aa8b5383c7f64b9495 (diff) |
Clean up GrGLSLFragmentProcessor-derived classes
In some other patches it was observed that many of the GrGLSLFragmentProcessor-derived classes needlessly pass a parameter to their constructors.
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1666773002
Review URL: https://codereview.chromium.org/1666773002
Diffstat (limited to 'src/gpu/effects/GrConvexPolyEffect.cpp')
-rw-r--r-- | src/gpu/effects/GrConvexPolyEffect.cpp | 24 |
1 files changed, 10 insertions, 14 deletions
diff --git a/src/gpu/effects/GrConvexPolyEffect.cpp b/src/gpu/effects/GrConvexPolyEffect.cpp index dc7a528501..f95d1a2fb3 100644 --- a/src/gpu/effects/GrConvexPolyEffect.cpp +++ b/src/gpu/effects/GrConvexPolyEffect.cpp @@ -81,9 +81,11 @@ const GrFragmentProcessor* AARectEffect::TestCreate(GrProcessorTestData* d) { class GLAARectEffect : public GrGLSLFragmentProcessor { public: - GLAARectEffect(const GrProcessor&); + GLAARectEffect() { + fPrevRect.fLeft = SK_ScalarNaN; + } - virtual void emitCode(EmitArgs&) override; + void emitCode(EmitArgs&) override; static inline void GenKey(const GrProcessor&, const GrGLSLCaps&, GrProcessorKeyBuilder*); @@ -96,10 +98,6 @@ private: typedef GrGLSLFragmentProcessor INHERITED; }; -GLAARectEffect::GLAARectEffect(const GrProcessor& effect) { - fPrevRect.fLeft = SK_ScalarNaN; -} - void GLAARectEffect::emitCode(EmitArgs& args) { const AARectEffect& aare = args.fFp.cast<AARectEffect>(); const char *rectName; @@ -161,16 +159,18 @@ void AARectEffect::onGetGLSLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyB } GrGLSLFragmentProcessor* AARectEffect::onCreateGLSLInstance() const { - return new GLAARectEffect(*this); + return new GLAARectEffect; } ////////////////////////////////////////////////////////////////////////////// class GrGLConvexPolyEffect : public GrGLSLFragmentProcessor { public: - GrGLConvexPolyEffect(const GrProcessor&); + GrGLConvexPolyEffect() { + fPrevEdges[0] = SK_ScalarNaN; + } - virtual void emitCode(EmitArgs&) override; + void emitCode(EmitArgs&) override; static inline void GenKey(const GrProcessor&, const GrGLSLCaps&, GrProcessorKeyBuilder*); @@ -183,10 +183,6 @@ private: typedef GrGLSLFragmentProcessor INHERITED; }; -GrGLConvexPolyEffect::GrGLConvexPolyEffect(const GrProcessor&) { - fPrevEdges[0] = SK_ScalarNaN; -} - void GrGLConvexPolyEffect::emitCode(EmitArgs& args) { const GrConvexPolyEffect& cpe = args.fFp.cast<GrConvexPolyEffect>(); @@ -309,7 +305,7 @@ void GrConvexPolyEffect::onGetGLSLProcessorKey(const GrGLSLCaps& caps, } GrGLSLFragmentProcessor* GrConvexPolyEffect::onCreateGLSLInstance() const { - return new GrGLConvexPolyEffect(*this); + return new GrGLConvexPolyEffect; } GrConvexPolyEffect::GrConvexPolyEffect(GrPrimitiveEdgeType edgeType, int n, const SkScalar edges[]) |