commit 406145dc1eb3bd909b9528d2443ed414570dda24
author Lingfeng Yang <lfy@google.com> Mon Jan 06 16:26:44 2020 -0800
committer Lingfeng Yang <lfy@google.com> Tue Jan 07 00:37:53 2020 +0000
tree 835b70d8b51fcc4060bd8c69c630f895cc9f3eac
parent d7b8efd32c66b8ecd09e5d326ebe2801bb02b353

Revert "glShaderSource should respect length parameter"

Reason: breaks dEQP-GLES2.functional.shader_api in negative length parameter case
This reverts commit 481836a5a6ed03e14f9df9745c3734ffc5d1a204.
Change-Id: I8989aaebc7feffe1462f741ceda8f56303b19279

system/GLESv2_enc/GL2Encoder.cpp

diff --git a/system/GLESv2_enc/GL2Encoder.cpp b/system/GLESv2_enc/GL2Encoder.cpp
index 686323b..783812c 100755
--- a/system/GLESv2_enc/GL2Encoder.cpp
+++ b/system/GLESv2_enc/GL2Encoder.cpp
@@ -1743,15 +1743,8 @@
 
     // Track original sources---they may be translated in the backend
     std::vector<std::string> orig_sources;
-    if (length) {
-        for (int i = 0; i < count; i++) {
-            orig_sources.push_back(std::string((const char*)(string[i]),
-                                               (const char*)(string[i]) + length[i]));
-        }
-    } else {
-        for (int i = 0; i < count; i++) {
-            orig_sources.push_back(std::string((const char*)(string[i])));
-        }
+    for (int i = 0; i < count; i++) {
+        orig_sources.push_back(std::string((const char*)(string[i])));
     }
     shaderData->sources = orig_sources;