fixup! better error reporting from IntlMessage constructor

Author: pilif

ext/intl/msgformat/msgformat.c

@@ -88,17 +88,12 @@ static int msgfmt_ctor(INTERNAL_FUNCTION_PARAMETERS)
 		parse_error_str = intl_parse_error_to_string( &parse_error );
 		spprintf( &msg, 0, "pattern syntax error (%s)", parse_error_str.s? ZSTR_VAL(parse_error_str.s) : "" );
 		smart_str_free( &parse_error_str );
+
 		intl_error_set_code( NULL, INTL_DATA_ERROR_CODE( mfo ) );
+		intl_errors_set_custom_msg( INTL_DATA_ERROR_P( mfo ), msg, 1 );
 
-		if( msg != NULL )
-		{
-			intl_errors_set_custom_msg( INTL_DATA_ERROR_P( mfo ), msg, 1 );
-			efree( msg );
-			return FAILURE;
-		} else {
-			intl_errors_set_custom_msg( INTL_DATA_ERROR_P( mfo ), "Unknown parser error", 0 );
-			return FAILURE;
-		}
+		efree( msg );
+		return FAILURE;
 	}
 
 	INTL_CTOR_CHECK_STATUS(mfo, "msgfmt_create: message formatter creation failed");