Укажите отработанное время

Отработанное время:
Продуктивное время:
Bug 6087 - comdlg32: отправить патч в winehq   Make a simular bug
Summary: comdlg32: отправить патч в winehq
Status: DEFERRED
Alias: None
Product: WINE@Etersoft
Classification: Продукты (Products)
Component: Печать ; Диалог печати (show other bugs)
Version: 1.1
Hardware: PC Linux
: P5 trivial
Target Milestone: ---
Assignee: Евгений
QA Contact:
URL:
Whiteboard:
Keywords: TO_WINEHQ
Depends on: 225 985
Blocks: 443 3000 6009
  Show dependency treegraph
 
In work:
Reported: 2010-09-29 16:50 MSD by Виталий Перов
Modified: 2015-02-28 16:56 MSK (History)
2 users (show)

See Also:
Заявки RT:
Связано с:
Дата напоминания:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Виталий Перов 2010-09-29 16:50:48 MSD
У нас давно приложен патч:

commit 19adc7372207171d0526a1ff21a85278a154ac13
Author: Константин Кондратюк <kondratyuk@builder.office.etersoft.ru>
Date:   Tue Apr 8 13:22:20 2008 +0400

    comdlg32: Fix eterbugs #225, #985

Хотелось бы выяснить нужен ли он ещё, и если да, то попытаться отправить его в winehq.

На первый взгляд патч выглядит не совсем корректно:
Если GetDefaultPrinterW возвращает ошибку, то проводятся какие-то минипуляции, чтобы получить имя принтера в обход.
Возможно стоит всё-таки разобраться почему не работает GetDefaultPrinter?
Comment 1 Константин Кондратюк 2010-09-29 17:21:32 MSD
(In reply to comment #0)
> У нас давно приложен патч:
> 
> commit 19adc7372207171d0526a1ff21a85278a154ac13
> Author: Константин Кондратюк
> <kondratyuk@builder.office.etersoft.ru>
> Date:   Tue Apr 8 13:22:20 2008 +0400

Так давно, что я ещё не умел настраивать пользовательские данные в git :)
Кстати, это как раз день, когда мы с cvs на git переехали.

Что касается исправления, то я не думаю, что оно достойно отправиться в winehq. Может, это и не совсем хак, но очень похоже.

Вообще, это было просто быстрое решение насущной задачи. Наверное, лучше бы его было запихнуть в GetDefaultPrinter(), но как-то не пришло в голову. Есть ли смысл тратить время на переделку одного "хака" в другой "хак" - не знаю.
Comment 2 Виталий Перов 2010-09-29 17:27:52 MSD
>Есть ли смысл тратить
>время на переделку одного "хака" в другой
>"хак" - не знаю.

Ну, сейчас это выглядит как хак.
Но на первый взгляд патч очень нужный.
Глубоко не разбирался, но думаю несложно довести его до состояния хорошего патча.
Так что смысл есть.
Comment 3 Виталий Перов 2010-11-10 18:56:54 MSK
Бага не важная, пока откладываю
Comment 4 Vitaly Lipatov 2014-09-11 18:48:04 MSK
Откладываем задачи, к которым не обращались более 100 дней.