Avoid un-necessary re-calculations of width in for loop of emitFS function

Committed on behalf of kondapallykalyan@gmail.com

Review URL: https://codereview.appspot.com/6294075/

git-svn-id: http://skia.googlecode.com/svn/trunk@4270 2bbb7eff-a529-9590-31e7-b0007b416f81

diff --git a/src/gpu/effects/GrConvolutionEffect.cpp b/src/gpu/effects/GrConvolutionEffect.cpp
index d1ccc917ec27173da960c296e04c35cbcfbf96a1..d0462c6292c161aba62aeecf76a8cfad1e9966c5 100644 (file)
--- a/src/gpu/effects/GrConvolutionEffect.cpp
+++ b/src/gpu/effects/GrConvolutionEffect.cpp
@@ -88,9 +88,10 @@ void GrGLConvolutionEffect::emitFS(GrGLShaderBuilder* state,
     code->appendf("\t\t%s = vec4(0, 0, 0, 0);\n", outputColor);
 
     code->appendf("\t\tvec2 coord = %s;\n", state->fSampleCoords.c_str());
-
+    
+    int width = this ->width();
     // Manually unroll loop because some drivers don't; yields 20-30% speedup.
-    for (int i = 0; i < this->width(); i++) {
+    for (int i = 0; i < width; i++) {
         GrStringBuilder index;
         GrStringBuilder kernelIndex;
         index.appendS32(i);