Code review fixes

Author: kocsismate

Zend/tests/readonly_props/readonly_assign_no_sideeffect.phpt

@@ -8,8 +8,7 @@ class Foo {
         public readonly string $bar
     ) {}
 
-    public function write()
-    {
+    public function write() {
         $this->bar = new S();
     }
 }

Zend/zend_execute.c

@@ -1009,9 +1009,7 @@ static zend_never_inline zval* zend_assign_to_typed_prop(zend_property_info *inf
 {
 	zval tmp;
 
-	if (UNEXPECTED(
-		(info->flags & ZEND_ACC_READONLY) && !(Z_PROP_FLAG_P(property_val) & IS_PROP_REINITABLE)
-	)) {
+	if (UNEXPECTED((info->flags & ZEND_ACC_READONLY) && !(Z_PROP_FLAG_P(property_val) & IS_PROP_REINITABLE))) {
 		zend_readonly_property_modification_error(info);
 		return &EG(uninitialized_zval);
 	}
@@ -1024,9 +1022,7 @@ static zend_never_inline zval* zend_assign_to_typed_prop(zend_property_info *inf
 		return &EG(uninitialized_zval);
 	}
 
-	if (UNEXPECTED(info->flags & ZEND_ACC_READONLY)) {
-		Z_PROP_FLAG_P(property_val) &= ~IS_PROP_REINITABLE;
-	}
+	Z_PROP_FLAG_P(property_val) &= ~IS_PROP_REINITABLE;
 
 	return zend_assign_to_variable(property_val, &tmp, IS_TMP_VAR, EX_USES_STRICT_TYPES());
 }

Zend/zend_object_handlers.c

@@ -837,9 +837,7 @@ ZEND_API zval *zend_std_write_property(zend_object *zobj, zend_string *name, zva
 					variable_ptr = &EG(error_zval);
 					goto exit;
 				}
-				if (UNEXPECTED(prop_info->flags & ZEND_ACC_READONLY)) {
-					Z_PROP_FLAG_P(variable_ptr) &= ~IS_PROP_REINITABLE;
-				}
+				Z_PROP_FLAG_P(variable_ptr) &= ~IS_PROP_REINITABLE;
 				value = &tmp;
 			}