Re: new PostgreSQL API
| From: | Hartmut Holzgraefe | Date: | Mon, 07 Jul 2008 14:51:58 +0000 |
| Subject: | Re: new PostgreSQL API | ||
| References: | 1 2 3 4 5 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-38823@lists.php.net to get a copy of this message | ||
Hartmut Holzgraefe wrote:
i've started looking into the patch yesterday, there are still some small issues with it though ... i'll provide you with more detailed comments later today or early tomorrow ...open questions: - should the functions return an error if using the specified oid isn't possible or just throw a warning and return the oid that was actually assigned and used by the server? changes: - there was a mismatch between autoconf macro names and what was tested against in the source - ZEND_ARG_INFO entries for the new parameter - pg_lo_create() - option parsing is more lightweight and only
one #ifdef is needed- added a test case file - updated documentation - pg_lo_import() - new parameter is now placed after file name, not before,
as we usually do with new parameters- only one #ifdef block needed - updated documentation - still need to add a test case, function compiles but
i haven't tested it yet- possible coding style issues, still breaking in a new emacs installation ... -- Hartmut Holzgraefe, MySQL Regional Support Manager EMEA Sun Microsystems GmbH, Sonnenallee 1, 85551 Kirchheim-Heimstetten Amtsgericht Muenchen: HRB161028 Geschaeftsfuehrer: Thomas Schroeder, Wolfgang Engels, Dr. Roland Boemer Vorsitzender des Aufsichtsrates: Martin Haering