Эмулятор терминала на qt4, widget Опции

[void*]

немного покопался в коде. Действительно обильные комментарии затрудняют чтение кода, да и стиль программирования местами не очень понравился: постоянные приведения в стиле Си, использование void* (что весьма нежелательно, да и по-моему не аргументированно). Ну а так, в принципе неплохо, но еще очень огорчило отсутствие полосы прокрутки

[un-defined]

немного покопался в коде. Действительно обильные комментарии затрудняют чтение кода, да и стиль программирования местами не очень понравился: постоянные приведения в стиле Си, использование void* (что весьма нежелательно, да и по-моему не аргументированно). Ну а так, в принципе неплохо, но еще очень огорчило отсутствие полосы прокрутки

За исходный стиль винить разработчиков konsole в KDE4 Мне надо было для начала добиться работоспособности, а рефакторингом займусь потом, если время будет.
Если есть предложения по улучшению - патчи принимаются.

Комментарии мне лично не мешают.

Полоса прокрутки отключена сознательно - мне так внешний вид больше нравится. Ctrl+PgUP,PgDn еще никто не отменял, а размером history можно управлять. В принципе никто не мешает включить, ну или опцию добавить.. может и сделаю, если уж прямо очень убедительно надо (хотя я, опять повторю, такой необходимости не вижу)

Вообще просторов для улучшения там море, никому не возбраняется поучаствовать.

[void*]

В принципе никто не мешает включить, ну или опцию добавить.. может и сделаю, если уж прямо очень убедительно надо

было бы очень даже неплохо

[un-defined]

было бы очень даже неплохо

Да не вопрос
В cvs забирать.


Кстати, по поводу использования void* - что именно там непонятно и нежелательно?

Сообщение отредактировал un-defined - 16.7.2008, 0:02

[void*]

теперь другое дело правда теперь почему-то стал слишком большой шрифт и отступ от текущего пути, вот скрин

насчет void* - лично я не совсем понимаю зачем в классе держать указатели на void и при каждом их использовании приводить их к нужному типу:

void *_session;
void *_terminalDisplay;

примеры:

((TerminalDisplay*)_terminalDisplay)->setSize(80, 40);

((Session*)_session)->addView((TerminalDisplay*)_terminalDisplay);

[un-defined]

теперь другое дело правда теперь почему-то стал слишком большой шрифт и отступ от текущего пути, вот скрин

Шрифт моноширинный должен быть.
По умолчанию стоит Monospace, 10.
Я ставил еще terminus, 12
С другими шрифтами могут быть как раз такие проблемы, так как положение курсора вычисляется как (размер символа) Х (количество символов). А в немоноширинных шрифтах размер символа непостоянен..

Но вообще я в этот раз не трогал ту часть, которая за шрифты отвечает

насчет void* - лично я не совсем понимаю зачем в классе держать указатели на void и при каждом их использовании приводить их к нужному типу:

void *_session;
void *_terminalDisplay;

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

[void*]

Шрифт моноширинный должен быть.
По умолчанию стоит Monospace, 10.
Я ставил еще terminus, 12
С другими шрифтами могут быть как раз такие проблемы, так как положение курсора вычисляется как (размер символа) Х (количество символов). А в немоноширинных шрифтах размер символа непостоянен..

в принципе да, с Monospace все вроде нормально

ЗЫ я конечно наверно уже совсем обнаглел, но может сделаешь еще и горизонтальную полосу прокрутки или вместо обоих полос прокрутки враппер из QAbstractScrollArea?

Сообщение отредактировал void* - 16.7.2008, 10:17

[Litkevich Yuriy]

вместо обоих полос прокрутки враппер

это плохая идея, должна быть прокрутка

[un-defined]

А зачем нужна горизонтальная прокрутка?

[Litkevich Yuriy]

строки могут быть длинные, а перенос не информативен, т.к. нарушает реальный вывод, пример, сборка Qt.