[Wine-devel] [Wine-patches] [eterhack] ntdll : add ThreadQuerySetWin32StartAddress in NtQueryInformationThread

[Dmitry Timoshkov]

Roman Dadkov <romand на etersoft.ru> wrote:

> [PATCH] ntdll : add ThreadQuerySetWin32StartAddress in
> NtQueryInformationThread

Цитата из
http://winehq.org.ru/%D0%9F%D0%B0%D1%82%D1%87%D0%B8_%D0%B4%D0%BB%D1%8F_WINE@Etersoft

"Перед созданием хака, специфичного для WINE на Etersoft, нужно приложить максимум
усилий для по возможности полного исследования проблемы и её источника, создания
минимальной программы для воспроизведения проблемы, и конечно для реального
исправления, а не очередного костыля в коде.
Если есть уверенность, что исправление (скорее всего) правильное, рекомендуется
пропускать его через фильтр wine-patches на winehq.org"

Роман, Ваша основная задача не просто решить проблему, а решить ее
правильно, т.е. приложив максимум усилий и избежать добавления "очередного
костыля в коде". А добавление хаков в такой низкоуровневый код как ntdll и
wineserver должно быть прямо запрещено без чрезвычайной необходимости.

Хотел бы напомнить, что добавление очередного подобного хака очень сильно
затрудняет работу по поиску и диагностированию проблем в коде Wine, не говоря
уже об очевидных проблемах импорта из апстрима. Повторю еще раз, что подобные
хаки - это в большинстве случаев непротестированный, не прошедший минимального
контроля качества вредоносный код, 

Поэтому предлагаю:
1. откатить это патч.
2. отправить патч в wine-patches на winehq.org и сделать все возможное. чтобы
патч был принят. Если патч не принят - это оценка Вашего качества работы.

-- 
Dmitry.