implement GroupFlagMask

[rsa]

Патч, перерабатывающий работу с маской функций членов райда.
https://github.com/rsa/mangos/commit/a5e57729fc5211bb6a2fc9ab6b920e5230814fd0
оно де диффом
https://github.com/rsa/mangos/commit/a5e57729fc5211bb6a2fc9ab6b920e5230814fd0.diff
насколько я помню, этот кусок кода у меня от чистого ядра практически не отличается, поэтому должно пойти и на чистое.
Патч перерабатывает работу с функциями "помощник" "главный танк" "наводчик", которые можно назначить члену райда. Оно и раньше в общем работало но откровенно через ...пу. Теперь заимплеменчены верные комбинации и права.
Дропнута куча старинного кода из класса Group (весь код перемещен в 1 функцию), а также хранение (совершенно ненужное) гуидов танка, наводчика и помощника в классе и базе. Теперь вместо них аккуратная битмаска, та самая что шлется клиенту.
На интерес тим я уже давно не рассчитываю, так что может кому-нибудь пригодится.
Да, добавочно зашита работа с ролями (из LFG/LFD) но без имплемента оных код ничего делать не будет.

[Ambal]

На интерес тим я уже давно не рассчитываю

Мы решили вас немного расстроить и всеже проявить интерес к этому патчу :yes3: С вас портирование под чистое ядро + может имеет смысл заменить названия флагов на нечто типа (GROUP_)MEMBER_ASSISTANT для пущей читабельности?
Со всех остальных - тестирование патча и отчет по проделанной работе :)

[rsa]

Гм... Дон поражен в самую пятку...
Таким образом имеем.
Патч на чистое ядро (11299):
https://gist.github.com/896270
Заодно поправил некоторые опечатки. Оказалось, что в чистом ядре группы еще не имеют ObjectGuid, потому пришлось слегка переделать :)
Заменить название флагов конечно можно, но думаю не стоит. Имена устоявшиеся и используются в сторонних продуктах. Не люблю резать совместимость если это не требуется.
Тестирование - у меня идет уже неделю, проблем вроде нет. Массово тестирование началось вчера, пока тишина. Да, валгринд молчит как рыба об лед.

PS как же убого в даларане на чистом ядре... ни салютов, ни света, ни охотников со зверюшками... давно уже не запускал...

[Ambal]

Благодарствуем :) В случае если в течение одной-двух недель проблем и претензий не будет, то патч будет закоммичен в репозиторий.

[Ambal]

Как продвигаются тесты этого патча? Неделя прошла, ни ответа, ни привета :)

[rsa]

а что тут можно сказать? визуально почти ничего не изменилось, за исключением нормальной работы отображения функций. проблем нет. сейчас не менее десятка серверов (и предположительно около полусотни форков форков) живут с этим патчем и не жужжат. молчание в данном случае говорит что стало как минимум не хуже, ибо про проблемы меня обычно напрягают сразу :)

[PSZ]

Да :)

[TOM_RUS]

В патче некорректно обрабатываются флаги игрока в группе, почему-то ASSIST, MAIN_ASSIST и MAIN_TANK объявлены уникальными. На самом деле это не так и количество игроков с данными флагами в группе не ограничено.

[rsa]

Ну я рад, что получил еще одно мнение на эту тему. В действительности у меня есть несколько видео с оффа, в которых при установке этих флагов он с других игроков слетает, и несколько видео в которых он не слетает. Думаю что это зависит от конкретного патча. К сожалению установить истину уже не выйдет, можно сделать и так и так.
PS насчет флага ASSIST где-то в районе 3.07 лично рейдом рулил - он был уникальным, и я по этому поводу матюкался.

[Lordronn]

PS насчет флага ASSIST где-то в районе 3.07 лично рейдом рулил - он был уникальным, и я по этому поводу матюкался.

3.3.5 - собирал рейд в ЦЛК 10. Раздал 3 ассиста. Главные танк и дд - не уникальны. Главным ДД маркируют всех хилов. Танков тоже в рейде несколько может быть

[rsa]

поправить - дело 30 секунд. только я откровенно за...лся сначала делать по одной информации, потом появляется совершенно противоположная, а потом вторую опять опровергают. примеры даже в этом форуме есть - тред про ЦЛК (кстати до сих пор ораторы не договорились, лупит ребрад танка шипом в героике или нет. мнений ровно пополам. видео есть обоих сортов.), а более других примеров только за крайние пару месяцев у меня штук 6.
так что как народ между собой договорится - так и сделаю. по мнению большинства.

[Vladimir]

Я примерно тоже самое чуствовал при обсуждении патча на irc ;) Так что это не заговор по вашему доведению до белого каления ^^

[Йоха]

поправить - дело 30 секунд. только я откровенно за...лся сначала делать по одной информации, потом появляется совершенно противоположная, а потом вторую опять опровергают.

Как говорится "истина где-то рядом" =)))
Дело в том что близы часто нерфят/апают босов и их абилки, а так же многие вещи в игре, поэтому сделать какую-то "правильную" реализацию можно только ориентируясь на конкретный билд.
Мне, как опытному рейдеру со стажем, эта ситуация очень знакома.

Так что не парься, делай как нравится, кому надо исправит под тот вариант который ему больше по душе.

[RomanRom2]

сделай оба варианта, а "переключалку" вынеси в конфиг :)

[rsa]

уже собственно... проблема только со снятием флагов - штатного способа у близзов, если принимать флаги неуникальными, нету. только все кучей командой "разжаловать".

[TOM_RUS]

Да не нужно здесь никаких опций, я вообще в первый раз тут услышал что эти флаги оказывается уникальные, кто вообще такое сказал?