Fix [-Wjump-misses-init] in spl_directory.c by adding an inner scope

Girgias · Girgias · commit 925679b057a6 · 2020-04-18T14:52:48.000+02:00
diff --git a/ext/spl/spl_directory.c b/ext/spl/spl_directory.c
@@ -516,6 +516,7 @@ static spl_filesystem_object *spl_filesystem_object_create_type(int num_args, sp
 			}
 			break;
 		case SPL_FS_FILE:
+		{
 			ce = ce ? ce : source->file_class;
 
 			if (UNEXPECTED(zend_update_class_constants(ce) != SUCCESS)) {
@@ -563,6 +564,7 @@ static spl_filesystem_object *spl_filesystem_object_create_type(int num_args, sp
 				}
 			}
 			break;
+		}
 		case SPL_FS_DIR:
 			zend_restore_error_handling(&error_handling);
 			zend_throw_exception_ex(spl_ce_RuntimeException, 0, "Operation not supported");

Original file line number	Diff line number	Diff line change
`@@ -516,6 +516,7 @@ static spl_filesystem_object *spl_filesystem_object_create_type(int num_args, sp`
`516`	`516`	`}`
`517`	`517`	`break;`
`518`	`518`	`case SPL_FS_FILE:`
	`519`	`+ {`
`519`	`520`	`ce = ce ? ce : source->file_class;`
`520`	`521`
`521`	`522`	`if (UNEXPECTED(zend_update_class_constants(ce) != SUCCESS)) {`
`@@ -563,6 +564,7 @@ static spl_filesystem_object *spl_filesystem_object_create_type(int num_args, sp`
`563`	`564`	`}`
`564`	`565`	`}`
`565`	`566`	`break;`
	`567`	`+ }`
`566`	`568`	`case SPL_FS_DIR:`
`567`	`569`	`zend_restore_error_handling(&error_handling);`
`568`	`570`	`zend_throw_exception_ex(spl_ce_RuntimeException, 0, "Operation not supported");`