skia2/tests/sksl/errors/ArrayOfVoid.glsl

### Compilation failed:

error: 3: type 'void' not allowed in this context
error: 4: type 'void' not allowed in this context
error: 6: type 'void' may not be used in an array
error: 7: type 'void' not allowed in this context
error: 8: type 'void' not allowed in this context
error: 9: type 'void' may not be used in an array
6 errors
Fix assertion when arrays are double-declared. Multi-dimensional arrays aren't legal in GLSL/SkSL, so this should be caught and flagged as an error. The parser now verifies that a variable's type isn't an array-type before accepting a `[` token to open an array on the variable name. This CL also refactors the IR generator's `convertArraySize` method to make sure that various checks are made for all callers. Originally this restructuring was used to verify array multi-dimensionality, but that didn't detect errors inside struct declarations (which get no error checking inside the IR generator) so the IR generator updates no longer need to check the array dimensions. Bug: skia:11322 Change-Id: Id33f4bdfb544019ddf995a8196c3c09cfe5a4525 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/369916 Auto-Submit: John Stiles <johnstiles@google.com> Reviewed-by: Brian Osman <brianosman@google.com> Reviewed-by: Ethan Nicholas <ethannicholas@google.com> Commit-Queue: John Stiles <johnstiles@google.com> 2021-02-12 22:07:51 +00:00			`### Compilation failed:`

			`error: 3: type 'void' not allowed in this context`
			`error: 4: type 'void' not allowed in this context`
Allow function declarations to take (void). This is allowed by GLSL, so we allow it too. GLSL ES 1.0, Section 6.1: "The idiom “(void)” as a parameter list is provided for convenience." Change-Id: I551c505d3de518a75acd5e306f09f0f0767e43f2 Bug: skia:12025 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/411300 Commit-Queue: John Stiles <johnstiles@google.com> Reviewed-by: Brian Osman <brianosman@google.com> 2021-05-21 18:11:38 +00:00			`error: 6: type 'void' may not be used in an array`
Fix assertion when arrays are double-declared. Multi-dimensional arrays aren't legal in GLSL/SkSL, so this should be caught and flagged as an error. The parser now verifies that a variable's type isn't an array-type before accepting a `[` token to open an array on the variable name. This CL also refactors the IR generator's `convertArraySize` method to make sure that various checks are made for all callers. Originally this restructuring was used to verify array multi-dimensionality, but that didn't detect errors inside struct declarations (which get no error checking inside the IR generator) so the IR generator updates no longer need to check the array dimensions. Bug: skia:11322 Change-Id: Id33f4bdfb544019ddf995a8196c3c09cfe5a4525 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/369916 Auto-Submit: John Stiles <johnstiles@google.com> Reviewed-by: Brian Osman <brianosman@google.com> Reviewed-by: Ethan Nicholas <ethannicholas@google.com> Commit-Queue: John Stiles <johnstiles@google.com> 2021-02-12 22:07:51 +00:00			`error: 7: type 'void' not allowed in this context`
Allow function declarations to take (void). This is allowed by GLSL, so we allow it too. GLSL ES 1.0, Section 6.1: "The idiom “(void)” as a parameter list is provided for convenience." Change-Id: I551c505d3de518a75acd5e306f09f0f0767e43f2 Bug: skia:12025 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/411300 Commit-Queue: John Stiles <johnstiles@google.com> Reviewed-by: Brian Osman <brianosman@google.com> 2021-05-21 18:11:38 +00:00			`error: 8: type 'void' not allowed in this context`
Improve error-checking during array-size conversion. The fuzzer noticed insufficient guards in IndexExpression::Convert when converting an array size from an IntLiteral to a SKSL_INT. We had code in IRGenerator which did this properly, so I moved our array-size conversion logic into SkSLType and had IndexExpression share it. Also, a variety of tests around similar error conditions were added. Change-Id: I51529dea25f9029f81ae236511610069d66be29f Bug: oss-fuzz:37462 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/441236 Commit-Queue: John Stiles <johnstiles@google.com> Auto-Submit: John Stiles <johnstiles@google.com> Reviewed-by: Brian Osman <brianosman@google.com> 2021-08-23 13:24:33 +00:00			`error: 9: type 'void' may not be used in an array`
			`6 errors`