Re: cvs: php-src(PHP_5_3) /ext/odbc php_odbc.c php_odbc.h php_odbc_includes.h
| From: | Jani Taskinen | Date: | Tue, 02 Sep 2008 09:03:54 +0000 |
| Subject: | Re: cvs: php-src(PHP_5_3) /ext/odbc php_odbc.c php_odbc.h php_odbc_includes.h | ||
| References: | 1 2 3 | Groups: | php.cvs |
| Request: | Send a blank email to php-cvs+get-52896@lists.php.net to get a copy of this message | ||
Patrick van Kleef wrote:
I have merged these fixes to HEAD, PHP5_3 and PHP5_2 branch as per your request. Jani: Please move the NEWS item fixes to PHP5_2 so i can see how you want this committed for future reference.Done.
I will start looking at the registered ODBC bugs in the system to see what problems there are and how they can be fixed. However this will not all happen today ;)You should watch your coding style and whitespace pollution. f.e. whitespace: we only use tabs. And control structures, use { } generously, shortening stuff like 'if (foo) bar;' is not the way to go.. (not saying you did that, just FYI :) And "splitting" lines: If you have to do that: Get a bigger screen! (short: don't do it) --Jani