[PHP-CVS-DAILY] cvs: php-src / ChangeLog

2006-08-26 Thread changelog
changelog Sun Aug 27 01:30:50 2006 UTC Modified files: /php-srcChangeLog Log: ChangeLog update http://cvs.php.net/viewvc.cgi/php-src/ChangeLog?r1=1.2434r2=1.2435diff_format=u Index: php-src/ChangeLog diff -u php-src/ChangeLog:1.2434

[PHP-CVS] cvs: CVSROOT / avail

2006-08-26 Thread Pierre-Alain Joye
pajoye Sat Aug 26 11:03:00 2006 UTC Modified files: /CVSROOTavail Log: - Tom H. karma for pear Audio_CAPTCHA http://cvs.php.net/viewvc.cgi/CVSROOT/avail?r1=1.1171r2=1.1172diff_format=u Index: CVSROOT/avail diff -u CVSROOT/avail:1.1171

[PHP-CVS] cvs: php-src(PHP_5_2) /ext/zip php_zip.c

2006-08-26 Thread Pierre-Alain Joye
pajoye Sat Aug 26 12:23:43 2006 UTC Modified files: (Branch: PHP_5_2) /php-src/ext/zipphp_zip.c Log: - don't rely/use VCWD_REALPATH, use expand_filepath instead NB: that'd to be true for all this VCWD mess

[PHP-CVS] cvs: php-src /ext/zip php_zip.c

2006-08-26 Thread Pierre-Alain Joye
pajoye Sat Aug 26 12:29:59 2006 UTC Modified files: /php-src/ext/zipphp_zip.c Log: - MFB: don't rely/use VCWD_REALPATH, use expand_filepath instead NB: that'd to be true for all this VCWD mess

[PHP-CVS] cvs: php-src(PHP_4_4) / NEWS /ext/standard aggregation.c /ext/standard/tests/aggregation bug37812.phpt

2006-08-26 Thread Hannes Magnusson
bjori Sat Aug 26 14:15:07 2006 UTC Added files: (Branch: PHP_4_4) /php-src/ext/standard/tests/aggregation bug37812.phpt Modified files: /php-src/ext/standard aggregation.c /php-srcNEWS Log: Fixed bug #37812

RE: [PHP-CVS] cvs: php-src(PHP_5_2) /ext/zip php_zip.c

2006-08-26 Thread Andi Gutmans
I missed the discussion about this. Is there a bug in VCWD_REALPATH()? Can you explain? Thanks. -Original Message- From: Pierre-Alain Joye [mailto:[EMAIL PROTECTED] Sent: Saturday, August 26, 2006 5:24 AM To: php-cvs@lists.php.net Subject: [PHP-CVS] cvs: php-src(PHP_5_2) /ext/zip

RE: [PHP-CVS] cvs: php-src(PHP_5_2) /ext/bz2 bz2.c php_bz2.h /ext/ctype ctype.c php_ctype.h /ext/json json.c php_json.h /ext/pspell php_pspell.h pspell.c /ext/tidy php_tidy.h tidy.c /ext/zlib php

2006-08-26 Thread Andi Gutmans
Why is this important? We never defined these as static. As long as you move them to the .c file (for whatever reason you have that might be justified) you really don't have to define them as static IMO. -Original Message- From: Nuno Lopes [mailto:[EMAIL PROTECTED] Sent: Monday,

Re: [PHP-CVS] cvs: php-src(PHP_5_2) /ext/bz2 bz2.c php_bz2.h /ext/ctype ctype.c php_ctype.h /ext/json json.c php_json.h /ext/pspell php_pspell.h pspell.c /ext/tidy php_tidy.h tidy.c /ext/zlib php

2006-08-26 Thread Ilia Alshanetsky
Andi, I think the idea here was to minimize memory utilization and allow the compiler to optimize the code better, something that having functions defined as statics appears to help with. On 26-Aug-06, at 11:30 AM, Andi Gutmans wrote: Why is this important? We never defined these as

RE: [PHP-CVS] cvs: php-src(PHP_5_2) /ext/bz2 bz2.c php_bz2.h /ext/ctype ctype.c php_ctype.h /ext/json json.c php_json.h /ext/pspell php_pspell.h pspell.c /ext/tidy php_tidy.h tidy.c /ext/zlib php

2006-08-26 Thread Andi Gutmans
I don't think it helps. It won't inline these functions in most cases anyway, and having or not having a symbol shouldn't make a difference. I'd prefer to stick to our standards and keep all extensions the same. Who knows if/when we'll want to play around with the symbols also. It could come in