From: bsalomon@google.com <bsalomon@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81>
Date: Fri, 15 Jun 2012 21:50:15 +0000 (+0000)
Subject: Avoid un-necessary re-calculations of width in for loop of emitFS function
X-Git-Tag: submit/tizen/20180928.044319~15867
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=fc0d23a6ba476c7fac0d0581880a930c1951efa0;p=platform%2Fupstream%2FlibSkiaSharp.git

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 d1ccc917ec..d0462c6292 100644
--- 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);