John Doe wrote: ↑Fri 07 Dec, 2018 08:21
sToLp wrote: ↑Thu 06 Dec, 2018 23:05
local cas = type(m.key)=="string" and m.key:match("%S+$")
Имя переменной cas обозначало набор модификаторов CtrlAltShift, теперь это не так, значит имя стоит изменить.
Да, это я сделаю. Просто пока что не успеваю за обилием Ваших и
IgorZ весьма плодотворных идей. За которые я вам безмерно благодарен!
Тип m.key проверять излишне.
Учту, но почему это так, я, к сожалению, пока что не просёк. По-возможности, поясните, пожалуйста, хотя бы чуть подробнее.
John Doe wrote: ↑Fri 07 Dec, 2018 08:21sToLp wrote: ↑Thu 06 Dec, 2018 23:05
m.description:gsub("&","")
А не лучше ли удалить ненужные амперсанды сразу, в исходнике?
Зачем это делать динамически?
Этого я пока что менять не буду. Сначала какое-то время поупражняюсь на практике. Мне эта возможность нравится на случай если надо будет из этого набора макросов перенимать что-то в другие макросы, в которых может, ведь,
MMacro
и не применяться. А здесь уже как бы готовые примеры. А могут быть и обратные случаи, когда
Macro
будет преобразовываться в
MMacro
и тогда тоже не потребуется лишняя работа. Поэтому данная автоматика мне импонирует.
John Doe wrote: ↑Fri 07 Dec, 2018 08:23
IgorZ wrote: ↑Fri 07 Dec, 2018 06:18
если key отсутствует
key не может отсутствовать.
Если отсутствует, значит и MMacro не надо.
Вот и я сначала примерно так же подумал и убрал проверку. А потом, после справедливого
указания от
IgorZ, перепроверил и убедился, что при отсутствии ключа MMacro действительно падает. Разумеется, Вы правы, что без ключа MMacro не имеет смысла. Однако, на практике этот принцип временно может и позабыться. Поэтому пусть уж лучше проверка останется, на всякий пожарный.