Страница 52 из 243
Добавлено: Пн янв 21, 2013 15:50
MVV
Loopback,
Приделай минус для отключения: /EV-. А по умолчанию пусть разворачивает.
Добавлено: Пн янв 21, 2013 15:58
KopBuH
Loopback писал(а):Вообще, сейчас уже очевидно, что идея разворачивать переменные только при явно заданном ключе не самая удачная, поскольку переменные приходится разворачивать в подавляющем большинстве случаев. Нужно было делать обратный ключ.
но если сейчас сделаешь так, то придется еще много чего менять, не только ShellExec.
к тому же всем придется еще также переписывать свои конфиги. если люди не читают справку, то ты тут не виноват
.
Добавлено: Пн янв 21, 2013 16:04
MVV
KopBuH,
1. Конфиги и так переделывать надо (пару примеров я приводил),
2. Вряд ли у кого-то что-то сломается, если вдруг добавится раскрытие переменных окружения по умолчанию - символ процента, тем более более одного раза в строке, в путях встречается крайне редко. А наличие параметра /EV ничего не сломает, если по умолчанию переменные и так раскрываются.
Добавлено: Пн янв 21, 2013 16:40
Loopback
А у меня давно уже крутится мысль сделать директиву, чтобы все переменные в пределах ее действия разворачивались по-умолчанию.
Добавлено: Пн янв 21, 2013 17:11
MVV
Можно и так. Тогда прописать её включение в дефолтный конфиг. Но поддержку параметров /EV и /EV- оставить для определённости конкретных случаев.
Добавлено: Пн янв 21, 2013 18:24
ask-rus
---------------------------
Autorun plugin error
---------------------------
Error in command at line [33]:
Unknown command: Pragma
---------------------------
ОК
---------------------------
...
Загружать Autorun_SysInfo.dll обязательно?
Добавлено: Пн янв 21, 2013 18:33
MVV
ask-rus,
Какая версия-то плагина у тебя (номер беты)? Pragma, SysInfo - это относилось к последней, 12-й.
Если не загрузишь плагин, его переменные будут недоступны.
Добавлено: Пн янв 21, 2013 18:44
ask-rus
Обновился.
Какая-то ошибка:
---------------------------
Autorun plugin error
---------------------------
Error in command at line [18]:
Incorrect syntax in expression: 64 == 32
---------------------------
ОК
---------------------------
Добавлено: Пн янв 21, 2013 18:49
MVV
Ну а теперь читай последние 2-3 страницы, я про это уже писал.
Добавлено: Пн янв 21, 2013 18:53
ask-rus
Новая версия - круто. Оставил только свои рабочие строки и ошибки не стало.
Добавлено спустя 3 минуты:
MVV,
Loopback,
KopBuH
Спасибо.
Добавлено: Пн янв 21, 2013 20:53
Loopback
Пока ехал домой, думал. Нет, все же директива будет не очень хорошо, поскольку тогда придется городить поддержку этого функционала в плагинах, чего совсем не хочется.
Все же идея MVV выглядит более правильной. Можно оставить /EV для совместимости, документировав его как устаревший (deprecated). А ввести обратный флаг (/EV- или /NEV или /NE) для отмены разворачивания. Флаг /EE можно оставить, поскольку эскейпы нужны редко.
А можно /EV и совсем убрать - поскольку новая версия планируется как можорное обновление, и некоторые "breaking changes" допустимы.
Добавлено: Пн янв 21, 2013 21:11
MVV
А ввести обратный флаг (/EV- или /NEV или /NE)
Я предложил лучший вариант -
/EV-. Это очень популярный способ отключения различных функций параметрами командной строки.
поскольку новая версия планируется как можорное обновление
В этом плюс вечных бет.
Добавлено: Пн янв 21, 2013 23:17
Flasher
Loopback писал(а):С этой не так просто. У нее для разных параметров совершенно разный тип возвращаемых данных, напрямую не сделаешь.
Что можешь в таком случае предложить?
Добавлено: Пн янв 21, 2013 23:50
MVV
Flasher,
Единственное, что можно предложить в таком случае - выделять самые необходимые возвращаемые значения и вызывать функцию для них персонально, возвращая нужные поля структур.
Добавлено: Вт янв 22, 2013 00:05
Flasher
MVV
"Выделить" - это выбрать? Сходу так список не накидаю, надо обдумать. Сперва подожду, согласен ли с этим автор.