diff options
-rw-r--r-- | ext/dbx/dbx.c | 16 | ||||
-rw-r--r-- | ext/fbsql/php_fbsql.c | 6 | ||||
-rw-r--r-- | ext/imap/php_imap.c | 12 | ||||
-rw-r--r-- | ext/mcal/php_mcal.c | 4 | ||||
-rw-r--r-- | ext/msql/php_msql.c | 4 | ||||
-rw-r--r-- | ext/mssql/php_mssql.c | 5 | ||||
-rw-r--r-- | ext/mysql/php_mysql.c | 4 | ||||
-rw-r--r-- | ext/sybase/php_sybase_db.c | 5 | ||||
-rw-r--r-- | ext/sybase_ct/php_sybase_ct.c | 5 |
9 files changed, 17 insertions, 44 deletions
diff --git a/ext/dbx/dbx.c b/ext/dbx/dbx.c index ac6665a223..43b7c5c6e1 100644 --- a/ext/dbx/dbx.c +++ b/ext/dbx/dbx.c @@ -302,14 +302,7 @@ ZEND_FUNCTION(dbx_connect) RETURN_LONG(0); } - if (object_init(return_value) != SUCCESS) { - php_error_docref(NULL TSRMLS_CC, E_ERROR, "dbx: unable to create resulting object..."); - FREE_ZVAL(dbx_module); - zval_dtor(db_name); /* to free stringvalue memory */ - FREE_ZVAL(db_name); - FREE_ZVAL(rv_dbx_handle); - RETURN_LONG(0); - } + object_init(return_value); zend_hash_update(Z_OBJPROP_P(return_value), "handle", 7, (void *)&(rv_dbx_handle), sizeof(zval *), NULL); zend_hash_update(Z_OBJPROP_P(return_value), "module", 7, (void *)&(dbx_module), sizeof(zval *), NULL); @@ -427,11 +420,8 @@ ZEND_FUNCTION(dbx_query) * rv_result_handle holds a resource */ /* init return_value as object (of rows) */ - if (object_init(return_value) != SUCCESS) { - php_error_docref(NULL TSRMLS_CC, E_ERROR, "unable to create resulting object..."); - FREE_ZVAL(rv_result_handle); - RETURN_LONG(0); - } + object_init(return_value); + /* add result_handle property to return_value */ zend_hash_update(Z_OBJPROP_P(return_value), "handle", 7, (void *)&(rv_result_handle), sizeof(zval *), NULL); /* init info property as array and add to return_value as a property */ diff --git a/ext/fbsql/php_fbsql.c b/ext/fbsql/php_fbsql.c index bc1979dac5..7602098513 100644 --- a/ext/fbsql/php_fbsql.c +++ b/ext/fbsql/php_fbsql.c @@ -3059,10 +3059,8 @@ PHP_FUNCTION(fbsql_fetch_field) RETURN_FALSE; } } - if (object_init(return_value)==FAILURE) - { - RETURN_FALSE; - } + object_init(return_value); + add_property_string(return_value, "name", (char*)fbccmdLabelName(fbcmdColumnMetaDataAtIndex(result->metaData, column)), 1); add_property_string(return_value, "table", (char*)fbccmdTableName(fbcmdColumnMetaDataAtIndex(result->metaData, column)), 1); add_property_long(return_value, "max_length", fbcdmdLength(fbccmdDatatype(fbcmdColumnMetaDataAtIndex(result->metaData, column)))); diff --git a/ext/imap/php_imap.c b/ext/imap/php_imap.c index ca315598b5..41918579fa 100644 --- a/ext/imap/php_imap.c +++ b/ext/imap/php_imap.c @@ -1866,9 +1866,7 @@ PHP_FUNCTION(imap_mailboxmsginfo) ZEND_FETCH_RESOURCE(imap_le_struct, pils *, streamind, -1, "imap", le_imap); /* Initialize return object */ - if (object_init(return_value) == FAILURE) { - RETURN_FALSE; - } + object_init(return_value); unreadmsg = 0; deletedmsg = 0; @@ -2497,9 +2495,7 @@ PHP_FUNCTION(imap_status) convert_to_string_ex(mbx); convert_to_long_ex(flags); - if (object_init(return_value) == FAILURE) { - RETURN_FALSE; - } + object_init(return_value); if (mail_status(imap_le_struct->imap_stream, Z_STRVAL_PP(mbx), Z_LVAL_PP(flags))) { add_property_long(return_value, "flags", IMAPG(status_flags)); @@ -2548,9 +2544,7 @@ PHP_FUNCTION(imap_bodystruct) RETURN_FALSE; } - if (object_init(return_value) == FAILURE) { - RETURN_FALSE; - } + object_init(return_value); body=mail_body(imap_le_struct->imap_stream, Z_LVAL_PP(msg), Z_STRVAL_PP(section)); if (body->type <= TYPEMAX) { diff --git a/ext/mcal/php_mcal.c b/ext/mcal/php_mcal.c index 0c73edb383..6ecd3a5419 100644 --- a/ext/mcal/php_mcal.c +++ b/ext/mcal/php_mcal.c @@ -1367,9 +1367,7 @@ PHP_FUNCTION(mcal_next_recurrence) calevent_next_recurrence(mcal_le_struct->event, &mydate, Z_LVAL_PP(weekstart)); - if (object_init(return_value) == FAILURE) { - RETURN_FALSE; - } + object_init(return_value); if (mydate.has_date) { add_property_long(return_value, "year", mydate.year); diff --git a/ext/msql/php_msql.c b/ext/msql/php_msql.c index ddd97293cc..f4736f5df1 100644 --- a/ext/msql/php_msql.c +++ b/ext/msql/php_msql.c @@ -1095,9 +1095,7 @@ PHP_FUNCTION(msql_fetch_field) if (!msql_result || (msql_field=msqlFetchField(msql_result))==NULL) { RETURN_FALSE; } - if (object_init(return_value)==FAILURE) { - RETURN_FALSE; - } + object_init(return_value); add_property_string(return_value, "name",(msql_field->name?msql_field->name:empty_string), 1); add_property_string(return_value, "table",(msql_field->table?msql_field->table:empty_string), 1); diff --git a/ext/mssql/php_mssql.c b/ext/mssql/php_mssql.c index b44deff3c6..c60065c73e 100644 --- a/ext/mssql/php_mssql.c +++ b/ext/mssql/php_mssql.c @@ -1490,9 +1490,8 @@ PHP_FUNCTION(mssql_fetch_field) RETURN_FALSE; } - if (object_init(return_value)==FAILURE) { - RETURN_FALSE; - } + object_init(return_value); + add_property_string(return_value, "name",result->fields[field_offset].name, 1); add_property_long(return_value, "max_length",result->fields[field_offset].max_length); add_property_string(return_value, "column_source",result->fields[field_offset].column_source, 1); diff --git a/ext/mysql/php_mysql.c b/ext/mysql/php_mysql.c index eeefc0aeda..ef46b204a0 100644 --- a/ext/mysql/php_mysql.c +++ b/ext/mysql/php_mysql.c @@ -2111,9 +2111,7 @@ PHP_FUNCTION(mysql_fetch_field) if ((mysql_field=mysql_fetch_field(mysql_result))==NULL) { RETURN_FALSE; } - if (object_init(return_value)==FAILURE) { - RETURN_FALSE; - } + object_init(return_value); add_property_string(return_value, "name",(mysql_field->name?mysql_field->name:empty_string), 1); add_property_string(return_value, "table",(mysql_field->table?mysql_field->table:empty_string), 1); diff --git a/ext/sybase/php_sybase_db.c b/ext/sybase/php_sybase_db.c index e8e1026185..d96dad533d 100644 --- a/ext/sybase/php_sybase_db.c +++ b/ext/sybase/php_sybase_db.c @@ -1219,9 +1219,8 @@ PHP_FUNCTION(sybase_fetch_field) RETURN_FALSE; } - if (object_init(return_value)==FAILURE) { - RETURN_FALSE; - } + object_init(return_value); + add_property_string(return_value, "name",result->fields[field_offset].name, 1); add_property_long(return_value, "max_length",result->fields[field_offset].max_length); add_property_string(return_value, "column_source",result->fields[field_offset].column_source, 1); diff --git a/ext/sybase_ct/php_sybase_ct.c b/ext/sybase_ct/php_sybase_ct.c index a4782ed5d7..6c33f7ede2 100644 --- a/ext/sybase_ct/php_sybase_ct.c +++ b/ext/sybase_ct/php_sybase_ct.c @@ -1824,9 +1824,8 @@ PHP_FUNCTION(sybase_fetch_field) RETURN_FALSE; } - if (object_init(return_value)==FAILURE) { - RETURN_FALSE; - } + object_init(return_value); + add_property_string(return_value, "name", result->fields[field_offset].name, 1); add_property_long(return_value, "max_length", result->fields[field_offset].max_length); add_property_string(return_value, "column_source", result->fields[field_offset].column_source, 1); |