Bug 6087

Summary: comdlg32: отправить патч в winehq
Product: WINE@Etersoft Reporter: Виталий Перов <vitperov>
Component: Печать ; Диалог печатиAssignee: Евгений <evcalipt>
Status: CLOSED INVALID QA Contact:
Severity: trivial    
Priority: P5 CC: kondratyuk, lav, olezha
Version: 1.1Keywords: TO_WINEHQ
Target Milestone: ---   
Hardware: PC   
OS: Linux   
Whiteboard:
Заявки RT: Связано с:
Дата напоминания:
Bug Depends on: 225, 985    
Bug Blocks: 443, 3000, 6009    

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 дней.
Comment 5 Олег Шевченко 2024-04-15 18:59:56 MSK
Задача по релизу W@E 1.0.8
Уже давно не актуально, аннулирую.