summaryrefslogtreecommitdiff
path: root/ext/intl/msgformat
diff options
context:
space:
mode:
authorGustavo André dos Santos Lopes <cataphract@php.net>2012-05-06 00:48:17 +0200
committerGustavo André dos Santos Lopes <cataphract@php.net>2012-05-13 20:53:05 +0100
commit407455876e486df45a6b6c91b483060b0479a0b3 (patch)
tree3d18ce222152f7d059a2db292e4f928b5ab8d8fc /ext/intl/msgformat
parentf1621485ad7c9b02b4df42c21a472223fe106f51 (diff)
downloadphp-git-407455876e486df45a6b6c91b483060b0479a0b3.tar.gz
Set global error if insufficient numeric args. WS.
Diffstat (limited to 'ext/intl/msgformat')
-rwxr-xr-xext/intl/msgformat/msgformat_format.c6
-rwxr-xr-xext/intl/msgformat/msgformat_helpers.cpp16
2 files changed, 11 insertions, 11 deletions
diff --git a/ext/intl/msgformat/msgformat_format.c b/ext/intl/msgformat/msgformat_format.c
index d88bc00181..099dbcb4b6 100755
--- a/ext/intl/msgformat/msgformat_format.c
+++ b/ext/intl/msgformat/msgformat_format.c
@@ -44,10 +44,10 @@ static void msgfmt_do_format(MessageFormatter_object *mfo, zval *args, zval *ret
/* umsg_format_arg_count() always returns 0 for named argument patterns,
* so this check is ignored and un-substituted {name} strings
* in a pattern are returned unmodified. */
- if(count < umsg_format_arg_count(MSG_FORMAT_OBJECT(mfo))) {
+ if (count < umsg_format_arg_count(MSG_FORMAT_OBJECT(mfo))) {
/* Not enough aguments for format! */
- intl_error_set( INTL_DATA_ERROR_P(mfo), U_ILLEGAL_ARGUMENT_ERROR,
- "msgfmt_format: not enough parameters", 0 TSRMLS_CC );
+ intl_errors_set(INTL_DATA_ERROR_P(mfo), U_ILLEGAL_ARGUMENT_ERROR,
+ "msgfmt_format: not enough parameters", 0 TSRMLS_CC);
RETVAL_FALSE;
return;
}
diff --git a/ext/intl/msgformat/msgformat_helpers.cpp b/ext/intl/msgformat/msgformat_helpers.cpp
index 5c114e8302..503dca9ad9 100755
--- a/ext/intl/msgformat/msgformat_helpers.cpp
+++ b/ext/intl/msgformat/msgformat_helpers.cpp
@@ -117,14 +117,14 @@ static HashTable *umsg_parse_format(const MessagePattern& mp, UErrorCode& uec)
parts_count = mp.countParts();
// See MessageFormat::cacheExplicitFormats()
- /*
- * Looking through the pattern, go to each arg_start part type.
- * The arg-typeof that tells us the argument type (simple, complicated)
- * then the next part is either the arg_name or arg number
- * and then if it's simple after that there could be a part-type=arg-type
+ /*
+ * Looking through the pattern, go to each arg_start part type.
+ * The arg-typeof that tells us the argument type (simple, complicated)
+ * then the next part is either the arg_name or arg number
+ * and then if it's simple after that there could be a part-type=arg-type
* while substring will tell us number, spellout, etc.
- * If the next thing isn't an arg-type then assume string.
- */
+ * If the next thing isn't an arg-type then assume string.
+ */
/* The last two "parts" can at most be ARG_LIMIT and MSG_LIMIT
* which we need not examine. */
for (int32_t i = 0; i < parts_count - 2 && U_SUCCESS(uec); i++) {
@@ -226,7 +226,7 @@ static HashTable *umsg_parse_format(const MessagePattern& mp, UErrorCode& uec)
}
*storedType = type;
- } /* visiting each part */
+ } /* visiting each part */
if (U_FAILURE(uec)) {
zend_hash_destroy(ret);