implement GroupFlagMask - Ru-MaNGOS

rsa · 30.03.2011, 20:58

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

**Ambal** · 30.03.2011, 22:56

Цитата:

Сообщение от rsa

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

Мы решили вас немного расстроить и всеже проявить интерес к этому патчу

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

rsa · 31.03.2011, 15:28

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

Заменить название флагов конечно можно, но думаю не стоит. Имена устоявшиеся и используются в сторонних продуктах. Не люблю резать совместимость если это не требуется.
Тестирование - у меня идет уже неделю, проблем вроде нет. Массово тестирование началось вчера, пока тишина. Да, валгринд молчит как рыба об лед.

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

**Ambal** · 31.03.2011, 15:41

Благодарствуем

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

**Ambal** · 08.04.2011, 10:03

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

rsa · 08.04.2011, 12:25

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

PSZ · 08.04.2011, 19:11

Да

**TOM_RUS** · 16.04.2011, 00:23

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

rsa · 16.04.2011, 13:42

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

Lordronn · 16.04.2011, 14:10

Цитата:

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

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

rsa · 16.04.2011, 16:02

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

**Vladimir** · 16.04.2011, 20:53

Я примерно тоже самое чуствовал при обсуждении патча на irc

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

Йоха · 16.04.2011, 20:53

Цитата:

Сообщение от rsa

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

Как говорится "истина где-то рядом"

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

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

**RomanRom2** · 16.04.2011, 22:51

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

rsa · 16.04.2011, 22:53

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

**TOM_RUS** · 16.04.2011, 23:09

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

30.03.2011, 20:58	#1
rsa Почетный флудер Старожил Регистрация: 08.03.2010 Адрес: Мурманск, Россия Сообщений: 788 Сказал(а) спасибо: 55 Поблагодарили 333 раз(а) в 151 сообщениях Записей в дневнике: 1	implement GroupFlagMask Патч, перерабатывающий работу с маской функций членов райда. https://github.com/rsa/mangos/commit...920e5230814fd0 оно де диффом https://github.com/rsa/mangos/commit...230814fd0.diff насколько я помню, этот кусок кода у меня от чистого ядра практически не отличается, поэтому должно пойти и на чистое. Патч перерабатывает работу с функциями "помощник" "главный танк" "наводчик", которые можно назначить члену райда. Оно и раньше в общем работало но откровенно через ...пу. Теперь заимплеменчены верные комбинации и права. Дропнута куча старинного кода из класса Group (весь код перемещен в 1 функцию), а также хранение (совершенно ненужное) гуидов танка, наводчика и помощника в классе и базе. Теперь вместо них аккуратная битмаска, та самая что шлется клиенту. На интерес тим я уже давно не рассчитываю, так что может кому-нибудь пригодится. Да, добавочно зашита работа с ролями (из LFG/LFD) но без имплемента оных код ничего делать не будет.

08.04.2011, 19:11	#7
PSZ Ученый Регистрация: 07.03.2010 Сообщений: 138 Сказал(а) спасибо: 200 Поблагодарили 143 раз(а) в 49 сообщениях	Да Последний раз редактировалось Ambal; 08.04.2011 в 22:00. Причина: Боцман, приборы. - 50! - Что 50?! - А что приборы (с) мохнатая шутка юмора

16.04.2011, 00:23	#8
TOM_RUS MaNGOS Dev Регистрация: 11.03.2010 Сообщений: 468 Сказал(а) спасибо: 0 Поблагодарили 514 раз(а) в 163 сообщениях	В патче некорректно обрабатываются флаги игрока в группе, почему-то ASSIST, MAIN_ASSIST и MAIN_TANK объявлены уникальными. На самом деле это не так и количество игроков с данными флагами в группе не ограничено. Последний раз редактировалось TOM_RUS; 16.04.2011 в 00:29.

16.04.2011, 20:53	#12
Vladimir MaNGOS Dev Регистрация: 09.02.2010 Сообщений: 594 Сказал(а) спасибо: 315 Поблагодарили 438 раз(а) в 181 сообщениях	Я примерно тоже самое чуствовал при обсуждении патча на irc Так что это не заговор по вашему доведению до белого каления ^^ __________________ Так как устал объяснять знайте ICQ не пользуюсь

31.03.2011, 15:28	#3
rsa Почетный флудер Старожил Регистрация: 08.03.2010 Адрес: Мурманск, Россия Сообщений: 788 Сказал(а) спасибо: 55 Поблагодарили 333 раз(а) в 151 сообщениях Записей в дневнике: 1	Гм... Дон поражен в самую пятку... Таким образом имеем. Патч на чистое ядро (11299): https://gist.github.com/896270 Заодно поправил некоторые опечатки. Оказалось, что в чистом ядре группы еще не имеют ObjectGuid, потому пришлось слегка переделать Заменить название флагов конечно можно, но думаю не стоит. Имена устоявшиеся и используются в сторонних продуктах. Не люблю резать совместимость если это не требуется. Тестирование - у меня идет уже неделю, проблем вроде нет. Массово тестирование началось вчера, пока тишина. Да, валгринд молчит как рыба об лед. PS как же убого в даларане на чистом ядре... ни салютов, ни света, ни охотников со зверюшками... давно уже не запускал...

31.03.2011, 15:41	#4
Ambal MaNGOS Dev Регистрация: 22.06.2010 Сообщений: 78 Сказал(а) спасибо: 24 Поблагодарили 71 раз(а) в 25 сообщениях	Благодарствуем В случае если в течение одной-двух недель проблем и претензий не будет, то патч будет закоммичен в репозиторий.

08.04.2011, 10:03	#5
Ambal MaNGOS Dev Регистрация: 22.06.2010 Сообщений: 78 Сказал(а) спасибо: 24 Поблагодарили 71 раз(а) в 25 сообщениях	Как продвигаются тесты этого патча? Неделя прошла, ни ответа, ни привета

08.04.2011, 12:25	#6
rsa Почетный флудер Старожил Регистрация: 08.03.2010 Адрес: Мурманск, Россия Сообщений: 788 Сказал(а) спасибо: 55 Поблагодарили 333 раз(а) в 151 сообщениях Записей в дневнике: 1	а что тут можно сказать? визуально почти ничего не изменилось, за исключением нормальной работы отображения функций. проблем нет. сейчас не менее десятка серверов (и предположительно около полусотни форков форков) живут с этим патчем и не жужжат. молчание в данном случае говорит что стало как минимум не хуже, ибо про проблемы меня обычно напрягают сразу

16.04.2011, 13:42	#9
rsa Почетный флудер Старожил Регистрация: 08.03.2010 Адрес: Мурманск, Россия Сообщений: 788 Сказал(а) спасибо: 55 Поблагодарили 333 раз(а) в 151 сообщениях Записей в дневнике: 1	Ну я рад, что получил еще одно мнение на эту тему. В действительности у меня есть несколько видео с оффа, в которых при установке этих флагов он с других игроков слетает, и несколько видео в которых он не слетает. Думаю что это зависит от конкретного патча. К сожалению установить истину уже не выйдет, можно сделать и так и так. PS насчет флага ASSIST где-то в районе 3.07 лично рейдом рулил - он был уникальным, и я по этому поводу матюкался.

16.04.2011, 16:02	#11
rsa Почетный флудер Старожил Регистрация: 08.03.2010 Адрес: Мурманск, Россия Сообщений: 788 Сказал(а) спасибо: 55 Поблагодарили 333 раз(а) в 151 сообщениях Записей в дневнике: 1	поправить - дело 30 секунд. только я откровенно за...лся сначала делать по одной информации, потом появляется совершенно противоположная, а потом вторую опять опровергают. примеры даже в этом форуме есть - тред про ЦЛК (кстати до сих пор ораторы не договорились, лупит ребрад танка шипом в героике или нет. мнений ровно пополам. видео есть обоих сортов.), а более других примеров только за крайние пару месяцев у меня штук 6. так что как народ между собой договорится - так и сделаю. по мнению большинства.

16.04.2011, 22:51	#14
RomanRom2 WowCore Dev Регистрация: 31.03.2010 Сообщений: 468 Сказал(а) спасибо: 73 Поблагодарили 106 раз(а) в 70 сообщениях	сделай оба варианта, а "переключалку" вынеси в конфиг

16.04.2011, 22:53	#15
rsa Почетный флудер Старожил Регистрация: 08.03.2010 Адрес: Мурманск, Россия Сообщений: 788 Сказал(а) спасибо: 55 Поблагодарили 333 раз(а) в 151 сообщениях Записей в дневнике: 1	уже собственно... проблема только со снятием флагов - штатного способа у близзов, если принимать флаги неуникальными, нету. только все кучей командой "разжаловать".

16.04.2011, 23:09	#16
TOM_RUS MaNGOS Dev Регистрация: 11.03.2010 Сообщений: 468 Сказал(а) спасибо: 0 Поблагодарили 514 раз(а) в 163 сообщениях	Да не нужно здесь никаких опций, я вообще в первый раз тут услышал что эти флаги оказывается уникальные, кто вообще такое сказал?