| Summary: | hardcoded fonts in About dialog | ||
|---|---|---|---|
| Product: | Branch 4.1 | Reporter: | Dmitry Chistikov <dd1email> |
| Component: | xpdf | Assignee: | Victor Forsyuk <force> |
| Status: | CLOSED WONTFIX | QA Contact: | qa-4.1 <qa-4.1> |
| Severity: | minor | ||
| Priority: | P3 | ||
| Version: | unspecified | ||
| Hardware: | all | ||
| OS: | Linux | ||
Что неправильно - соглашусь. Поэтому, если предложите патч устраняющий указанный недостаток, апстрим вряд ли откажется. Но данная особенность xpdf не тянет на баг, ради которого стоило бы выпускать обновление пакета для устаревшего бранча... По-моему, данная особенность xpdf тянет на баг. Насколько я вижу, в Sisyphus-версии тоже есть эти самые hardcoded шрифты. Вы имеете в виду, что баг нужно было вешать на другой продукт, потому что в этой ветке он не будет исправляться? (В ответ на комментарий №3) > По-моему, данная особенность xpdf тянет на баг. Любую программу можно улучшить. То что Вы описали это особенность данной программы - использование в этом месте hradcoded шрифтов. Если Вы можете предложить лучший код - патчи принимаются. > Насколько я вижу, в > Sisyphus-версии тоже есть эти самые hardcoded шрифты. Вы имеете в виду, что баг > нужно было вешать на другой продукт, потому что в этой ветке он не будет > исправляться? В этой ветке нецелесообразно выпускать обновление для такого улучшения "потребительских свойств" xpdf. |
$ fgrep -C2 -- '-*-times-' xpdf-3.02/xpdf/XPDFViewer.cc //----- fonts aboutBigFont = createFontList("-*-times-bold-i-normal--20-*-*-*-*-*-iso8859-1"); aboutVersionFont = createFontList("-*-times-medium-r-normal--16-*-*-*-*-*-iso8859-1"); aboutFixedFont = createFontList("-*-courier-medium-r-normal--12-*-*-*-*-*-iso8859-1"); Способа переопределить эти шрифты я не нашел, так что about dialog у меня квадратиками (соответствующие шрифты не установлены) и в stderr вываливаются три соответствующих warning'а. На мой взгляд, hardcoded без возможности переопределения имена шрифтов - это неправильно.