[FIX] SMSG_ATTACKSTOP

[Lordronn]

Стало интересно, что же это за неизвестные 4 байта. Копая снифф я заметил, что значение колеблется в зависимости от типа таргета(victim - а) Если victim == player - 1, а если unit или pet - 0

--- /src/game/Unit.cpp	Thu Feb 24 12:31:30 2011
+++ /src/game/Unit.cpp	Sun Feb 27 16:36:36 2011
@@ -2855,7 +2855,7 @@
     WorldPacket data( SMSG_ATTACKSTOP, (4+16) );            // we guess size
     data << GetPackGUID();
     data << victim->GetPackGUID();                          // can be 0x00...
-    data << uint32(0);                                      // can be 0x1
+    data << uint32(victim->GetTypeId()==TYPEID_PLAYER ? 1 : 0); // victim is player
     SendMessageToSet(&data, true);
     DETAIL_FILTER_LOG(LOG_FILTER_COMBAT, "%s %u stopped attacking %s %u", (GetTypeId()==TYPEID_PLAYER ? "player" : "creature"), GetGUIDLow(), (victim->GetTypeId()==TYPEID_PLAYER ? "player" : "creature"),victim->GetGUIDLow());

[zergtmn]

Почему 4 байта, а не 1? В некоторых других пакетах bool передается в 1 байте.

[Lordronn]

Также в некоторых пакетах почему-то булл идет в 4 байта. В коде мангоса и тринити отправляются 4 байта, в парсере своем тоже читаю 4 байта и все нормально - не крашит.

[zergtmn]

Предположение не верно.

S->C: SMSG_ATTACKSTOP (0x144), Size: 20, Timestamp: 2010-09-17 07:29:06
-----------------------------------------------------------------------------
GUID: { 0xF13000747010EBCE, Type: Creature, Entry: 29808, Counter: 1108942 }
Victim GUID: { 0xF13000746110E54D, Type: Creature, Entry: 29793, Counter: 1107277 }
Unknown (dword): 1

[Lordronn]

Attacker: GUID (High: Unit Entry: 48029, Counter: 986876)
Target: GUID (High: Player Entry: 0, Counter: 47358415)
Target is player: true

Attacker: GUID (High: Player Entry: 0, Counter: 47358415)
Target: GUID (High: Unit Entry: 48029, Counter: 986876)
Target is player: false

Attacker: GUID (High: Unit Entry: 48029, Counter: 987159)
Target: GUID (High: Player Entry: 0, Counter: 47358415)
Target is player: true

Attacker: GUID (High: Unit Entry: 48267, Counter: 983783)
Target: GUID (High: Player Entry: 0, Counter: 51678088)
Target is player: true

Хотя возможно это просто случайность)

[TOM_RUS]

Судя по коду в клиенте, этот параметр как-то влияет на позицию/ориентацию юнитов относительно друг друга, если ида все правильно декомпилировала, то значения могут быть только 0 или 1.

[SeT]

Если установить 1 то моб будет стараться "смотреть" в сторону цели, даже если цель убита.

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

[virusav]

Может, установка параметра в 1 решит проблему https://github.com/cmangos/issues/issues/590 ?
Там как раз нпц не хватает ориентации на игрока при наложении на него ауры.

[RomanRom2]

раз уж мы некропостим, то у меня вот так:

function msgBuild(var Buf: TBuffer; var m: T_SMSG_ATTACKSTOP): longint; overload;
begin
  pkt.InitCmd      (Buf, SMSG_ATTACKSTOP);
  pkt.AddGUID      (Buf, m.AttackerGUID);
  pkt.AddGUID      (Buf, m.TargetGUID);
  pkt.AddLong      (Buf, m.Following);
  result:= pkt.pktLen;
end;

это от классика 1.12
вообще булеан у близзов мнооооого где передается в виде инта, а мнооооого где в виде байта. это нормально. просто одну фичу делала одна команда программистов (которая не экономит трафик), другую - другая (которая экономит).

[SeT]

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

Может, установка параметра в 1 решит проблему https://github.com/cmangos/issues/issues/590 ?
Там как раз нпц не хватает ориентации на игрока при наложении на него ауры.

Я не помню такой проблемы на TC, там тоже всегда шлется 0.

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

раз уж мы некропостим, то у меня вот так:

function msgBuild(var Buf: TBuffer; var m: T_SMSG_ATTACKSTOP): longint; overload;
begin
  pkt.InitCmd      (Buf, SMSG_ATTACKSTOP);
  pkt.AddGUID      (Buf, m.AttackerGUID);
  pkt.AddGUID      (Buf, m.TargetGUID);
  pkt.AddLong      (Buf, m.Following);
  result:= pkt.pktLen;
end;

это от классика 1.12
вообще булеан у близзов мнооооого где передается в виде инта, а мнооооого где в виде байта. это нормально. просто одну фичу делала одна команда программистов (которая не экономит трафик), другую - другая (которая экономит).

И при каких случаях вы шлете 1? ) Я не могу вспомнить случаев, когда после остановки атаки цели нужно продолжать держать ориентацию на цель. Если еще можно что то придумать у существ в скриптах, то зачем это надо делать для игроков, ума не представляю.

[Amaru]

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

Я не помню такой проблемы на TC, там тоже всегда шлется 0.


И при каких случаях вы шлете 1? ) Я не могу вспомнить случаев, когда после остановки атаки цели нужно продолжать держать ориентацию на цель. Если еще можно что то придумать у существ в скриптах, то зачем это надо делать для игроков, ума не представляю.

Некоторые мобы, когда добивают игрока, некоторое время все еще смотрят на него, а потом продолжают делать свои дела - переключаются на другую цель, или убегают на точку спавна

[RomanRom2]

это скорее всего какие то нетипичные сценарии. в основном конечно посылается ноль. просто был вопрос про "неизвестное" поле following, я рассказал. это действительно и было открыто/выяснено мной лично в свое время: сначала я анализировал сниффы на предмет, какие значения бывают. затем экспериментировал на сандбоксе, подставляя эти значения и смотрел как ведет себя клиент. по другому не умею.

[MaS0n]

Это не нетипичные сценарии, этот параметр называется NowDead. 1 шлется в том случае, если атака была остановлена по причине смерти цели. И да, это отвечает за facing нападавшего относительно виктима. Почему uint32 в классике и до каты? Возможно дело не только в команде программистов, на альфе была другая боевая система, возможно здесь предполагалась причина (reason) остановки атаки и делать какие-то действия нападающим в зависимости от переданного значения. Систему переделали, отказались от фичи, а пакет остался.

[SeT]

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

Это не нетипичные сценарии, этот параметр называется NowDead. 1 шлется в том случае, если атака была остановлена по причине смерти цели. И да, это отвечает за facing нападавшего относительно виктима. Почему uint32 в классике и до каты? Возможно дело не только в команде программистов, на альфе была другая боевая система, возможно здесь предполагалась причина (reason) остановки атаки и делать какие-то действия нападающим в зависимости от переданного значения. Систему переделали, отказались от фичи, а пакет остался.

Так называется параметр у ТС в 6.х. Они взяли это имя с клиента?
Все равно дурацкая система выходит, сначала посылается true, а потом еще надо послать false, чтобы клиент перестал смотреть на цель...

[RomanRom2]

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

[MaS0n]

https://www.ownedcore.com/forums/wor...ml#post3159112

TOM_RUS же упоминал, дебаг клиент MacOS x64. Есть и 4.1.0 тоже где-то же в этой теме

Этот пакет не менялся, поэтому название параметра в нем с тех далеких времен.

PS:
Ну и из альфа-дебаг клиента

void __thiscall CGUnit_C::OnAttackStop(CGUnit_C *this, unsigned __int64 previousTarget, int nowDead)

[RomanRom2]

ок, спасибо.
у себя я еще не дошел, что бы эти нюансы реализовывать. помню что разбирал, а сделать не сделал еще.