Maximus5 / conemu-old-issues

Old issues imported from googlecode. Project was moved to
https://conemu.github.io
6 stars 1 forks source link

[patch] Rudimentary BDF support #511

Closed Maximus5 closed 9 years ago

Maximus5 commented 9 years ago

Originally reported on Google Code with ID 511

Advantages:
1) Unicode+Bitmap (Windows supports either, not both)
2) Faster rendering

Reported by thecybershadow on 2012-03-19 08:11:09


Maximus5 commented 9 years ago
Продолжение будет?
Сходу:
* при выборе bdf ломается шрифт при курсиве (в панелях для группы файлов настроен курсив,
отрисовка похоже идет системным шрифтом)
* слетают цвета в редакторе при Colorer-TrueMod-Far3. (если не bdf - все ок)
* шрифты не различимы в списке (окно настроек). рядом может и ttf лежать. наверное
логично было бы для bdf добавить суффикс, типа "Small [bdf]"

Reported by ConEmu.Maximus5 on 2012-03-19 11:34:58

Maximus5 commented 9 years ago
ОК, сделаем.

Reported by thecybershadow on 2012-03-19 19:49:54

Maximus5 commented 9 years ago
> при выборе bdf ломается шрифт при курсиве

У меня не воспроизводится, но патч может поможет. Если нет, прошу шаги для воспроизведения.

Остается вопрос с рамочным шрифтом. Там в CreateFontIndirectMy довольно запутанная
логика, и я даже не знаю как правильно к этому подобраться. "Правильно" тут означало
бы позволить пользователю указать BDF-шрифт и как рамочный шрифт, но я не представляю
как это можно обобщить, и боюсь логику кода трогать. Может вы посмотрите?

Reported by thecybershadow on 2012-03-20 03:20:24


Maximus5 commented 9 years ago
All OK

Рамки посмотрю.

Reported by ConEmu.Maximus5 on 2012-03-20 06:23:28

Maximus5 commented 9 years ago
Не, пока не ОК

Can't retrieve font family from file:
C:\Far3\ConEmu\unifont-5.1.20080820.bdf
Continue?

Family может быть и не указан. Используем имя файла?
Шрифт качал как unifont-5.1.20080820.bdf.gz

Кроме того, зачем проверка "if (familyName.size() >= LF_FACESIZE)"?
Обрезать наверное в этом случае нужно?

Reported by ConEmu.Maximus5 on 2012-03-20 13:07:27

Maximus5 commented 9 years ago
Более того. Распаковал комплект gohufont.
Получил 8 одинаковых "GohuFont [BDF]". Как-то не айс. Таки склоняюсь к имени файла
вместо опционального FAMILY_NAME

Reported by ConEmu.Maximus5 on 2012-03-20 13:12:33

Maximus5 commented 9 years ago
Хотя не... если есть семейство - наверное лучше показывать одно имя, чтобы разные шрифты
можно было создавать (GohuFont - italic/bold/...)

Reported by ConEmu.Maximus5 on 2012-03-20 15:24:16

Maximus5 commented 9 years ago
> Обрезать наверное в этом случае нужно?

Да, обрезать думаю было бы лучше. Проблема с обрезанием - если есть несколько шрифтов,
с названием отличающимся только после 31й буквы. "Bitstream Vera Sans Mono" - уже 24
буквы (LF_FACESIZE=32).

> Используем имя файла?

Да. Сделаем.

> если есть семейство - наверное лучше показывать одно имя

Поддержка семейств изначально не входило в план "rudimentary", но ладно, я сделаю.

Reported by thecybershadow on 2012-03-21 01:12:32

Maximus5 commented 9 years ago
UniFont только частично monospace, между прочим. Не знаю, если стоит поддерживать растерные
не-monospace шрифты.

Reported by thecybershadow on 2012-03-21 05:29:43


Maximus5 commented 9 years ago
Выкинул тормознутый std, скорость загрузки bdf-шрифтов увеличилась вдвое. Исходники
на svn.

На очереди.
Есть подозрение, что нужно выкинуть SetPixel.
Нет смысла грузить (инициализировать) шрифт, если он не используется в настройках.
Т.е. при регистрации - смотрим только параметры шрифта, а уж если он используется -
тогда уж полный цикл.
А потом будет виртуальный underline/bold/italic.

Reported by ConEmu.Maximus5 on 2012-03-21 21:30:34

Maximus5 commented 9 years ago
Вот без setpixel, на базе svn еще не пробовал.

Часть параметров шрифта - если он рамочный, а это без просмотра всего файла не узнать.

Reported by thecybershadow on 2012-03-21 23:20:39


Maximus5 commented 9 years ago
Т.е. не весь, но хотя бы до первого рамочного символа. Что может означать весь.
Можно кэш сделать, если до этого дойдет.

Для скорости в теории можно добавить поддержку формата PCF, он двоичный.

Reported by thecybershadow on 2012-03-21 23:25:22

Maximus5 commented 9 years ago
На счет просмотра всего файла.
Мысль была такая. Даже если его весь просматривать, ну или до первого BITMAP, или (как
вариант) до тех пор, пока не будут найдены (по количеству) все искомые параметры,
НО если НЕ звать при этом CreateBitmap() и пропускать обработку блока BITMAP - должно
быть а) быстрее; б) экономнее по памяти. Это может стать особо актуально, если шрифтов
в папке много.

Reported by ConEmu.Maximus5 on 2012-03-22 05:50:44

Maximus5 commented 9 years ago
Надо посмотреть где время выполнение проводит.

При запуске с холодным дисковым кешем, большинство времени пройдет именно на чтение
файла, а не на анализе содержимого. Думаю следует либо кеш шрифтов делать, либо в документации
указывать, что не следует оставлять неиспользуемые шрифты в папке ConEmu.

Reported by thecybershadow on 2012-03-22 05:54:46

Maximus5 commented 9 years ago
Если шрифт большой - тот же unifont 10M - то основное время было в getline, впрочем
и осталось в цикле обработки. Ну, допустим, на холодной машине - секунда на чтение,
остальные 5 - на сканирование строк.

На счет кеша - это хорошая мысль. Сделаю, и не только для bdf, но и остальных шрифтов
(они тоже ведь сканируются).

Reported by ConEmu.Maximus5 on 2012-03-22 06:00:47

Maximus5 commented 9 years ago
После убиения SetPixel время запуска еще в 2 раза сократилось.

Reported by ConEmu.Maximus5 on 2012-03-22 12:23:00

Maximus5 commented 9 years ago
теперь можно и в "рамках" использовать

Reported by ConEmu.Maximus5 on 2012-04-17 21:05:13

Maximus5 commented 9 years ago

Reported by ConEmu.Maximus5 on 2013-10-23 23:18:20