commit 51b2685bbc8609dd15de8727f8fb7a7add0e8a0b
author jbauman@chromium.org <jbauman@chromium.org@736b8ea6-26fd-11df-bfd4-992fa37f6226> Tue Feb 21 23:53:23 2012 +0000
committer jbauman@chromium.org <jbauman@chromium.org@736b8ea6-26fd-11df-bfd4-992fa37f6226> Tue Feb 21 23:53:23 2012 +0000
tree adeda2269090268713c63cdf6a2c02c8aaa8ce71
parent 0725e7df04131fd2902f467b54bc8c3dd8474020

Don't use strlen inside of loop

This could hurt performance quite a bit.

BUG=296
TEST=

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

git-svn-id: https://angleproject.googlecode.com/svn/trunk@993 736b8ea6-26fd-11df-bfd4-992fa37f6226

src/libGLESv2/Shader.cpp

diff --git a/src/libGLESv2/Shader.cpp b/src/libGLESv2/Shader.cpp
index 6277e84..ac58abe 100644
--- a/src/libGLESv2/Shader.cpp
+++ b/src/libGLESv2/Shader.cpp
@@ -104,11 +104,8 @@
 
     if (mInfoLog)
     {
-        while (index < bufSize - 1 && index < (int)strlen(mInfoLog))
-        {
-            infoLog[index] = mInfoLog[index];
-            index++;
-        }
+        index = std::min(bufSize - 1, (int)strlen(mInfoLog));
+        memcpy(infoLog, mInfoLog, index);
     }
 
     if (bufSize)
@@ -152,11 +149,8 @@
 
     if (source)
     {
-        while (index < bufSize - 1 && index < (int)strlen(source))
-        {
-            buffer[index] = source[index];
-            index++;
-        }
+        index = std::min(bufSize - 1, (int)strlen(source));
+        memcpy(buffer, source, index);
     }
 
     if (bufSize)