diff options
author | Nikita Popov <nikita.ppv@gmail.com> | 2021-01-06 11:29:38 +0100 |
---|---|---|
committer | Nikita Popov <nikita.ppv@gmail.com> | 2021-01-06 11:29:38 +0100 |
commit | 73f989a83b8338cd8348134b6df8dded27fae300 (patch) | |
tree | d0021e6ceda8d35c4542eefad95c222f60c2e15e /ext/mysqli/mysqli_api.c | |
parent | 9052f3b7d246c76f2f9732c43f52c8816886790d (diff) | |
parent | 44a311dbbebce3ffb511c5523a326688f2a9a3ab (diff) | |
download | php-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.c | 35 |
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)); } /* }}} */ |