[Wine-devel] [Wine-patches] [eter-2.1] [1/2] winspool.drv: Set pDevModeInputA to NULL if fMode not equal DM_IN_BUFFER(eterbug#8821)

[Dmitry Timoshkov]

Dmitry Timoshkov <dtimoshkov на etersoft.ru> wrote:

> > -    if(!pDeviceName) {
> > +    if(!pDeviceName && !*pDeviceName) {
> >          LPCWSTR lpNameW = get_opened_printer_name(hPrinter);
> >          if(!lpNameW) {
> >  		ERR("no name from hPrinter?\n");
> 
> Этот кусок, отправленный в wine-patches отдельно, отмечен как 'Rejected'
> в http://source.winehq.org/patches. Думаю, что вполне понятно почему.

В догонку к объяснялкам, что делать, если патч не приняли:

Поставьте себя на место AJ (или любого другого мейнтейнера open source
проекта): ему приходится выполнять свою собственную работу (часто очень
непростую и требующую времени и концентрации), плюс обрабатывать чужие
патчи. "Обрабатывать" - мягко сказано, это и просмотрт на предмет явных
логических ошибок, и понимание почему и зачем создатель патча принял то
или иное решение, и наконец просто компиляция и выполнение 'make test'
*для каждого отдельного патча*.

Что Вы лично делаете, если Вам непонятен смысл присланного патча и
отправитель не потрудился потратить время на объяснения? Правильно,
откладываете этот патч "на потом", а фактически это значит "навсегда".

Теперь представьте, что должен сделать AJ после того, как увидел выше
приведенный патч? Правильно, отметить его как "Rejected" и пропустить
всю остальную пачку от этого же создателя, подразумевая, что качество
остальных патчей такое же.

Что Вы можете сделать со своей стороны?
1. Отправить только этот один единственный патч снова. Сегодня!
Исправленный и проверенный Вами 100 раз снова и снова, не забыв и про
'make test'.
2. Убедиться, что тема этого нового патча соответствует правилам для
повторно отправляемых.
3. Не забыть хотя бы в этот раз написать (максимально подробно) причины
и цели создания этого патча.
4. Не ждать еще неделю или того хуже несколько месяцев перед повторной
отправкой. Вспомните, что мгновенная реакция на комментарий к Вашему
патчу будет оценена, как и ее отсутствие (а статус "Rejected" - это
именно реакция, и просмотр страницы со статусом патчей должно стать
Вашей привычкой в начале каждого рабочего дня после очередной пачки
коммитов AJ в wine-cvs на winehq.org).

-- 
Dmitry.