Address review comments

Author: Girgias

Zend/tests/type_declarations/standalone_null.phpt

@@ -10,7 +10,5 @@ function test(null $v): null {
 var_dump(test(null));
 
 ?>
-===DONE===
 --EXPECT--
 NULL
-===DONE===

Zend/tests/type_declarations/typed_properties_110.phpt

@@ -7,7 +7,14 @@ class Foo {
 }
 
 $foo = new Foo();
+$foo->value = null;
+
+try {
+    $foo->value = 1;
+} catch (\TypeError $e) {
+    echo $e->getMessage();
+}
+
 ?>
-===DONE===
 --EXPECT--
-===DONE===
+Cannot assign int to property Foo::$value of type null

Zend/tests/type_declarations/typed_properties_111.phpt

@@ -7,7 +7,14 @@ class Foo {
 }
 
 $foo = new Foo();
+$foo->value = false;
+
+try {
+    $foo->value = true;
+} catch (\TypeError $e) {
+    echo $e->getMessage();
+}
+
 ?>
-===DONE===
 --EXPECT--
-===DONE===
+Cannot assign bool to property Foo::$value of type false

Zend/zend_compile.c

@@ -6186,11 +6186,11 @@ static bool zend_type_contains_traversable(zend_type type) {
 static zend_type zend_compile_typename(
 		zend_ast *ast, bool force_allow_null) /* {{{ */
 {
-	bool is_marked_nullable = false;
+	bool is_marked_nullable = ast->attr & ZEND_TYPE_NULLABLE;
 	zend_ast_attr orig_ast_attr = ast->attr;
 	zend_type type = ZEND_TYPE_INIT_NONE(0);
-	if (ast->attr & ZEND_TYPE_NULLABLE) {
-		is_marked_nullable = true;
+
+	if (is_marked_nullable) {
 		ast->attr &= ~ZEND_TYPE_NULLABLE;
 	}