Skill packet differences (Behead)

[Terotrous]

For Behead, and maybe other skills, Zenmya reveals that the official sends skillhitinfo along with the ZC_Skill_Melee_Ground packet, though in our implementations we typically send no hitinfo with it and instead send it later in a ZC_Skill_Hit_Info packet. ZC_Skill_Melee_Ground also has the effect of playing the skill's animation on the client side.

In Behead's specific case, it has 2 hitboxes, the first of which uses the ZC_Skill_Melee_Ground packet, and the second creates a ZC_Skill_Hit_Info packet. Currently it sends two ZC_Skill_Hit_Info packets, which is visually indistinguishable in gameplay, but may indicate that our implementation of the skill packets could be wrong / innaccurate.

[exectails]

Good call creating an issue for it. Generally speaking, we should stick to the way officials handle communication, because we can never be sure whether we might be missing something. Even if everything seems to be in order visually, there could be hidden side effects, or officials could change how a packet works, which might lead to glitches in our alternative implementation that are difficult to fix. If you or anyone else got a packet log, we should look into updating this skill handler.