From:             [EMAIL PROTECTED]
Operating system: Irrelevant
PHP version:      5.0.1
PHP Bug Type:     MCAL related
Bug description:  Parameters of mcal_fetch_event()

Description:
------------
Proto for mcal_fetch_event() says it accepts 2 or 3 parameters but code
says it accepts 1, 2 or 3 parameters. I belive proto is right in this case
and I am sending a patch for it.

Patch proposal:

RCS file: /repository/pecl/mcal/php_mcal.c,v
retrieving revision 1.59
diff -u -r1.59 php_mcal.c
--- mcal/php_mcal.c     16 Jun 2003 14:03:30 -0000      1.59
+++ mcal/php_mcal.c     18 Aug 2004 12:02:38 -0000
@@ -459,7 +459,7 @@
        CALEVENT *myevent;
        int myargcount=ZEND_NUM_ARGS();
        
-       if (myargcount < 1 || myargcount > 3 ||
zend_get_parameters_ex(myargcount, &streamind, &eventid, &options) ==
FAILURE) {
+       if (myargcount < 2 || myargcount > 3 ||
zend_get_parameters_ex(myargcount, &streamind, &eventid, &options) ==
FAILURE) {
                WRONG_PARAM_COUNT;
        }
        convert_to_long_ex(streamind);



-- 
Edit bug report at http://bugs.php.net/?id=29753&edit=1
-- 
Try a CVS snapshot (php4):  http://bugs.php.net/fix.php?id=29753&r=trysnapshot4
Try a CVS snapshot (php5):  http://bugs.php.net/fix.php?id=29753&r=trysnapshot5
Fixed in CVS:               http://bugs.php.net/fix.php?id=29753&r=fixedcvs
Fixed in release:           http://bugs.php.net/fix.php?id=29753&r=alreadyfixed
Need backtrace:             http://bugs.php.net/fix.php?id=29753&r=needtrace
Need Reproduce Script:      http://bugs.php.net/fix.php?id=29753&r=needscript
Try newer version:          http://bugs.php.net/fix.php?id=29753&r=oldversion
Not developer issue:        http://bugs.php.net/fix.php?id=29753&r=support
Expected behavior:          http://bugs.php.net/fix.php?id=29753&r=notwrong
Not enough info:            http://bugs.php.net/fix.php?id=29753&r=notenoughinfo
Submitted twice:            http://bugs.php.net/fix.php?id=29753&r=submittedtwice
register_globals:           http://bugs.php.net/fix.php?id=29753&r=globals
PHP 3 support discontinued: http://bugs.php.net/fix.php?id=29753&r=php3
Daylight Savings:           http://bugs.php.net/fix.php?id=29753&r=dst
IIS Stability:              http://bugs.php.net/fix.php?id=29753&r=isapi
Install GNU Sed:            http://bugs.php.net/fix.php?id=29753&r=gnused
Floating point limitations: http://bugs.php.net/fix.php?id=29753&r=float

Reply via email to