]> granicus.if.org Git - php/commitdiff
Remove double "wrong param count" warnings
authorHannes Magnusson <bjori@php.net>
Sat, 7 Oct 2006 22:55:18 +0000 (22:55 +0000)
committerHannes Magnusson <bjori@php.net>
Sat, 7 Oct 2006 22:55:18 +0000 (22:55 +0000)
ext/dba/dba.c
ext/mhash/mhash.c
ext/session/session.c
ext/shmop/shmop.c
ext/standard/basic_functions.c
ext/xmlwriter/php_xmlwriter.c
ext/zip/php_zip.c

index 24276dfedf0620795f1ff382a0c96ba067375c8d..d4bab3141d95203d3e6e28a64c8ce8745decf3d5 100644 (file)
@@ -1160,7 +1160,6 @@ PHP_FUNCTION(dba_handlers)
        zend_bool full_info = 0;
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|b", &full_info) == FAILURE) {
-               ZEND_WRONG_PARAM_COUNT();
                RETURN_FALSE;
        }
 
index e175f8b79afaf9cc5340d07e798c5442785f91f7..23302fdc9a75334e3a54b0ada67180377e6280ee 100644 (file)
@@ -111,7 +111,7 @@ PHP_FUNCTION(mhash_get_block_size)
        long hash;
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "l", &hash) == FAILURE) {
-               WRONG_PARAM_COUNT;
+               return;
        }
 
        RETURN_LONG(mhash_get_block_size(hash));
@@ -127,7 +127,7 @@ PHP_FUNCTION(mhash_get_hash_name)
        long hash;
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "l", &hash) == FAILURE) {
-               WRONG_PARAM_COUNT;
+               return;
        }
 
        name = mhash_get_hash_name(hash);
@@ -153,7 +153,7 @@ PHP_FUNCTION(mhash)
        char *data, *key=NULL;
        
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ls|s", &hash, &data, &data_len, &key, &key_len) == FAILURE) {
-               WRONG_PARAM_COUNT;
+               return;
        }
        
        bsize = mhash_get_block_size(hash);
@@ -202,7 +202,7 @@ PHP_FUNCTION(mhash_keygen_s2k)
        int password_len, salt_len;
        
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "lssl", &hash, &password, &password_len, &in_salt, &salt_len, &bytes) == FAILURE) {
-               WRONG_PARAM_COUNT;
+               return;
        }
        if (bytes <= 0){
                php_error_docref(NULL TSRMLS_CC, E_WARNING, "the byte parameter must be greater than 0");
index 2d955f746f9513d929566af8564be21e02c6c138..11324b5b67033c90f8748548b6499f5a4b802ae2 100644 (file)
@@ -1416,7 +1416,7 @@ PHP_FUNCTION(session_regenerate_id)
        zend_bool del_ses = 0;
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|b", &del_ses) == FAILURE) {
-               WRONG_PARAM_COUNT;
+               return;
        }
 
        if (SG(headers_sent)) {
index a4578c02ea6d4ed44c58fb45ee733de2ef870fc4..dea3bfe7e03804615502f04749be709b1e592499 100644 (file)
@@ -122,7 +122,7 @@ PHP_FUNCTION(shmop_open)
        zend_uchar flag_type;
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ltll", &key, &flags, &flags_len, &flag_type, &mode, &size) == FAILURE) {
-               WRONG_PARAM_COUNT;
+               return;
        }
 
        if (flag_type == IS_UNICODE) {
@@ -216,7 +216,7 @@ PHP_FUNCTION(shmop_read)
        char *return_string;
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "lll", &shmid, &start, &count) == FAILURE) {
-               WRONG_PARAM_COUNT;
+               return;
        }
 
        shmop = zend_list_find(shmid, &type);
@@ -256,7 +256,7 @@ PHP_FUNCTION(shmop_close)
        int type;
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "l", &shmid) == FAILURE) {
-               WRONG_PARAM_COUNT;
+               return;
        }
 
        shmop = zend_list_find(shmid, &type);
@@ -279,7 +279,7 @@ PHP_FUNCTION(shmop_size)
        int type;
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "l", &shmid) == FAILURE) {
-               WRONG_PARAM_COUNT;
+               return;
        }
 
        shmop = zend_list_find(shmid, &type);
@@ -305,7 +305,7 @@ PHP_FUNCTION(shmop_write)
        int data_len;
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "lS|l", &shmid, &data, &data_len, &offset) == FAILURE) {
-               WRONG_PARAM_COUNT;
+               return;
        }
 
        shmop = zend_list_find(shmid, &type);
@@ -341,7 +341,7 @@ PHP_FUNCTION(shmop_delete)
        int type;
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "l", &shmid) == FAILURE) {
-               WRONG_PARAM_COUNT;
+               return;
        }
 
        shmop = zend_list_find(shmid, &type);
index 1a9faf97b75b2f17ba37b47c7925f2da09ba92be..579f6cef065922a88b18eb64028b088dafcbf92a 100644 (file)
@@ -4702,8 +4702,8 @@ PHP_FUNCTION(time_nanosleep)
        long tv_sec, tv_nsec;
        struct timespec php_req, php_rem;
        
-       if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ll", &tv_sec, &tv_nsec)) {
-               WRONG_PARAM_COUNT;
+       if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ll", &tv_sec, &tv_nsec) == FAILURE) {
+               return;
        }
 
        php_req.tv_sec = (time_t) tv_sec;
@@ -4731,8 +4731,8 @@ PHP_FUNCTION(time_sleep_until)
        struct timeval tm;
        struct timespec php_req, php_rem;
        
-       if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "d", &d_ts)) {
-               WRONG_PARAM_COUNT;
+       if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "d", &d_ts) == FAILURE) {
+               return;
        }
 
        if (gettimeofday((struct timeval *) &tm, NULL) != 0) {
index e45c0a8a39280904508d437acb3b58cb4210f0c9..ed5b18fedee976f02b989a1f325c3c5a744975b8 100644 (file)
@@ -1365,7 +1365,6 @@ PHP_FUNCTION(xmlwriter_open_uri)
 #endif
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "t", &source, &source_len, &source_type) == FAILURE) {
-               WRONG_PARAM_COUNT;
                return;
        }
        
index f4eff316349d3b2a2608d324e5b82af19c0b472f..da939a019d32369c952f08409675e0237a76b0cc 100644 (file)
@@ -866,7 +866,6 @@ ZIPARCHIVE_METHOD(open)
        ze_zip_object *ze_obj = NULL;
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s|l", &filename, &filename_len, &flags) == FAILURE) {
-               WRONG_PARAM_COUNT;
                return;
        }
 
@@ -955,7 +954,6 @@ ZIPARCHIVE_METHOD(addFile)
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s|sll",
                        &filename, &filename_len, &entry_name, &entry_name_len, &offset_start, &offset_len) == FAILURE) {
-               WRONG_PARAM_COUNT;
                return;
        }
 
@@ -1022,7 +1020,6 @@ ZIPARCHIVE_METHOD(addFromString)
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ss",
                        &name, &name_len, &buffer, &buffer_len) == FAILURE) {
-               WRONG_PARAM_COUNT;
                return;
        }
 
@@ -1086,7 +1083,6 @@ ZIPARCHIVE_METHOD(statName)
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s|l",
                        &name, &name_len, &flags) == FAILURE) {
-               WRONG_PARAM_COUNT;
                return;
        }
 
@@ -1114,7 +1110,6 @@ ZIPARCHIVE_METHOD(statIndex)
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "l|l",
                        &index, &flags) == FAILURE) {
-               WRONG_PARAM_COUNT;
                return;
        }
 
@@ -1144,7 +1139,6 @@ ZIPARCHIVE_METHOD(locateName)
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s|l",
                        &name, &name_len, &flags) == FAILURE) {
-               WRONG_PARAM_COUNT;
                return;
        }
        if (name_len<1) {
@@ -1183,7 +1177,6 @@ ZIPARCHIVE_METHOD(getNameIndex)
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "l|l",
                        &index, &flags) == FAILURE) {
-               WRONG_PARAM_COUNT;
                return;
        }
 
@@ -1213,7 +1206,6 @@ ZIPARCHIVE_METHOD(setArchiveComment)
        ZIP_FROM_OBJECT(intern, this);
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s", &comment, &comment_len) == FAILURE) {
-               WRONG_PARAM_COUNT;
                return;
        }
        if (zip_set_archive_comment(intern, (const char *)comment, (int)comment_len)) {
@@ -1241,7 +1233,6 @@ ZIPARCHIVE_METHOD(getArchiveComment)
        ZIP_FROM_OBJECT(intern, this);
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|l", &flags) == FAILURE) {
-               WRONG_PARAM_COUNT;
                return;
        }
 
@@ -1268,7 +1259,6 @@ ZIPARCHIVE_METHOD(setCommentName)
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ss",
                        &name, &name_len, &comment, &comment_len) == FAILURE) {
-               WRONG_PARAM_COUNT;
                return;
        }
 
@@ -1297,7 +1287,6 @@ ZIPARCHIVE_METHOD(setCommentIndex)
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ls",
                        &index, &comment, &comment_len) == FAILURE) {
-               WRONG_PARAM_COUNT;
                return;
        }
 
@@ -1327,7 +1316,6 @@ ZIPARCHIVE_METHOD(getCommentName)
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s|l",
                        &name, &name_len, &flags) == FAILURE) {
-               WRONG_PARAM_COUNT;
                return;
        }
 
@@ -1356,7 +1344,6 @@ ZIPARCHIVE_METHOD(getCommentIndex)
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "l|l",
                                &index, &flags) == FAILURE) {
-               WRONG_PARAM_COUNT;
                return;
        }
 
@@ -1381,7 +1368,6 @@ ZIPARCHIVE_METHOD(deleteIndex)
        ZIP_FROM_OBJECT(intern, this);
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "l", &index) == FAILURE) {
-               WRONG_PARAM_COUNT;
                return;
        }
 
@@ -1414,7 +1400,6 @@ ZIPARCHIVE_METHOD(deleteName)
        ZIP_FROM_OBJECT(intern, this);
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s", &name, &name_len) == FAILURE) {
-               WRONG_PARAM_COUNT;
                return;
        }
        if (name_len < 1) {
@@ -1447,7 +1432,6 @@ ZIPARCHIVE_METHOD(renameIndex)
        ZIP_FROM_OBJECT(intern, this);
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ls", &index, &new_name, &new_name_len) == FAILURE) {
-               WRONG_PARAM_COUNT;
                return;
        }
 
@@ -1483,7 +1467,6 @@ ZIPARCHIVE_METHOD(renameName)
        ZIP_FROM_OBJECT(intern, this);
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ss", &name, &name_len, &new_name, &new_name_len) == FAILURE) {
-               WRONG_PARAM_COUNT;
                return;
        }
 
@@ -1516,7 +1499,6 @@ ZIPARCHIVE_METHOD(unchangeIndex)
        ZIP_FROM_OBJECT(intern, this);
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "l", &index) == FAILURE) {
-               WRONG_PARAM_COUNT;
                return;
        }
 
@@ -1549,7 +1531,6 @@ ZIPARCHIVE_METHOD(unchangeName)
        ZIP_FROM_OBJECT(intern, this);
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s", &name, &name_len) == FAILURE) {
-               WRONG_PARAM_COUNT;
                return;
        }
 
@@ -1636,7 +1617,6 @@ ZIPARCHIVE_METHOD(extractTo)
        }
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s|z", &pathto, &pathto_len, &zval_files) == FAILURE) {
-               WRONG_PARAM_COUNT;
                return;
        }
 
@@ -1732,13 +1712,11 @@ static void php_zip_get_from(INTERNAL_FUNCTION_PARAMETERS, int type) /* {{{ */
 
        if (type == 1) {
                if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s|ll", &filename, &filename_len, &len, &flags) == FAILURE) {
-                       WRONG_PARAM_COUNT;
                        return;
                }
                PHP_ZIP_STAT_PATH(intern, filename, filename_len, flags, sb);
        } else {
                if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "l|ll", &index, &len, &flags) == FAILURE) {
-                       WRONG_PARAM_COUNT;
                        return;
                }
                PHP_ZIP_STAT_INDEX(intern, index, 0, sb);
@@ -1809,7 +1787,6 @@ ZIPARCHIVE_METHOD(getStream)
        ZIP_FROM_OBJECT(intern, this);
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s", &filename, &filename_len) == FAILURE) {
-               WRONG_PARAM_COUNT;
                return;
        }