1000iq fix

src/tests/exprtype/exprtype_test.go

@@ -2201,72 +2201,44 @@ function f() {
 	runExprTypeTest(t, &exprTypeTestParams{code: code})
 }
 
-func TestArrayTypesWithUnpack_Primitives(t *testing.T) {
+func TestArrayTypesWithUnpackPrimitives(t *testing.T) {
 	code := `<?php
-declare(strict_types = 1);
 function f() {
 // with int type
-$a = [1, 2, 3];
-$b = [0, ...$a];
-exprtype($b, "int[]");
+$a = 1;
+$b = [...$a];
+exprtype($b, "mixed");
 
 // with float type
-$a1 = [1.1, 2.2, 3.3];
-$b1 = [0.0, ...$a1];
-exprtype($b1, "float[]");
+$a1 = 1.1;
+$b1 = [...$a1];
+exprtype($b1, "mixed");
 
 // with string type
-$a2 = ["a", "b", "c"];
-$b2 = ["d", ...$a2];
-exprtype($b2, "string[]");
+$a2 = "a";
+$b2 = [...$a2];
+exprtype($b2, "mixed");
 
 // with bool type
-$a3 = [true, false, true];
-$b3 = [false, ...$a3];
-exprtype($b3, "bool[]");
+$a3 = true;
+$b3 = [...$a3];
+exprtype($b3, "mixed");
 
-// mixed types
-$a4 = [1, "a", 3.3, true];
-$b4 = ["start", ...$a4];
-exprtype($b4, "mixed[]");
+// null
+ $a4 = null;
+ $b4 = [...$a4];
+exprtype($b4, "mixed");
 
-// with two unpack
-$a5 = [1, 2, 3];
-$b5 = [4, 5, 6];
-$c5 = [...$a5, ...$b5];
-exprtype($c5, "int[]");
-
-// with two unpack with different types
-$a6 = [1, 2, 3];
-$c6 = [...$a6, ...$a2];
-exprtype($c6, "mixed[]");
-
-// one unpack
-$a7 = [true, false, true];
-$b7 = [...$a7];
-exprtype($b7, "bool[]");
-
-// with two unpack and just type
-$a8 = [1.1, 2.2, 3.3];
-$b8 = [4.4, 5.5, 6.6];
-$c8 = [...$a8, 7.7, ...$b8];
-exprtype($c8, "float[]");
+// class
+class Foo {}
+ $a5 = new Foo();
+ $b5 = [...$a5];
+exprtype($b5, "mixed");
 }
 `
 	runExprTypeTest(t, &exprTypeTestParams{code: code})
 }
 
-// @see issue1214
-func TestArrayTypesWithUnpackNullNegative(t *testing.T) {
-	linttest.SimpleNegativeTest(t, `<?php
-declare(strict_types = 1);
-
-function test() {
- $special_items = null;
- $_ = [...$special_items];
-}`)
-}
-
 func TestPropertyTypeHints(t *testing.T) {
 	code := `<?php
 class Too {}

src/types/lazy.go

@@ -291,8 +291,12 @@ func FormatSimpleType(s string) (res string) {
 	return s
 }
 
+func IsAfterWMaxed(typ string) (res bool) {
+	return typ == "" || typ[0] >= WMax
+}
+
 func FormatType(s string) (res string) {
-	if s == "" || s[0] >= WMax {
+	if IsAfterWMaxed(s) {
 		return FormatSimpleType(s)
 	}
 

src/types/map.go

@@ -411,7 +411,7 @@ func (m Map) LazyArrayElemType() Map {
 	mm := make(map[string]struct{}, m.Len())
 	for typ := range m.m {
 		// TODO: https://github.com/VKCOM/noverify/issues/1227
-		if typ == "null" {
+		if IsAfterWMaxed(typ) {
 			break
 		}