Обработка команд внешних Вьюера/Редактора

Нашли баг? Воспользуйтесь поиском чтобы убедиться что его не нашли до вас и если он не упоминается добавьте его описание (желательно пошаговое).
Чем для автора программы отличаются баги, недоработки и фичи.

Модератор: motyara

Ответить
xaizek
Сообщения: 10
Зарегистрирован: Ср фев 24, 2010 4:44 pm

Обработка команд внешних Вьюера/Редактора

Сообщение xaizek »

Версия 0.96 (билд 782)
Это скорее недоработка, а не баг.

Речь об одноимённых полях ввода на странице настроек "Операции->Правка/просмотр".
При вводе в поле значения "gvim" всё работает (путь к нему есть в PATH).
Но если добавить параметр - "gvim --remote-tab-silent", то редактор не открывается.
Если же указать полный путь к программе - "E:\programs\editors\vim\vim72\gvim.exe --remote-tab-silent", то всё работает.

И ещё, при неудачном запуске приложения для просмотра/редактирования всплывает диалог настроек. Хотелось бы увидеть какое-нибудь сообщение. Так как подобное поведение программы не очевидно, и даже не сразу понятно, что она хочет.
Аватара пользователя
Max Diesel
Автор программы
Сообщения: 3431
Зарегистрирован: Пт окт 12, 2007 3:26 pm
Контактная информация:

Сообщение Max Diesel »

Когда файл указан с полным его путем, программе проще разобраться в том какая часть строки является именем файла, а какая параметрами...
Аватара пользователя
Qwertiy
Охотник за багами
Сообщения: 1199
Зарегистрирован: Вс янв 31, 2010 12:12 am

Сообщение Qwertiy »

Max Diesel писал(а):Когда файл указан с полным его путем, программе проще разобраться в том какая часть строки является именем файла, а какая параметрами...
Может быть стоит добавить поле "Параметры запуска", как это сделано для панели инструментов?
Аватара пользователя
Max Diesel
Автор программы
Сообщения: 3431
Зарегистрирован: Пт окт 12, 2007 3:26 pm
Контактная информация:

Сообщение Max Diesel »

Qwertiy писал(а):Может быть стоит добавить поле "Параметры запуска", как это сделано для панели инструментов?
Обычно вьюеры/редакторы не требуют для запуска указания дополнительных ключей, делать отдельные поля "Параметры запуска" для них несколько неоправданно. Зато поле с именем файла вьюера/редактора имеет с правой стороны кнопку указания файла программы (то есть вписывание непосредственно имени программы рассматривается как некий экстремальный вид ее указания). В планах есть модификация вкладки вьюера/редактора, возможно будут добавлены и параметры запуска.
xaizek
Сообщения: 10
Зарегистрирован: Ср фев 24, 2010 4:44 pm

Re:

Сообщение xaizek »

Max Diesel писал(а):Когда файл указан с полным его путем, программе проще разобраться в том какая часть строки является именем файла, а какая параметрами...
Ясно. В принципе тут нету никаких неудобств, просто не ожидал.
Кстати, можно не вводить дополнительное поле, а анализировать строку следующим образом:
Для путей с пробелами требовать заключение их в двойные кавычки ("). Далее выделить в строке её первую часть, либо по первому пробелу, либо по второму символу кавычек (если строка с него и начиналась), так как в пути он встречаться не может. Первая часть и есть имя файла, всё остальное - параметры.
Аватара пользователя
Qwertiy
Охотник за багами
Сообщения: 1199
Зарегистрирован: Вс янв 31, 2010 12:12 am

Сообщение Qwertiy »

xaizek писал(а):Кстати, можно не вводить дополнительное поле, а анализировать строку следующим образом:
Для путей с пробелами требовать заключение их в двойные кавычки ("). Далее выделить в строке её первую часть, либо по первому пробелу, либо по второму символу кавычек (если строка с него и начиналась), так как в пути он встречаться не может. Первая часть и есть имя файла, всё остальное - параметры.
На мой взгляд, поле лучше, хотя бы ради единства интерфейса с настройкой кнопок панели инструментов.
xaizek
Сообщения: 10
Зарегистрирован: Ср фев 24, 2010 4:44 pm

Re:

Сообщение xaizek »

Qwertiy писал(а): На мой взгляд, поле лучше, хотя бы ради единства интерфейса с настройкой кнопок панели инструментов.
Насчёт единства интерфейса согласен, да и с отдельным полем наверное проще обращаться чем помнить формат для ввода значения.
Ответить