Adding closing tag to test and removing unreacheable statement

Author: CViniciusSDias

ext/standard/string.c

@@ -1042,19 +1042,14 @@ PHP_FUNCTION(implode)
 		RETURN_THROWS();
 	}
 
-	if (pieces == NULL) {
-		if (arg1_array == NULL) {
-			zend_type_error("%s(): Argument #1 ($array) must be of type array, string given", get_active_function_name());
-			RETURN_THROWS();
-		}
+	if (pieces != NULL && arg1_str == NULL) {
+		zend_argument_type_error(1, "must be of type string, array given");
+		RETURN_THROWS();
+	}
 
+	if (pieces == NULL) {
 		arg1_str = ZSTR_EMPTY_ALLOC();
 		pieces = arg1_array;
-	} else {
-		if (arg1_str == NULL) {
-			zend_argument_type_error(1, "must be of type string, array given");
-			RETURN_THROWS();
-		}
 	}
 
 	php_implode(arg1_str, pieces, return_value);

ext/standard/tests/strings/implode_error.phpt

@@ -22,6 +22,7 @@ try {
 } catch (TypeError $e) {
     echo $e->getMessage(), "\n";
 }
+?>
 --EXPECTF--
 implode(): If argument #1 ($separator) is of type string, argument #2 ($array) must be of type array, null given
 implode(): If argument #1 ($separator) is of type string, argument #2 ($array) must be of type array, null given