indeyets Sat Jun 21 21:51:05 2008 UTC Modified files: (Branch: PHP_5_3) /php-src/ext/curl interface.c Log: updated to the new parameter-parsing api http://cvs.php.net/viewvc.cgi/php-src/ext/curl/interface.c?r1=1.62.2.14.2.27.2.9&r2=1.62.2.14.2.27.2.10&diff_format=u Index: php-src/ext/curl/interface.c diff -u php-src/ext/curl/interface.c:1.62.2.14.2.27.2.9 php-src/ext/curl/interface.c:1.62.2.14.2.27.2.10 --- php-src/ext/curl/interface.c:1.62.2.14.2.27.2.9 Thu Feb 7 02:08:45 2008 +++ php-src/ext/curl/interface.c Sat Jun 21 21:51:05 2008 @@ -16,7 +16,7 @@ +----------------------------------------------------------------------+ */ -/* $Id: interface.c,v 1.62.2.14.2.27.2.9 2008/02/07 02:08:45 iliaa Exp $ */ +/* $Id: interface.c,v 1.62.2.14.2.27.2.10 2008/06/21 21:51:05 indeyets Exp $ */ #define ZEND_INCLUDE_FULL_WINDOWS_HEADERS @@ -1149,15 +1149,15 @@ Copy a cURL handle along with all of it's preferences */ PHP_FUNCTION(curl_copy_handle) { - zval **zid; + zval *zid; CURL *cp; php_curl *ch; php_curl *dupch; - if (ZEND_NUM_ARGS() != 1 || zend_get_parameters_ex(1, &zid) == FAILURE) { - WRONG_PARAM_COUNT; + if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "r", &zid) == FAILURE) { + return; } - ZEND_FETCH_RESOURCE(ch, php_curl *, zid, -1, le_curl_name, le_curl); + ZEND_FETCH_RESOURCE(ch, php_curl *, &zid, -1, le_curl_name, le_curl); cp = curl_easy_duphandle(ch->cp); if (!cp) { @@ -1609,18 +1609,17 @@ Set an option for a cURL transfer */ PHP_FUNCTION(curl_setopt) { - zval **zid, **zoption, **zvalue; + zval *zid, *zvalue; + long zoption; php_curl *ch; - if (ZEND_NUM_ARGS() != 3 || zend_get_parameters_ex(3, &zid, &zoption, &zvalue) == FAILURE) { - WRONG_PARAM_COUNT; + if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "rlz", &zid, &zoption, &zvalue) == FAILURE) { + return; } - ZEND_FETCH_RESOURCE(ch, php_curl *, zid, -1, le_curl_name, le_curl); - - convert_to_long_ex(zoption); + ZEND_FETCH_RESOURCE(ch, php_curl *, &zid, -1, le_curl_name, le_curl); - if (!_php_curl_setopt(ch, Z_LVAL_PP(zoption), zvalue, return_value TSRMLS_CC)) { + if (!_php_curl_setopt(ch, zoption, &zvalue, return_value TSRMLS_CC)) { RETURN_TRUE; } else { RETURN_FALSE; @@ -1637,7 +1636,7 @@ long option; HashPosition pos; char *string_key; - int str_key_len; + uint str_key_len; if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "za", &zid, &arr) == FAILURE) { RETURN_FALSE; @@ -1681,15 +1680,15 @@ Perform a cURL session */ PHP_FUNCTION(curl_exec) { - zval **zid; + zval *zid; php_curl *ch; CURLcode error; - if (ZEND_NUM_ARGS() != 1 || zend_get_parameters_ex(1, &zid) == FAILURE) { - WRONG_PARAM_COUNT; + if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "r", &zid) == FAILURE) { + return; } - ZEND_FETCH_RESOURCE(ch, php_curl *, zid, -1, le_curl_name, le_curl); + ZEND_FETCH_RESOURCE(ch, php_curl *, &zid, -1, le_curl_name, le_curl); _php_curl_cleanup_handle(ch); @@ -1723,16 +1722,16 @@ Get information regarding a specific transfer */ PHP_FUNCTION(curl_getinfo) { - zval **zid, - **zoption; + zval *zid; php_curl *ch; - int option, argc = ZEND_NUM_ARGS(); + int argc = ZEND_NUM_ARGS(); + long option; - if (argc < 1 || argc > 2 || zend_get_parameters_ex(argc, &zid, &zoption) == FAILURE) { - WRONG_PARAM_COUNT; + if (zend_parse_parameters(argc TSRMLS_CC, "r|l", &zid, &option) == FAILURE) { + return; } - ZEND_FETCH_RESOURCE(ch, php_curl *, zid, -1, le_curl_name, le_curl); + ZEND_FETCH_RESOURCE(ch, php_curl *, &zid, -1, le_curl_name, le_curl); if (argc < 2) { char *s_code; @@ -1807,7 +1806,6 @@ CAAS("request_header", ch->header.str); } } else { - option = Z_LVAL_PP(zoption); switch (option) { case CURLINFO_PRIVATE: case CURLINFO_EFFECTIVE_URL: @@ -1872,14 +1870,14 @@ Return a string contain the last error for the current session */ PHP_FUNCTION(curl_error) { - zval **zid; + zval *zid; php_curl *ch; - if (ZEND_NUM_ARGS() != 1 || zend_get_parameters_ex(1, &zid) == FAILURE) { - WRONG_PARAM_COUNT; + if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "r", &zid) == FAILURE) { + return; } - ZEND_FETCH_RESOURCE(ch, php_curl *, zid, -1, le_curl_name, le_curl); + ZEND_FETCH_RESOURCE(ch, php_curl *, &zid, -1, le_curl_name, le_curl); ch->err.str[CURL_ERROR_SIZE] = 0; RETURN_STRING(ch->err.str, 1); @@ -1890,14 +1888,14 @@ Return an integer containing the last error number */ PHP_FUNCTION(curl_errno) { - zval **zid; + zval *zid; php_curl *ch; - if (ZEND_NUM_ARGS() != 1 || zend_get_parameters_ex(1, &zid) == FAILURE) { - WRONG_PARAM_COUNT; + if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "r", &zid) == FAILURE) { + return; } - ZEND_FETCH_RESOURCE(ch, php_curl *, zid, -1, le_curl_name, le_curl); + ZEND_FETCH_RESOURCE(ch, php_curl *, &zid, -1, le_curl_name, le_curl); RETURN_LONG(ch->err.no); } @@ -1907,14 +1905,14 @@ Close a cURL session */ PHP_FUNCTION(curl_close) { - zval **zid; + zval *zid; php_curl *ch; - if (ZEND_NUM_ARGS() != 1 || zend_get_parameters_ex(1, &zid) == FAILURE) { - WRONG_PARAM_COUNT; + if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "r", &zid) == FAILURE) { + return; } - ZEND_FETCH_RESOURCE(ch, php_curl *, zid, -1, le_curl_name, le_curl); + ZEND_FETCH_RESOURCE(ch, php_curl *, &zid, -1, le_curl_name, le_curl); if (ch->in_callback) { php_error_docref(NULL TSRMLS_CC, E_WARNING, "Attempt to close cURL handle from a callback"); @@ -1924,7 +1922,7 @@ if (ch->uses) { ch->uses--; } else { - zend_list_delete(Z_LVAL_PP(zid)); + zend_list_delete(Z_LVAL_P(zid)); } } /* }}} */
-- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php