diff options
| author | Anatol Belski <ab@php.net> | 2014-12-13 23:06:14 +0100 | 
|---|---|---|
| committer | Anatol Belski <ab@php.net> | 2014-12-13 23:06:14 +0100 | 
| commit | bdeb220f48825642f84cdbf3ff23a30613c92e86 (patch) | |
| tree | 1a6cf34d20420e4815b4becb21311a4457d84103 /ext/pdo_sqlite/sqlite_driver.c | |
| parent | bb66f385d09e7e55390e9f57fcbca08f6b43ff91 (diff) | |
| download | php-git-bdeb220f48825642f84cdbf3ff23a30613c92e86.tar.gz | |
first shot remove TSRMLS_* things
Diffstat (limited to 'ext/pdo_sqlite/sqlite_driver.c')
| -rw-r--r-- | ext/pdo_sqlite/sqlite_driver.c | 102 | 
1 files changed, 48 insertions, 54 deletions
| diff --git a/ext/pdo_sqlite/sqlite_driver.c b/ext/pdo_sqlite/sqlite_driver.c index 413b50b9d4..8e1de906a5 100644 --- a/ext/pdo_sqlite/sqlite_driver.c +++ b/ext/pdo_sqlite/sqlite_driver.c @@ -31,7 +31,7 @@  #include "php_pdo_sqlite_int.h"  #include "zend_exceptions.h" -int _pdo_sqlite_error(pdo_dbh_t *dbh, pdo_stmt_t *stmt, const char *file, int line TSRMLS_DC) /* {{{ */ +int _pdo_sqlite_error(pdo_dbh_t *dbh, pdo_stmt_t *stmt, const char *file, int line) /* {{{ */  {  	pdo_sqlite_db_handle *H = (pdo_sqlite_db_handle *)dbh->driver_data;  	pdo_error_type *pdo_err = stmt ? &stmt->error_code : &dbh->error_code; @@ -78,7 +78,7 @@ int _pdo_sqlite_error(pdo_dbh_t *dbh, pdo_stmt_t *stmt, const char *file, int li  	}  	if (!dbh->methods) { -		zend_throw_exception_ex(php_pdo_get_exception(), einfo->errcode TSRMLS_CC, "SQLSTATE[%s] [%d] %s", +		zend_throw_exception_ex(php_pdo_get_exception(), einfo->errcode, "SQLSTATE[%s] [%d] %s",  				*pdo_err, einfo->errcode, einfo->errmsg);  	} @@ -86,7 +86,7 @@ int _pdo_sqlite_error(pdo_dbh_t *dbh, pdo_stmt_t *stmt, const char *file, int li  }  /* }}} */ -static int pdo_sqlite_fetch_error_func(pdo_dbh_t *dbh, pdo_stmt_t *stmt, zval *info TSRMLS_DC) +static int pdo_sqlite_fetch_error_func(pdo_dbh_t *dbh, pdo_stmt_t *stmt, zval *info)  {  	pdo_sqlite_db_handle *H = (pdo_sqlite_db_handle *)dbh->driver_data;  	pdo_sqlite_error_info *einfo = &H->einfo; @@ -99,7 +99,7 @@ static int pdo_sqlite_fetch_error_func(pdo_dbh_t *dbh, pdo_stmt_t *stmt, zval *i  	return 1;  } -static void pdo_sqlite_cleanup_callbacks(pdo_sqlite_db_handle *H TSRMLS_DC) +static void pdo_sqlite_cleanup_callbacks(pdo_sqlite_db_handle *H)  {  	struct pdo_sqlite_func *func; @@ -152,14 +152,14 @@ static void pdo_sqlite_cleanup_callbacks(pdo_sqlite_db_handle *H TSRMLS_DC)  	}  } -static int sqlite_handle_closer(pdo_dbh_t *dbh TSRMLS_DC) /* {{{ */ +static int sqlite_handle_closer(pdo_dbh_t *dbh) /* {{{ */  {  	pdo_sqlite_db_handle *H = (pdo_sqlite_db_handle *)dbh->driver_data;  	if (H) {  		pdo_sqlite_error_info *einfo = &H->einfo; -		pdo_sqlite_cleanup_callbacks(H TSRMLS_CC); +		pdo_sqlite_cleanup_callbacks(H);  		if (H->db) {  			sqlite3_close(H->db);  			H->db = NULL; @@ -175,7 +175,7 @@ static int sqlite_handle_closer(pdo_dbh_t *dbh TSRMLS_DC) /* {{{ */  }  /* }}} */ -static int sqlite_handle_preparer(pdo_dbh_t *dbh, const char *sql, zend_long sql_len, pdo_stmt_t *stmt, zval *driver_options TSRMLS_DC) +static int sqlite_handle_preparer(pdo_dbh_t *dbh, const char *sql, zend_long sql_len, pdo_stmt_t *stmt, zval *driver_options)  {  	pdo_sqlite_db_handle *H = (pdo_sqlite_db_handle *)dbh->driver_data;  	pdo_sqlite_stmt *S = ecalloc(1, sizeof(pdo_sqlite_stmt)); @@ -187,7 +187,7 @@ static int sqlite_handle_preparer(pdo_dbh_t *dbh, const char *sql, zend_long sql  	stmt->methods = &sqlite_stmt_methods;  	stmt->supports_placeholders = PDO_PLACEHOLDER_POSITIONAL|PDO_PLACEHOLDER_NAMED; -	if (PDO_CURSOR_FWDONLY != pdo_attr_lval(driver_options, PDO_ATTR_CURSOR, PDO_CURSOR_FWDONLY TSRMLS_CC)) { +	if (PDO_CURSOR_FWDONLY != pdo_attr_lval(driver_options, PDO_ATTR_CURSOR, PDO_CURSOR_FWDONLY)) {  		H->einfo.errcode = SQLITE_ERROR;  		pdo_sqlite_error(dbh);  		return 0; @@ -203,7 +203,7 @@ static int sqlite_handle_preparer(pdo_dbh_t *dbh, const char *sql, zend_long sql  	return 0;  } -static zend_long sqlite_handle_doer(pdo_dbh_t *dbh, const char *sql, zend_long sql_len TSRMLS_DC) +static zend_long sqlite_handle_doer(pdo_dbh_t *dbh, const char *sql, zend_long sql_len)  {  	pdo_sqlite_db_handle *H = (pdo_sqlite_db_handle *)dbh->driver_data;  	char *errmsg = NULL; @@ -219,18 +219,18 @@ static zend_long sqlite_handle_doer(pdo_dbh_t *dbh, const char *sql, zend_long s  	}  } -static char *pdo_sqlite_last_insert_id(pdo_dbh_t *dbh, const char *name, unsigned int *len TSRMLS_DC) +static char *pdo_sqlite_last_insert_id(pdo_dbh_t *dbh, const char *name, unsigned int *len)  {  	pdo_sqlite_db_handle *H = (pdo_sqlite_db_handle *)dbh->driver_data;  	char *id; -	id = php_pdo_int64_to_str(sqlite3_last_insert_rowid(H->db) TSRMLS_CC); +	id = php_pdo_int64_to_str(sqlite3_last_insert_rowid(H->db));  	*len = strlen(id);  	return id;  }  /* NB: doesn't handle binary strings... use prepared stmts for that */ -static int sqlite_handle_quoter(pdo_dbh_t *dbh, const char *unquoted, int unquotedlen, char **quoted, int *quotedlen, enum pdo_param_type paramtype  TSRMLS_DC) +static int sqlite_handle_quoter(pdo_dbh_t *dbh, const char *unquoted, int unquotedlen, char **quoted, int *quotedlen, enum pdo_param_type paramtype )  {  	*quoted = safe_emalloc(2, unquotedlen, 3);  	sqlite3_snprintf(2*unquotedlen + 3, *quoted, "'%q'", unquoted); @@ -238,7 +238,7 @@ static int sqlite_handle_quoter(pdo_dbh_t *dbh, const char *unquoted, int unquot  	return 1;  } -static int sqlite_handle_begin(pdo_dbh_t *dbh TSRMLS_DC) +static int sqlite_handle_begin(pdo_dbh_t *dbh)  {  	pdo_sqlite_db_handle *H = (pdo_sqlite_db_handle *)dbh->driver_data;  	char *errmsg = NULL; @@ -252,7 +252,7 @@ static int sqlite_handle_begin(pdo_dbh_t *dbh TSRMLS_DC)  	return 1;  } -static int sqlite_handle_commit(pdo_dbh_t *dbh TSRMLS_DC) +static int sqlite_handle_commit(pdo_dbh_t *dbh)  {  	pdo_sqlite_db_handle *H = (pdo_sqlite_db_handle *)dbh->driver_data;  	char *errmsg = NULL; @@ -266,7 +266,7 @@ static int sqlite_handle_commit(pdo_dbh_t *dbh TSRMLS_DC)  	return 1;  } -static int sqlite_handle_rollback(pdo_dbh_t *dbh TSRMLS_DC) +static int sqlite_handle_rollback(pdo_dbh_t *dbh)  {  	pdo_sqlite_db_handle *H = (pdo_sqlite_db_handle *)dbh->driver_data;  	char *errmsg = NULL; @@ -280,7 +280,7 @@ static int sqlite_handle_rollback(pdo_dbh_t *dbh TSRMLS_DC)  	return 1;  } -static int pdo_sqlite_get_attribute(pdo_dbh_t *dbh, zend_long attr, zval *return_value TSRMLS_DC) +static int pdo_sqlite_get_attribute(pdo_dbh_t *dbh, zend_long attr, zval *return_value)  {  	switch (attr) {  		case PDO_ATTR_CLIENT_VERSION: @@ -295,7 +295,7 @@ static int pdo_sqlite_get_attribute(pdo_dbh_t *dbh, zend_long attr, zval *return  	return 1;  } -static int pdo_sqlite_set_attr(pdo_dbh_t *dbh, zend_long attr, zval *val TSRMLS_DC) +static int pdo_sqlite_set_attr(pdo_dbh_t *dbh, zend_long attr, zval *val)  {  	pdo_sqlite_db_handle *H = (pdo_sqlite_db_handle *)dbh->driver_data; @@ -310,7 +310,7 @@ static int pdo_sqlite_set_attr(pdo_dbh_t *dbh, zend_long attr, zval *val TSRMLS_  static int do_callback(struct pdo_sqlite_fci *fc, zval *cb,  		int argc, sqlite3_value **argv, sqlite3_context *context, -		int is_agg TSRMLS_DC) +		int is_agg)  {  	zval *zargs = NULL;  	zval retval; @@ -379,8 +379,8 @@ static int do_callback(struct pdo_sqlite_fci *fc, zval *cb,  	fc->fci.params = zargs; -	if ((ret = zend_call_function(&fc->fci, &fc->fcc TSRMLS_CC)) == FAILURE) { -		php_error_docref(NULL TSRMLS_CC, E_WARNING, "An error occurred while invoking the callback"); +	if ((ret = zend_call_function(&fc->fci, &fc->fcc)) == FAILURE) { +		php_error_docref(NULL, E_WARNING, "An error occurred while invoking the callback");  	}  	/* clean up the params */ @@ -448,26 +448,23 @@ static void php_sqlite3_func_callback(sqlite3_context *context, int argc,  	sqlite3_value **argv)  {  	struct pdo_sqlite_func *func = (struct pdo_sqlite_func*)sqlite3_user_data(context); -	TSRMLS_FETCH(); -	do_callback(&func->afunc, &func->func, argc, argv, context, 0 TSRMLS_CC); +	do_callback(&func->afunc, &func->func, argc, argv, context, 0);  }  static void php_sqlite3_func_step_callback(sqlite3_context *context, int argc,  	sqlite3_value **argv)  {  	struct pdo_sqlite_func *func = (struct pdo_sqlite_func*)sqlite3_user_data(context); -	TSRMLS_FETCH(); -	do_callback(&func->astep, &func->step, argc, argv, context, 1 TSRMLS_CC); +	do_callback(&func->astep, &func->step, argc, argv, context, 1);  }  static void php_sqlite3_func_final_callback(sqlite3_context *context)  {  	struct pdo_sqlite_func *func = (struct pdo_sqlite_func*)sqlite3_user_data(context); -	TSRMLS_FETCH(); -	do_callback(&func->afini, &func->fini, 0, NULL, context, 1 TSRMLS_CC); +	do_callback(&func->afini, &func->fini, 0, NULL, context, 1);  }  static int php_sqlite3_collation_callback(void *context, @@ -478,7 +475,6 @@ static int php_sqlite3_collation_callback(void *context,  	zval zargs[2];  	zval retval;  	struct pdo_sqlite_collation *collation = (struct pdo_sqlite_collation*) context; -	TSRMLS_FETCH();  	collation->fc.fci.size = sizeof(collation->fc.fci);  	collation->fc.fci.function_table = EG(function_table); @@ -493,8 +489,8 @@ static int php_sqlite3_collation_callback(void *context,  	collation->fc.fci.param_count = 2;  	collation->fc.fci.params = zargs; -	if ((ret = zend_call_function(&collation->fc.fci, &collation->fc.fcc TSRMLS_CC)) == FAILURE) { -		php_error_docref(NULL TSRMLS_CC, E_WARNING, "An error occurred while invoking the callback"); +	if ((ret = zend_call_function(&collation->fc.fci, &collation->fc.fcc)) == FAILURE) { +		php_error_docref(NULL, E_WARNING, "An error occurred while invoking the callback");  	} else if (!Z_ISUNDEF(retval)) {  		if (Z_TYPE(retval) != IS_LONG) {  			convert_to_long_ex(&retval); @@ -528,7 +524,7 @@ static PHP_METHOD(SQLite, sqliteCreateFunction)  	pdo_sqlite_db_handle *H;  	int ret; -	if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "sz|l", +	if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS(), "sz|l",  			&func_name, &func_name_len, &callback, &argc)) {  		RETURN_FALSE;  	} @@ -536,8 +532,8 @@ static PHP_METHOD(SQLite, sqliteCreateFunction)  	dbh = Z_PDO_DBH_P(getThis());  	PDO_CONSTRUCT_CHECK; -	if (!zend_is_callable(callback, 0, &cbname TSRMLS_CC)) { -		php_error_docref(NULL TSRMLS_CC, E_WARNING, "function '%s' is not callable", cbname->val); +	if (!zend_is_callable(callback, 0, &cbname)) { +		php_error_docref(NULL, E_WARNING, "function '%s' is not callable", cbname->val);  		zend_string_release(cbname);  		RETURN_FALSE;  	} @@ -598,7 +594,7 @@ static PHP_METHOD(SQLite, sqliteCreateAggregate)  	pdo_sqlite_db_handle *H;  	int ret; -	if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "szz|l", +	if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS(), "szz|l",  			&func_name, &func_name_len, &step_callback, &fini_callback, &argc)) {  		RETURN_FALSE;  	} @@ -606,14 +602,14 @@ static PHP_METHOD(SQLite, sqliteCreateAggregate)  	dbh = Z_PDO_DBH_P(getThis());  	PDO_CONSTRUCT_CHECK; -	if (!zend_is_callable(step_callback, 0, &cbname TSRMLS_CC)) { -		php_error_docref(NULL TSRMLS_CC, E_WARNING, "function '%s' is not callable", cbname->val); +	if (!zend_is_callable(step_callback, 0, &cbname)) { +		php_error_docref(NULL, E_WARNING, "function '%s' is not callable", cbname->val);  		zend_string_release(cbname);  		RETURN_FALSE;  	}  	zend_string_release(cbname); -	if (!zend_is_callable(fini_callback, 0, &cbname TSRMLS_CC)) { -		php_error_docref(NULL TSRMLS_CC, E_WARNING, "function '%s' is not callable", cbname->val); +	if (!zend_is_callable(fini_callback, 0, &cbname)) { +		php_error_docref(NULL, E_WARNING, "function '%s' is not callable", cbname->val);  		zend_string_release(cbname);  		RETURN_FALSE;  	} @@ -658,7 +654,7 @@ static PHP_METHOD(SQLite, sqliteCreateCollation)  	pdo_sqlite_db_handle *H;  	int ret; -	if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "sz", +	if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS(), "sz",  		&collation_name, &collation_name_len, &callback)) {  		RETURN_FALSE;  	} @@ -666,8 +662,8 @@ static PHP_METHOD(SQLite, sqliteCreateCollation)  	dbh = Z_PDO_DBH_P(getThis());  	PDO_CONSTRUCT_CHECK; -	if (!zend_is_callable(callback, 0, &cbname TSRMLS_CC)) { -		php_error_docref(NULL TSRMLS_CC, E_WARNING, "function '%s' is not callable", cbname->val); +	if (!zend_is_callable(callback, 0, &cbname)) { +		php_error_docref(NULL, E_WARNING, "function '%s' is not callable", cbname->val);  		zend_string_release(cbname);  		RETURN_FALSE;  	} @@ -701,7 +697,7 @@ static const zend_function_entry dbh_methods[] = {  	PHP_FE_END  }; -static const zend_function_entry *get_driver_methods(pdo_dbh_t *dbh, int kind TSRMLS_DC) +static const zend_function_entry *get_driver_methods(pdo_dbh_t *dbh, int kind)  {  	switch (kind) {  		case PDO_DBH_DRIVER_METHOD_KIND_DBH: @@ -712,13 +708,13 @@ static const zend_function_entry *get_driver_methods(pdo_dbh_t *dbh, int kind TS  	}  } -static void pdo_sqlite_request_shutdown(pdo_dbh_t *dbh TSRMLS_DC) +static void pdo_sqlite_request_shutdown(pdo_dbh_t *dbh)  {  	pdo_sqlite_db_handle *H = (pdo_sqlite_db_handle *)dbh->driver_data;  	/* unregister functions, so that they don't linger for the next  	 * request */  	if (H) { -		pdo_sqlite_cleanup_callbacks(H TSRMLS_CC); +		pdo_sqlite_cleanup_callbacks(H);  	}  } @@ -739,16 +735,16 @@ static struct pdo_dbh_methods sqlite_methods = {  	pdo_sqlite_request_shutdown  }; -static char *make_filename_safe(const char *filename TSRMLS_DC) +static char *make_filename_safe(const char *filename)  {  	if (*filename && memcmp(filename, ":memory:", sizeof(":memory:"))) { -		char *fullpath = expand_filepath(filename, NULL TSRMLS_CC); +		char *fullpath = expand_filepath(filename, NULL);  		if (!fullpath) {  			return NULL;  		} -		if (php_check_open_basedir(fullpath TSRMLS_CC)) { +		if (php_check_open_basedir(fullpath)) {  			efree(fullpath);  			return NULL;  		} @@ -763,8 +759,7 @@ static int authorizer(void *autharg, int access_type, const char *arg3, const ch  	char *filename;  	switch (access_type) {  		case SQLITE_COPY: { -			TSRMLS_FETCH(); -			filename = make_filename_safe(arg4 TSRMLS_CC); +					filename = make_filename_safe(arg4);  			if (!filename) {  				return SQLITE_DENY;  			} @@ -773,8 +768,7 @@ static int authorizer(void *autharg, int access_type, const char *arg3, const ch  		}  		case SQLITE_ATTACH: { -			TSRMLS_FETCH(); -			filename = make_filename_safe(arg3 TSRMLS_CC); +					filename = make_filename_safe(arg3);  			if (!filename) {  				return SQLITE_DENY;  			} @@ -788,7 +782,7 @@ static int authorizer(void *autharg, int access_type, const char *arg3, const ch  	}  } -static int pdo_sqlite_handle_factory(pdo_dbh_t *dbh, zval *driver_options TSRMLS_DC) /* {{{ */ +static int pdo_sqlite_handle_factory(pdo_dbh_t *dbh, zval *driver_options) /* {{{ */  {  	pdo_sqlite_db_handle *H;  	int i, ret = 0; @@ -801,10 +795,10 @@ static int pdo_sqlite_handle_factory(pdo_dbh_t *dbh, zval *driver_options TSRMLS  	H->einfo.errmsg = NULL;  	dbh->driver_data = H; -	filename = make_filename_safe(dbh->data_source TSRMLS_CC); +	filename = make_filename_safe(dbh->data_source);  	if (!filename) { -		zend_throw_exception_ex(php_pdo_get_exception(), 0 TSRMLS_CC, +		zend_throw_exception_ex(php_pdo_get_exception(), 0,  			"open_basedir prohibits opening %s",  			dbh->data_source);  		goto cleanup; @@ -823,7 +817,7 @@ static int pdo_sqlite_handle_factory(pdo_dbh_t *dbh, zval *driver_options TSRMLS  	}  	if (driver_options) { -		timeout = pdo_attr_lval(driver_options, PDO_ATTR_TIMEOUT, timeout TSRMLS_CC); +		timeout = pdo_attr_lval(driver_options, PDO_ATTR_TIMEOUT, timeout);  	}  	sqlite3_busy_timeout(H->db, timeout * 1000); | 
