As I commented in bug #13067, it's not clear whether the manual or the
source is correct...Whichever it is, I think this patch may be the right
way to move from returning an int to returning a bool...
Index: db.c
===================================================================
RCS file: /repository/php4/ext/db/db.c,v
retrieving revision 1.64
diff -u -r1.64 db.c
--- db.c        14 Aug 2001 17:46:17 -0000      1.64
+++ db.c        31 Aug 2001 00:09:27 -0000
@@ -549,7 +549,6 @@
 {
        pval *id, *key, *value;
        dbm_info *info;
-       int ret;

        if (ZEND_NUM_ARGS()!=3||zend_get_parameters(ht, 3, &id, &key, &value) == 
FAILURE) {
                WRONG_PARAM_COUNT;
@@ -563,8 +562,11 @@
                RETURN_FALSE;
        }

-       ret = php_dbm_replace(info, Z_STRVAL_P(key), Z_STRVAL_P(value) TSRMLS_CC);
-       RETURN_LONG(ret);
+       if (php_dbm_replace(info, Z_STRVAL_P(key), Z_STRVAL_P(value) TSRMLS_CC) == 0) 
+{
+               RETURN_TRUE;
+       } else {
+               RETURN_FALSE;
+       }
 }
 /* }}} */
-- 
PHP Development Mailing List <http://www.php.net/>
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
To contact the list administrators, e-mail: [EMAIL PROTECTED]

Reply via email to