commit ed6e54f1b4f8499ee73bbf331b7fe9f06641d8f2
author Ethan Nicholas <ethannicholas@google.com> Mon Jul 19 13:33:47 2021 -0400
committer Ethan Nicholas <ethannicholas@google.com> Mon Jul 19 19:19:59 2021 +0000
tree 6387e23152fe53eef90a8b15f5c468c389a3a903
parent 92c7cecad2d83ee79b0f334f5f38f8faec05f61c

Improved DSL Array error checking

Change-Id: I4b1235f7363309bd2aa3e2f1b3e00c94b43976bb
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/430058
Reviewed-by: John Stiles <johnstiles@google.com>
Commit-Queue: Ethan Nicholas <ethannicholas@google.com>

tests/SkSLDSLTest.cpp

diff --git a/tests/SkSLDSLTest.cpp b/tests/SkSLDSLTest.cpp
index dbd55ca..216d27e 100644
--- a/tests/SkSLDSLTest.cpp
+++ b/tests/SkSLDSLTest.cpp
@@ -556,6 +556,16 @@
     DSLExpression e = x + 1;
     REPORTER_ASSERT(r, e.type().isFloat());
     e.release();
+
+    {
+        ExpectError error(r, "error: array size must be positive\n");
+        Array(kFloat_Type, -1);
+    }
+
+    {
+        ExpectError error(r, "error: multidimensional arrays are not permitted\n");
+        Array(Array(kFloat_Type, 2), 2);
+    }
 }
 
 DEF_GPUTEST_FOR_MOCK_CONTEXT(DSLMatrices, r, ctxInfo) {