[patch/dev] комбопойнты для всех ;)

[rsa]

Надеюсь, не сообщу ничего нового когда сообщу что комбопойнты могут иметь не только игроки но и некоторые типы крич, в частности вехиклы и петы. Перебрав кучу патчей на эту тему, не нашел ничего стоящего (в т.ч. и в ТС). написал собственный, прсто перенеся работу с комбопойнтами в класс Unit и сделав обработку SMSG_PET_UPDATE_COMBO_POINTS.
https://github.com/rsa/mangos/commit...1cf3bc1bba7813
Тестирование (проведенное на драконах из Ока вечности) показало что вроде бы все работает. У воров тоже вроде бы ничего не поломалось.
Теперь вопросы - мой склероз напоминает, что какие-то петы имели комбо-удары. Но какие - я вспомнить не смог и проверить соответственно тоже. Никто не напомнит?
И еще, кто бы смог проверить патч потщательнее? У меня к сожалению очень мало времени что-то крайнее время...
PS всех с НГ!

[Lightguard]

Неужели все это действительно необходимо? Я не думаю, что Существуют нормальные петов использованием комбопойнты, только вехиклы.
Taк:

PHP код:

        WorldPacket data(GetVehicle() ? SMSG_PET_UPDATE_COMBO_POINTS : SMSG_UPDATE_COMBO_POINTS);
        if(!GetVehicle())
        {
            data << combotarget->GetPackGUID();
            data << uint8(m_comboPoints);
        }
        else
        {
            data << GetVehicleBase()->GetPackGUID();
            data << combotarget->GetPackGUID();
            data << uint8(m_comboPoints);
        } 


[rsa]

in second case your always be send zero.

[Lightguard]

I don't really understand that.

[Vladimir]

rsa, если добавлять combpoints владельцу изначально то как раз и будет работать без переносов там где они не нужны.

[rsa]

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

rsa, если добавлять combpoints владельцу изначально то как раз и будет работать без переносов там где они не нужны.

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

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