Кто может принять патч в PDO?

[NT Man]

Подскажите пожалуйста кто компетентен в приеме патчей?

 

[tony2001]

1) Майнтейнер соотв-щего экстеншена (список: http://svn.php.net/viewvc/php/php-src/trunk/EXTENSIONS?revision=312251&view=markup)
2) [email protected]

Но посмотреть патч я могу.

 

[NT Man]

tiny2001, спасибо, если не затруднит https://bugs.php.net/bug.php?id=55768
В принципе и самая первая редакция решает проблему, но Крис посоветовал синхронизироваться с oci8 расширением и включить проверку на ORA-1010. Тестировал с помощью valgrind, вроде проблем нет.

 

[NT Man]

EXTENSION: pdo_oci
PRIMARY MAINTAINER: Unknown
MAINTENANCE: Odd fixes
STATUS: Working
SINCE: 5.1

У pdo_oci похоже нет майнтейнера

 

[tony2001]

Ну, если Крис посмотрел и он не против, то всё ок должно быть.
Закоммитил: http://news.php.net/php.cvs/66678

Несколько (минорных) замечаний:
- не забывай про фолдинг {{{ }}}
- стиль if (some) return SUCCESS; не одобряется, лучше использовать
if (some) {
return SUCCESS;
}
- объявление переменных в середине кода не приветствуется, т.к. это ломает некоторые компиляторы (VC? я не помню какой именно).
но это всё ерунда, я поправил перед коммитом.

>У pdo_oci похоже нет майнтейнера
Я знаю, да.

 

[флоппик]

Ребята, вы молодцы. Правда-правда.