From: George Peter Banyard Date: Fri, 12 Mar 2021 03:05:04 +0000 (+0000) Subject: Boolify pdo_stmt_describe_columns() X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=262744ff0d0e587b16190d3340a820d0837af29b;p=php Boolify pdo_stmt_describe_columns() --- diff --git a/ext/pdo/pdo_dbh.c b/ext/pdo/pdo_dbh.c index 316c9e3f97..55dac17434 100644 --- a/ext/pdo/pdo_dbh.c +++ b/ext/pdo/pdo_dbh.c @@ -1084,7 +1084,7 @@ PHP_METHOD(PDO, query) /* now execute the statement */ PDO_STMT_CLEAR_ERR(); if (stmt->methods->executer(stmt)) { - int ret = 1; + bool ret = true; if (!stmt->executed) { if (stmt->dbh->alloc_own_columns) { ret = pdo_stmt_describe_columns(stmt); diff --git a/ext/pdo/pdo_stmt.c b/ext/pdo/pdo_stmt.c index dbd2666226..c7d44028c2 100644 --- a/ext/pdo/pdo_stmt.c +++ b/ext/pdo/pdo_stmt.c @@ -125,7 +125,7 @@ iterate: } /* }}} */ -int pdo_stmt_describe_columns(pdo_stmt_t *stmt) /* {{{ */ +bool pdo_stmt_describe_columns(pdo_stmt_t *stmt) /* {{{ */ { int col; @@ -133,7 +133,7 @@ int pdo_stmt_describe_columns(pdo_stmt_t *stmt) /* {{{ */ for (col = 0; col < stmt->column_count; col++) { if (!stmt->methods->describer(stmt, col)) { - return 0; + return false; } /* if we are applying case conversions on column names, do so now */ @@ -168,7 +168,7 @@ int pdo_stmt_describe_columns(pdo_stmt_t *stmt) /* {{{ */ } } - return 1; + return true; } /* }}} */ diff --git a/ext/pdo/php_pdo_int.h b/ext/pdo/php_pdo_int.h index 1e8dcccb9e..4027721200 100644 --- a/ext/pdo/php_pdo_int.h +++ b/ext/pdo/php_pdo_int.h @@ -39,7 +39,7 @@ extern zend_class_entry *pdo_dbstmt_ce; void pdo_dbstmt_free_storage(zend_object *std); zend_object_iterator *pdo_stmt_iter_get(zend_class_entry *ce, zval *object, int by_ref); extern zend_object_handlers pdo_dbstmt_object_handlers; -int pdo_stmt_describe_columns(pdo_stmt_t *stmt); +bool pdo_stmt_describe_columns(pdo_stmt_t *stmt); bool pdo_stmt_setup_fetch_mode(pdo_stmt_t *stmt, zend_long mode, uint32_t mode_arg_num, zval *args, uint32_t variadic_num_args);