Hi! > I'm not sure that this change is right. As it is now one could no > longer pass in "123" (as a string) or 123.0 (as a float) as > limit/offset. This goes against usual PHP semantics.
I agree, this: >> - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, >> "O|ll", &zobject, ce_inner, &intern->u.limit.offset, &intern->u.limit.count) >> == FAILURE) { >> + if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, >> "O|zz", &zobject, ce_inner, &tmp_offset, &tmp_count) == FAILURE) { Is wrong, especially added in 5.3/5.4. I'm getting complaints here about changes that only theoretically affect some future extensions, but here we got API change in stable version that would deny things like "1" in numeric context, which is completely contrary to how PHP does things. Please revert this change. -- Stanislav Malyshev, Software Architect SugarCRM: http://www.sugarcrm.com/ (408)454-6900 ext. 227 -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php