[PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2009-04-01 Thread Pierre-Alain Joye
pajoye Wed Apr 1 16:14:50 2009 UTC Modified files: /php-src/ext/pdo_pgsql pgsql_driver.c Log: - MFB: fix build when pqprepare does not exist http://cvs.php.net/viewvc.cgi/php-src/ext/pdo_pgsql/pgsql_driver.c?r1=1.72&r2=1.73&diff_format=u Index: php-src/ex

[PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c pgsql_statement.c /ext/pdo_pgsql/tests bug44861.phpt

2009-03-27 Thread Matteo Beccati
mbeccatiSat Mar 28 03:01:38 2009 UTC Modified files: /php-src/ext/pdo_pgsql pgsql_driver.c pgsql_statement.c /php-src/ext/pdo_pgsql/testsbug44861.phpt Log: MFB: - Fixed bug #44861 (scrollable cursor don't work with pgsql) http://cv

[PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2007-12-04 Thread Ilia Alshanetsky
iliaa Tue Dec 4 13:07:30 2007 UTC Modified files: /php-src/ext/pdo_pgsql pgsql_driver.c Log: MFB: Fixed bug #43493 (pdo_pgsql does not send username on connect when password is not available) http://cvs.php.net/viewvc.cgi/php-src/ext/pdo_pgsql/pg

[PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2006-12-18 Thread Ilia Alshanetsky
iliaa Mon Dec 18 17:56:51 2006 UTC Modified files: /php-src/ext/pdo_pgsql pgsql_driver.c Log: MFB: Fixed bug #39845 (Persistent connections generate a warning in pdo_pgsql). http://cvs.php.net/viewvc.cgi/php-src/ext/pdo_pgsql/pgsql_driver.c?r1=1.64&r

[PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c /ext/standard basic_functions.c

2006-12-16 Thread Hannes Magnusson
bjori Sat Dec 16 18:27:43 2006 UTC Modified files: /php-src/ext/standard basic_functions.c /php-src/ext/pdo_pgsql pgsql_driver.c Log: Fix ZTS build http://cvs.php.net/viewvc.cgi/php-src/ext/standard/basic_functions.c?r1=1.833&r2=1.834&diff_f

[PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c /ext/pgsql pgsql.c

2006-11-29 Thread Ilia Alshanetsky
iliaa Wed Nov 29 15:47:41 2006 UTC Modified files: /php-src/ext/pgsql pgsql.c /php-src/ext/pdo_pgsql pgsql_driver.c Log: MFB: Fixed bug #39663 (Memory leak in pg_get_notify() and a possible memory corruption on Windows in pgsql and pdo_pgsql extensio

[PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c pgsql_statement.c

2006-05-08 Thread Ilia Alshanetsky
iliaa Mon May 8 14:33:23 2006 UTC Modified files: /php-src/ext/pdo_pgsql pgsql_driver.c pgsql_statement.c Log: MFB: Fixed memory leaks when working with cursors in PDO PostgreSQL driver. http://cvs.php.net/viewcvs.cgi/php-src/ext/pdo_pgsql/pgsql_dr

[PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2006-03-14 Thread Edin Kadribasic
edink Tue Mar 14 11:14:05 2006 UTC Modified files: /php-src/ext/pdo_pgsql pgsql_driver.c Log: MFB: Fixed build http://cvs.php.net/viewcvs.cgi/php-src/ext/pdo_pgsql/pgsql_driver.c?r1=1.57&r2=1.58&diff_format=u Index: php-src/ext/pdo_pgsql/pgsql_driver.c di

[PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2006-01-29 Thread Ilia Alshanetsky
iliaa Sun Jan 29 17:36:12 2006 UTC Modified files: /php-src/ext/pdo_pgsql pgsql_driver.c Log: MFB51: Fixed bug #36176 (PDO_PGSQL - PDO::exec() does not return number of rows affected by the operation). http://cvs.php.net/viewcvs.cgi/php-src/ext/pdo_

[PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c pgsql_statement.c

2005-07-08 Thread Ilia Alshanetsky
iliaa Sat Jul 9 00:54:07 2005 EDT Modified files: /php-src/ext/pdo_pgsql pgsql_driver.c pgsql_statement.c Log: Added cursor closer handler. Fixed memory leak. http://cvs.php.net/diff.php/php-src/ext/pdo_pgsql/pgsql_driver.c?r1=1.50&r2=1.51&ty=u Inde

Re: [PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2005-07-07 Thread Ilia Alshanetsky
Uncommon is totally irrelevant. Stable, robust code is what is. What is this - the "Cross your fingers and hope it works programming language"?? Nah, it's more along the lines of "works for me" :-) Ilia -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/u

Re: [PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2005-07-07 Thread Christopher Kings-Lynne
It is malloc + PQescapeString + free, which in most cases are completely unnecessary. While as Christopher demonstrates sequence name could contain special chars, I'd wager that is a VERY uncommon situation. Uncommon is totally irrelevant. Stable, robust code is what is. What is this - the "

Re: [PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2005-07-07 Thread Andi Gutmans
Oh, that's a different story then... At 01:54 PM 7/7/2005 +0800, Christopher Kings-Lynne wrote: It does impose a v7.4 and above libpq requirement, however. Chris Andi Gutmans wrote: Yeah, definitely seems like PQexecParams is the safest and easiest way of doing so. In general, I'm always in f

Re: [PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2005-07-07 Thread Wez Furlong
How about using a stack variable for this temporary allocation? On 7/7/05, Ilia Alshanetsky <[EMAIL PROTECTED]> wrote: > Edink, > > It is malloc + PQescapeString + free, which in most cases are completely > unnecessary. While as Christopher demonstrates sequence name could > contain special chars

Re: [PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2005-07-07 Thread Ilia Alshanetsky
Edink, It is malloc + PQescapeString + free, which in most cases are completely unnecessary. While as Christopher demonstrates sequence name could contain special chars, I'd wager that is a VERY uncommon situation. Ilia Edin Kadribasic wrote: Hi Ilia, Would you please revert this change?

Re: [PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2005-07-07 Thread Edin Kadribasic
Hi Ilia, Would you please revert this change? Saving one malloc/free in a function that has to query DB server anyway is not going to give you much performance improvement, while other methods are not portable across different PostgreSQL versions. Edin On Thursday 07 July 2005 02:52, Ilia Als

Re: [PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2005-07-06 Thread Christopher Kings-Lynne
It does impose a v7.4 and above libpq requirement, however. Chris Andi Gutmans wrote: Yeah, definitely seems like PQexecParams is the safest and easiest way of doing so. In general, I'm always in favor of using bound parameters exactly for this reason. Andi At 10:44 AM 7/7/2005 +0800, Chris

Re: [PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2005-07-06 Thread Andi Gutmans
Yeah, definitely seems like PQexecParams is the safest and easiest way of doing so. In general, I'm always in favor of using bound parameters exactly for this reason. Andi At 10:44 AM 7/7/2005 +0800, Christopher Kings-Lynne wrote: By the way, using PQexecParams is by far the easiest fastest an

Re: [PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2005-07-06 Thread Christopher Kings-Lynne
Have you ever tried creating a sequence with an apostrophes in its name in PostgreSQL? Easily, and don't forget about schema qualification either: test=# create sequence "isn't this ""grand""?"; CREATE SEQUENCE test=# \ds List of relations Schema |Name | Type

Re: [PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2005-07-06 Thread Christopher Kings-Lynne
I don't get this at all? How come you reject sequences that contain apostrophes? Chris Ilia Alshanetsky wrote: iliaa Wed Jul 6 20:52:20 2005 EDT Modified files: /php-src/ext/pdo_pgsql pgsql_driver.c Log: Faster sequence id retrieval. http://cvs.p

Re: [PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2005-07-06 Thread Christopher Kings-Lynne
By the way, using PQexecParams is by far the easiest fastest and safest way of doing this... Chris Ilia Alshanetsky wrote: iliaa Wed Jul 6 22:17:21 2005 EDT Modified files: /php-src/ext/pdo_pgsql pgsql_driver.c Log: Leave it up to the user to decide if to

Re: [PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2005-07-06 Thread Christopher Kings-Lynne
Leave it up to the user to decide if to escape the sequence name or not. What was wrong with the original coding? Can't you juse leave teh PQescapeString in there? Chris http://cvs.php.net/diff.php/php-src/ext/pdo_pgsql/pgsql_driver.c?r1=1.47&r2=1.48&ty=u Index: php-src/ext/pdo_pgsql/pgs

Re: [PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2005-07-06 Thread Ilia Alshanetsky
Thanks for the suggestion, I'll look into it tomorrow morning. Ilia Christopher Kings-Lynne wrote: By the way, using PQexecParams is by far the easiest fastest and safest way of doing this... Chris Ilia Alshanetsky wrote: iliaaWed Jul 6 22:17:21 2005 EDT Modified files:

[PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2005-07-06 Thread Ilia Alshanetsky
iliaa Wed Jul 6 22:17:21 2005 EDT Modified files: /php-src/ext/pdo_pgsql pgsql_driver.c Log: Leave it up to the user to decide if to escape the sequence name or not. http://cvs.php.net/diff.php/php-src/ext/pdo_pgsql/pgsql_driver.c?r1=1.47&r2=1.48&ty=u

Re: [PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2005-07-06 Thread Ilia Alshanetsky
Christopher Kings-Lynne wrote: I don't get this at all? How come you reject sequences that contain apostrophes? Have you ever tried creating a sequence with an apostrophes in its name in PostgreSQL? Ilia -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.ne

[PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2005-07-06 Thread Ilia Alshanetsky
iliaa Wed Jul 6 20:52:20 2005 EDT Modified files: /php-src/ext/pdo_pgsql pgsql_driver.c Log: Faster sequence id retrieval. http://cvs.php.net/diff.php/php-src/ext/pdo_pgsql/pgsql_driver.c?r1=1.46&r2=1.47&ty=u Index: php-src/ext/pdo_pgsql/pgsql_driv

[PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2005-07-01 Thread Edin Kadribasic
edink Fri Jul 1 18:43:17 2005 EDT Modified files: /php-src/ext/pdo_pgsql pgsql_driver.c Log: Only check for InvalidOid when not looking up a sequence http://cvs.php.net/diff.php/php-src/ext/pdo_pgsql/pgsql_driver.c?r1=1.45&r2=1.46&ty=u Index: php-src/ext

[PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c pgsql_statement.c php_pdo_pgsql_int.h

2005-07-01 Thread Edin Kadribasic
edink Fri Jul 1 18:30:55 2005 EDT Modified files: /php-src/ext/pdo_pgsql pgsql_driver.c pgsql_statement.c php_pdo_pgsql_int.h Log: Fold PQresultErrorField() into a macro http://cvs.php.net/diff.php/php-src/ext/pdo_pgsql/p

[PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2005-07-01 Thread Edin Kadribasic
edink Fri Jul 1 17:54:52 2005 EDT Modified files: /php-src/ext/pdo_pgsql pgsql_driver.c Log: Added support for fetching current value of a sequence when the optional sequence name has been passed to PDO::lastInsertId() http://cvs.php.net/diff.php/ph

Re: [PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2005-06-30 Thread Ilia Alshanetsky
Nope, 32 bit value not a 64 bit one. ilia Wez Furlong wrote: Is this what you're really looking for? PDO_API char *php_pdo_int64_to_str(pdo_int64_t i64 TSRMLS_DC) --Wez. On 6/29/05, Ilia Alshanetsky <[EMAIL PROTECTED]> wrote: iliaa Wed Jun 29 20:07:44 2005 EDT Modified files:

Re: [PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2005-06-30 Thread Wez Furlong
Is this what you're really looking for? PDO_API char *php_pdo_int64_to_str(pdo_int64_t i64 TSRMLS_DC) --Wez. On 6/29/05, Ilia Alshanetsky <[EMAIL PROTECTED]> wrote: > iliaa Wed Jun 29 20:07:44 2005 EDT > > Modified files: > /php-src/ext/pdo_pgsql pgsql_driver.c > Log: >

[PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2005-06-29 Thread Ilia Alshanetsky
iliaa Wed Jun 29 20:07:44 2005 EDT Modified files: /php-src/ext/pdo_pgsql pgsql_driver.c Log: Silence warnings. http://cvs.php.net/diff.php/php-src/ext/pdo_pgsql/pgsql_driver.c?r1=1.42&r2=1.43&ty=u Index: php-src/ext/pdo_pgsql/pgsql_driver.c diff -u ph

[PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2005-06-28 Thread Ilia Alshanetsky
iliaa Tue Jun 28 10:54:45 2005 EDT Modified files: /php-src/ext/pdo_pgsql pgsql_driver.c Log: Make exec() return FALSE on error as do other drivers. http://cvs.php.net/diff.php/php-src/ext/pdo_pgsql/pgsql_driver.c?r1=1.41&r2=1.42&ty=u Index: php-src/ex

[PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2005-06-27 Thread Ilia Alshanetsky
iliaa Tue Jun 28 00:04:00 2005 EDT Modified files: /php-src/ext/pdo_pgsql pgsql_driver.c Log: Fixed double-free bug. http://cvs.php.net/diff.php/php-src/ext/pdo_pgsql/pgsql_driver.c?r1=1.40&r2=1.41&ty=u Index: php-src/ext/pdo_pgsql/pgsql_driver.c diff

[PHP-CVS] cvs: php-src /ext/pdo_pgsql pgsql_driver.c

2005-06-27 Thread Ilia Alshanetsky
iliaa Mon Jun 27 22:09:54 2005 EDT Modified files: /php-src/ext/pdo_pgsql pgsql_driver.c Log: fixed oid retrieval. http://cvs.php.net/diff.php/php-src/ext/pdo_pgsql/pgsql_driver.c?r1=1.39&r2=1.40&ty=u Index: php-src/ext/pdo_pgsql/pgsql_driver.c diff -u