summaryrefslogtreecommitdiff
path: root/ext/mysqli/mysqli_api.c
diff options
context:
space:
mode:
authorNikita Popov <nikita.ppv@gmail.com>2021-01-06 11:29:38 +0100
committerNikita Popov <nikita.ppv@gmail.com>2021-01-06 11:29:38 +0100
commit73f989a83b8338cd8348134b6df8dded27fae300 (patch)
treed0021e6ceda8d35c4542eefad95c222f60c2e15e /ext/mysqli/mysqli_api.c
parent9052f3b7d246c76f2f9732c43f52c8816886790d (diff)
parent44a311dbbebce3ffb511c5523a326688f2a9a3ab (diff)
downloadphp-git-73f989a83b8338cd8348134b6df8dded27fae300.tar.gz
Merge branch 'PHP-8.0'
* PHP-8.0: Fix/improve mysqli stubs
Diffstat (limited to 'ext/mysqli/mysqli_api.c')
-rw-r--r--ext/mysqli/mysqli_api.c35
1 files changed, 6 insertions, 29 deletions
diff --git a/ext/mysqli/mysqli_api.c b/ext/mysqli/mysqli_api.c
index 45c7dd196f..4f6b6c881f 100644
--- a/ext/mysqli/mysqli_api.c
+++ b/ext/mysqli/mysqli_api.c
@@ -614,17 +614,13 @@ PHP_FUNCTION(mysqli_character_set_name)
{
MY_MYSQL *mysql;
zval *mysql_link;
- const char *cs_name;
if (zend_parse_method_parameters(ZEND_NUM_ARGS(), getThis(), "O", &mysql_link, mysqli_link_class_entry) == FAILURE) {
RETURN_THROWS();
}
MYSQLI_FETCH_RESOURCE_CONN(mysql, mysql_link, MYSQLI_STATUS_VALID);
- cs_name = mysql_character_set_name(mysql->mysql);
- if (cs_name) {
- RETURN_STRING(cs_name);
- }
+ RETURN_STRING(mysql_character_set_name(mysql->mysql));
}
/* }}} */
@@ -801,16 +797,12 @@ PHP_FUNCTION(mysqli_error)
{
MY_MYSQL *mysql;
zval *mysql_link;
- const char *err;
if (zend_parse_method_parameters(ZEND_NUM_ARGS(), getThis(), "O", &mysql_link, mysqli_link_class_entry) == FAILURE) {
RETURN_THROWS();
}
MYSQLI_FETCH_RESOURCE_CONN(mysql, mysql_link, MYSQLI_STATUS_VALID);
- err = mysql_error(mysql->mysql);
- if (err) {
- RETURN_STRING(err);
- }
+ RETURN_STRING(mysql_error(mysql->mysql));
}
/* }}} */
@@ -1337,10 +1329,7 @@ PHP_FUNCTION(mysqli_get_client_info)
}
}
- const char * info = mysql_get_client_info();
- if (info) {
- RETURN_STRING(info);
- }
+ RETURN_STRING(mysql_get_client_info());
}
/* }}} */
@@ -2149,16 +2138,12 @@ PHP_FUNCTION(mysqli_sqlstate)
{
MY_MYSQL *mysql;
zval *mysql_link;
- const char *state;
if (zend_parse_method_parameters(ZEND_NUM_ARGS(), getThis(), "O", &mysql_link, mysqli_link_class_entry) == FAILURE) {
RETURN_THROWS();
}
MYSQLI_FETCH_RESOURCE_CONN(mysql, mysql_link, MYSQLI_STATUS_VALID);
- state = mysql_sqlstate(mysql->mysql);
- if (state) {
- RETURN_STRING(state);
- }
+ RETURN_STRING(mysql_sqlstate(mysql->mysql));
}
/* }}} */
@@ -2358,17 +2343,13 @@ PHP_FUNCTION(mysqli_stmt_error)
{
MY_STMT *stmt;
zval *mysql_stmt;
- const char * err;
if (zend_parse_method_parameters(ZEND_NUM_ARGS(), getThis(), "O", &mysql_stmt, mysqli_stmt_class_entry) == FAILURE) {
RETURN_THROWS();
}
MYSQLI_FETCH_RESOURCE_STMT(stmt, mysql_stmt, MYSQLI_STATUS_INITIALIZED);
- err = mysql_stmt_error(stmt->stmt);
- if (err) {
- RETURN_STRING(err);
- }
+ RETURN_STRING(mysql_stmt_error(stmt->stmt));
}
/* }}} */
@@ -2499,17 +2480,13 @@ PHP_FUNCTION(mysqli_stmt_sqlstate)
{
MY_STMT *stmt;
zval *mysql_stmt;
- const char * state;
if (zend_parse_method_parameters(ZEND_NUM_ARGS(), getThis(), "O", &mysql_stmt, mysqli_stmt_class_entry) == FAILURE) {
RETURN_THROWS();
}
MYSQLI_FETCH_RESOURCE_STMT(stmt, mysql_stmt, MYSQLI_STATUS_VALID);
- state = mysql_stmt_sqlstate(stmt->stmt);
- if (state) {
- RETURN_STRING(state);
- }
+ RETURN_STRING(mysql_stmt_sqlstate(stmt->stmt));
}
/* }}} */