From 9ea3d57fde28a5fe4487a111dc3dd49418235e5e Mon Sep 17 00:00:00 2001 From: senorblanco Date: Tue, 8 Jul 2014 09:16:22 -0700 Subject: Clean up SkImageFilter constructors. Now that all creation of SkImageFilters goes through factory Create() methods, there's no real reason for the convenience constructors. Some SkImageFilter subclasses which actually have zero DAG-able inputs were passing NULL to the superclass constructor. This actually means 1 input, with a NULL value, not zero inputs. This becomes more relevant for the upcoming cache infrastructure, where this indicates that the filter will use its src input, where in fact some of these filters do not (they are image generators only). Limiting SkImageFilter to a single constructor resolves this ambiguity. Along the way, I removed all of the default parameters to the constructors, since the Create methods always call them with the full argument list. BUG=skia: R=reed@google.com Author: senorblanco@chromium.org Review URL: https://codereview.chromium.org/376953003 --- include/core/SkImageFilter.h | 6 ------ 1 file changed, 6 deletions(-) (limited to 'include/core') diff --git a/include/core/SkImageFilter.h b/include/core/SkImageFilter.h index 012a7f728f..a571184aa2 100644 --- a/include/core/SkImageFilter.h +++ b/include/core/SkImageFilter.h @@ -201,12 +201,6 @@ public: protected: SkImageFilter(int inputCount, SkImageFilter** inputs, const CropRect* cropRect = NULL); - // Convenience constructor for 1-input filters. - explicit SkImageFilter(SkImageFilter* input, const CropRect* cropRect = NULL); - - // Convenience constructor for 2-input filters. - SkImageFilter(SkImageFilter* input1, SkImageFilter* input2, const CropRect* cropRect = NULL); - virtual ~SkImageFilter(); /** -- cgit v1.2.3