Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(7731)

Unified Diff: bench/LightingBench.cpp

Issue 23876006: Refactoring: get rid of the SkBenchmark void* parameter. (Closed) Base URL: https://skia.googlecode.com/svn/trunk
Patch Set: sync to head Created 7 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « bench/InterpBench.cpp ('k') | bench/LineBench.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: bench/LightingBench.cpp
diff --git a/bench/LightingBench.cpp b/bench/LightingBench.cpp
index 37c098b2f6c207547772b25d3028bd9a06c7c344..7a1b5998d8ddf2965dfe9c4a8504980964e41448 100644
--- a/bench/LightingBench.cpp
+++ b/bench/LightingBench.cpp
@@ -17,8 +17,7 @@
class LightingBaseBench : public SkBenchmark {
public:
- LightingBaseBench(void* param, bool small) : INHERITED(param), fIsSmall(small) {
- }
+ LightingBaseBench(bool small) : fIsSmall(small) { }
protected:
void draw(SkCanvas* canvas, SkImageFilter* imageFilter) const {
@@ -98,7 +97,7 @@ protected:
class LightingPointLitDiffuseBench : public LightingBaseBench {
public:
- LightingPointLitDiffuseBench(void* param, bool small) : INHERITED(param, small) {
+ LightingPointLitDiffuseBench(bool small) : INHERITED(small) {
}
protected:
@@ -117,7 +116,7 @@ private:
class LightingDistantLitDiffuseBench : public LightingBaseBench {
public:
- LightingDistantLitDiffuseBench(void* param, bool small) : INHERITED(param, small) {
+ LightingDistantLitDiffuseBench(bool small) : INHERITED(small) {
}
protected:
@@ -136,7 +135,7 @@ private:
class LightingSpotLitDiffuseBench : public LightingBaseBench {
public:
- LightingSpotLitDiffuseBench(void* param, bool small) : INHERITED(param, small) {
+ LightingSpotLitDiffuseBench(bool small) : INHERITED(small) {
}
protected:
@@ -156,7 +155,7 @@ private:
class LightingPointLitSpecularBench : public LightingBaseBench {
public:
- LightingPointLitSpecularBench(void* param, bool small) : INHERITED(param, small) {
+ LightingPointLitSpecularBench(bool small) : INHERITED(small) {
}
protected:
@@ -175,7 +174,7 @@ private:
class LightingDistantLitSpecularBench : public LightingBaseBench {
public:
- LightingDistantLitSpecularBench(void* param, bool small) : INHERITED(param, small) {
+ LightingDistantLitSpecularBench(bool small) : INHERITED(small) {
}
protected:
@@ -194,7 +193,7 @@ private:
class LightingSpotLitSpecularBench : public LightingBaseBench {
public:
- LightingSpotLitSpecularBench(void* param, bool small) : INHERITED(param, small) {
+ LightingSpotLitSpecularBench(bool small) : INHERITED(small) {
}
protected:
@@ -214,15 +213,15 @@ private:
///////////////////////////////////////////////////////////////////////////////
-DEF_BENCH( return new LightingPointLitDiffuseBench(p, true); )
-DEF_BENCH( return new LightingPointLitDiffuseBench(p, false); )
-DEF_BENCH( return new LightingDistantLitDiffuseBench(p, true); )
-DEF_BENCH( return new LightingDistantLitDiffuseBench(p, false); )
-DEF_BENCH( return new LightingSpotLitDiffuseBench(p, true); )
-DEF_BENCH( return new LightingSpotLitDiffuseBench(p, false); )
-DEF_BENCH( return new LightingPointLitSpecularBench(p, true); )
-DEF_BENCH( return new LightingPointLitSpecularBench(p, false); )
-DEF_BENCH( return new LightingDistantLitSpecularBench(p, true); )
-DEF_BENCH( return new LightingDistantLitSpecularBench(p, false); )
-DEF_BENCH( return new LightingSpotLitSpecularBench(p, true); )
-DEF_BENCH( return new LightingSpotLitSpecularBench(p, false); )
+DEF_BENCH( return new LightingPointLitDiffuseBench(true); )
+DEF_BENCH( return new LightingPointLitDiffuseBench(false); )
+DEF_BENCH( return new LightingDistantLitDiffuseBench(true); )
+DEF_BENCH( return new LightingDistantLitDiffuseBench(false); )
+DEF_BENCH( return new LightingSpotLitDiffuseBench(true); )
+DEF_BENCH( return new LightingSpotLitDiffuseBench(false); )
+DEF_BENCH( return new LightingPointLitSpecularBench(true); )
+DEF_BENCH( return new LightingPointLitSpecularBench(false); )
+DEF_BENCH( return new LightingDistantLitSpecularBench(true); )
+DEF_BENCH( return new LightingDistantLitSpecularBench(false); )
+DEF_BENCH( return new LightingSpotLitSpecularBench(true); )
+DEF_BENCH( return new LightingSpotLitSpecularBench(false); )
« no previous file with comments | « bench/InterpBench.cpp ('k') | bench/LineBench.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698