Как формироуется список АМ в macro-плагине?

[DInvalid]

4epT


Этот код, который ты написал, случайно не из макро.пл?
Так это просто вывод на экран списка ... как от него что то зависеть может?

Код: Выделить всё

	### parameter: list
	if ($arg eq 'list') {
		message(sprintf("The following macros are available:\n%smacros%s\n","-"x10,"-"x9), "list");
		foreach my $m (keys %macro) {message "$m\n" unless $m =~ /^tempMacro/}
		message(sprintf("%sautomacros%s\n", "-"x8, "-"x7), "list");
		foreach my $a (sort {
			($automacro{$a}->{priority} or 0) <=> ($automacro{$b}->{priority} or 0)
		} keys %automacro) {message "$a\n"}
		message(sprintf("%s\n","-"x25), "list");

И еще... как надо что бы они сортировались?
По порядку написания в файле?
Или по имени?
Или еще как то?

И вообще зачем это надо?
Если надо что бы они выполнялись в определенном порядке значит ставь приоритет.
Все что без приоритета - значит не важно как выполняется, нэ?

Переделать можно, но нужно ли?

[Aib0]

Т.е. он читает макры, хэширует их и работает со списком хэшей?
Бреднятина, имхо

[DInvalid]

Т.е. он читает макры, хэширует их и работает со списком хэшей? Бреднятина, имхо

Имхо, бредятина то что тут написано.
В перле хэш - это таблица, к полям которой обращаются по ключам.
И это самый нормальный способо работать с изменяемым списком различных данных. Но при просмотре хэша его ключи возвращаются не в порядке занесения а в его внутреннем порядке...

Предложите свой способ хранения если этот не нравится!

2 4epT Смотри вложение.

И еще, мне не нравится что:

priority <num>
Choose which automacros should be checked before others. The smaller <num> is is the sooner the automacro gets checked. If priority is not given, the priority is assumed to be 0 (zero: check first).

Я бы поменял порядок приоритетов... т.е. если priority больше то вызывать раньше... и тогда 0 по умолчанию был бы последним...

[4epT]

Не сомневался, что у тебя получится!!
Это именно то, что я хотел

Код: Выделить всё

PR:[]	N:[0]	 name - a2
PR:[]	N:[1]	 name - a3
PR:[]	N:[2]	 name - a4
PR:[]	N:[3]	 name - a5
PR:[]	N:[4]	 name - a6
PR:[]	N:[5]	 name - a7
PR:[]	N:[6]	 name - a8
PR:[]	N:[7]	 name - a9

PR-приоритет N-номер АМ (в файле macros.txt)

Это логично что сверху мы пишем АМ, которые мы хотим чтобы они запустились первыми. Единственное что следует добавить это отображение PR:[0], вместо PR:[].

А вот если ли какая-то надобность в номере АМ?

Я думаю, что с тиким раскладом условие priority можно вообще удалить с макро-плагина. Ибо оно у нас уже интуитивное (как и в конфиге)

pJ, давай за СВНим?

[DInvalid]

Это логично что сверху мы пишем АМ, которые мы хотим чтобы они запустились первыми. Единственное что следует добавить это отображение PR:[0], вместо PR:[].
А вот если ли какая-то надобность в номере АМ?

Т.е. смотри - если мы не указываем приорити то оно = 0 и эти автомакросы имеют приоритет над остальными.
Если укажем 1 то они будут выполнятся позже других, 2 - еще позже.
На мой взгляд это не интуитивно (?).

Насчет номера. Этот номер присваивается в процессе разбора файла макросов и он как раз нужен для сортировки по нему списка.
По другому не запомнить последовательность. А вывод его можно и убрать.

[piroJOKE]

pJ, давай за-СВН-им это?

Я не против. Но это требует как минимум переделки документации. Люди УЖЕ используют существующую схему. Пиши автору макро плагино - Анарчо, консультируйся с ним, плз.

[4epT]

эм.. переделки документации в чом?
нигде не указано как сортируются АМ

или ты про удаление priority??

DInvalid, помоему приоритет как раз так и ставится.

1 значит запустится в первую очередь, 2 - во вторую.

[piroJOKE]

Флуд:

4epT, а как ты тогда растолкуешь словосочетание "высокий приоритет"?

[4epT]

ФЛУД:

"высокий", значит "наиболее важный", важнее даже чем тот что на первом месте, а что у нас важнее 1? Канешна 0!! Если мы умножим на 0 любое число, то 0 его поглатит!!

Да здравствует НУЛЬ!!!

[piroJOKE]

Фигня какая-то. Короче, тебя можно понимать так, что ты уже всем доволен, и никаких доработок тебе как-бы нинада?

ЗЫ: А отрицательный приоритет допустим?

[4epT]

Единственное что следует добавить это отображение PR:[0], вместо PR:[].

ну и вывод думаю вместо:

Код: Выделить всё

PR:[]   N:[0]    name - a2 
PR:[]   N:[1]    name - a3 
PR:[]   N:[2]    name - a4 
PR:[]   N:[3]    name - a5 
PR:[]   N:[4]    name - a6 
PR:[]   N:[5]    name - a7 
PR:[]   N:[6]    name - a8 
PR:[]   N:[7]    name - a9

сделать в принципе как было или:

Код: Выделить всё

PR	name
0	a2 
0	a3 
2	a4 
34	a5 

ну типо того