jani Mon May 18 12:51:05 2009 UTC Modified files: (Branch: PHP_5_3) /php-src/ext/curl interface.c Log: MFH http://cvs.php.net/viewvc.cgi/php-src/ext/curl/interface.c?r1=1.62.2.14.2.27.2.45&r2=1.62.2.14.2.27.2.46&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.45 php-src/ext/curl/interface.c:1.62.2.14.2.27.2.46 --- php-src/ext/curl/interface.c:1.62.2.14.2.27.2.45 Mon May 18 12:01:44 2009 +++ php-src/ext/curl/interface.c Mon May 18 12:51:05 2009 @@ -16,7 +16,7 @@ +----------------------------------------------------------------------+ */ -/* $Id: interface.c,v 1.62.2.14.2.27.2.45 2009/05/18 12:01:44 pajoye Exp $ */ +/* $Id: interface.c,v 1.62.2.14.2.27.2.46 2009/05/18 12:51:05 jani Exp $ */ #define ZEND_INCLUDE_FULL_WINDOWS_HEADERS @@ -158,13 +158,14 @@ #define CAAZ(s, v) add_assoc_zval_ex(return_value, s, sizeof(s), (zval *) v); #if defined(PHP_WIN32) || defined(__GNUC__) - #define php_curl_ret(__ret) RETVAL_FALSE; return __ret; +# define php_curl_ret(__ret) RETVAL_FALSE; return __ret; #else - #define php_curl_ret(__ret) RETVAL_FALSE; return; +# define php_curl_ret(__ret) RETVAL_FALSE; return; #endif -static int php_curl_option_url(php_curl *ch, const char *url, const int len) { - CURLcode error=CURLE_OK; +static int php_curl_option_url(php_curl *ch, const char *url, const int len) /* {{{ */ +{ + CURLcode error = CURLE_OK; #if LIBCURL_VERSION_NUM < 0x071100 char *copystr = NULL; #endif @@ -201,6 +202,7 @@ return (error == CURLE_OK ? 1 : 0); } +/* }}} */ /* {{{ arginfo */ ZEND_BEGIN_ARG_INFO_EX(arginfo_curl_version, 0, 0, 0) @@ -282,7 +284,6 @@ ZEND_BEGIN_ARG_INFO(arginfo_curl_multi_close, 0) ZEND_ARG_INFO(0, mh) ZEND_END_ARG_INFO() - /* }}} */ /* {{{ curl_functions[] @@ -347,7 +348,7 @@ php_info_print_table_row(2, "Age", str); /* To update on each new cURL release using src/main.c in cURL sources */ - if(d->features) { + if (d->features) { struct feat { const char *name; int bitmask; @@ -457,7 +458,6 @@ le_curl = zend_register_list_destructors_ex(_php_curl_close, NULL, "curl", module_number); le_curl_multi_handle = zend_register_list_destructors_ex(_php_curl_multi_close, NULL, "curl", module_number); - /* See http://curl.haxx.se/lxr/source/docs/libcurl/symbols-in-versions or curl src/docs/libcurl/symbols-in-versions for a (almost) complete list of options and which version they were introduced */ @@ -938,11 +938,7 @@ /* {{{ curl_progress */ -static size_t curl_progress(void *clientp, - double dltotal, - double dlnow, - double ultotal, - double ulnow) +static size_t curl_progress(void *clientp, double dltotal, double dlnow, double ultotal, double ulnow) { php_curl *ch = (php_curl *) clientp; php_curl_progress *t = ch->handlers->progress; @@ -1001,8 +997,9 @@ if (Z_TYPE_P(retval_ptr) != IS_LONG) { convert_to_long_ex(&retval_ptr); } - if(0 != Z_LVAL_P(retval_ptr)) + if (0 != Z_LVAL_P(retval_ptr)) { rval = 1; + } zval_ptr_dtor(&retval_ptr); } zval_ptr_dtor(argv[0]); @@ -1016,7 +1013,6 @@ } /* }}} */ - /* {{{ curl_read */ static size_t curl_read(char *data, size_t size, size_t nmemb, void *ctx) @@ -1329,10 +1325,10 @@ Initialize a cURL session */ PHP_FUNCTION(curl_init) { - php_curl *ch; - CURL *cp; - char *url = NULL; - int url_len = 0; + php_curl *ch; + CURL *cp; + char *url = NULL; + int url_len = 0; if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|s", &url, &url_len) == FAILURE) { return; @@ -1388,14 +1384,14 @@ Copy a cURL handle along with all of it's preferences */ PHP_FUNCTION(curl_copy_handle) { - zval *zid; - CURL *cp; - php_curl *ch; - php_curl *dupch; + CURL *cp; + zval *zid; + php_curl *ch, *dupch; 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); cp = curl_easy_duphandle(ch->cp); @@ -1930,16 +1926,16 @@ PHP_FUNCTION(curl_setopt) { zval *zid, **zvalue; - long zoption; - php_curl *ch; + long options; + php_curl *ch; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "rlZ", &zid, &zoption, &zvalue) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "rlZ", &zid, &options, &zvalue) == FAILURE) { return; } ZEND_FETCH_RESOURCE(ch, php_curl *, &zid, -1, le_curl_name, le_curl); - if (!_php_curl_setopt(ch, zoption, zvalue, return_value TSRMLS_CC)) { + if (!_php_curl_setopt(ch, options, zvalue, return_value TSRMLS_CC)) { RETURN_TRUE; } else { RETURN_FALSE; @@ -1953,24 +1949,24 @@ { zval *zid, *arr, **entry; php_curl *ch; - long option; + ulong option; HashPosition pos; - char *string_key; - uint str_key_len; + char *string_key; + uint str_key_len; if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "za", &zid, &arr) == FAILURE) { - RETURN_FALSE; + return; } ZEND_FETCH_RESOURCE(ch, php_curl *, &zid, -1, le_curl_name, le_curl); zend_hash_internal_pointer_reset_ex(Z_ARRVAL_P(arr), &pos); while (zend_hash_get_current_data_ex(Z_ARRVAL_P(arr), (void **)&entry, &pos) == SUCCESS) { - if (zend_hash_get_current_key_ex(Z_ARRVAL_P(arr), &string_key, &str_key_len, &option, 0, &pos) == HASH_KEY_IS_STRING) { + if (zend_hash_get_current_key_ex(Z_ARRVAL_P(arr), &string_key, &str_key_len, &option, 0, &pos) != HASH_KEY_IS_LONG) { php_error_docref(NULL TSRMLS_CC, E_WARNING, "Array keys must be CURLOPT constants or equivalent integer values"); RETURN_FALSE; } - if (_php_curl_setopt(ch, option, entry, return_value TSRMLS_CC)) { + if (_php_curl_setopt(ch, (long) option, entry, return_value TSRMLS_CC)) { RETURN_FALSE; } zend_hash_move_forward_ex(Z_ARRVAL_P(arr), &pos); @@ -2000,9 +1996,9 @@ Perform a cURL session */ PHP_FUNCTION(curl_exec) { - zval *zid; - php_curl *ch; - CURLcode error; + CURLcode error; + zval *zid; + php_curl *ch; if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "r", &zid) == FAILURE) { return; @@ -2042,9 +2038,9 @@ Get information regarding a specific transfer */ PHP_FUNCTION(curl_getinfo) { - zval *zid; - php_curl *ch; - long option = 0; + zval *zid; + php_curl *ch; + long option = 0; if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "r|l", &zid, &option) == FAILURE) { return; @@ -2194,8 +2190,8 @@ Return a string contain the last error for the current session */ PHP_FUNCTION(curl_error) { - zval *zid; - php_curl *ch; + zval *zid; + php_curl *ch; if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "r", &zid) == FAILURE) { return; @@ -2212,8 +2208,8 @@ Return an integer containing the last error number */ PHP_FUNCTION(curl_errno) { - zval *zid; - php_curl *ch; + zval *zid; + php_curl *ch; if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "r", &zid) == FAILURE) { return; @@ -2229,8 +2225,8 @@ Close a cURL session */ PHP_FUNCTION(curl_close) { - zval *zid; - php_curl *ch; + zval *zid; + php_curl *ch; if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "r", &zid) == FAILURE) { return;
-- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php