• Listen to a special audio message from Bill Roper to the Hive Workshop community (Bill is a former Vice President of Blizzard Entertainment, Producer, Designer, Musician, Voice Actor) 🔗Click here to hear his message!
  • Read Evilhog's interview with Gregory Alper, the original composer of the music for WarCraft: Orcs & Humans 🔗Click here to read the full interview.
  • The Hive's 22nd Icon Contest: Creep Abilities is now concluded, time to vote for your favourite set of icons! Click here to vote!
  • ✅ The POLL for Hive's Texturing Contest #34 is OPEN! Vote for the TOP 3 SKINS! 🔗Click here to cast your vote!
  • ✅ The POLL for Hive's Techtree Contest #20 is OPEN! Vote for the TOP 3 FACTIONS! 🔗Click here to cast your vote!

Assassinate v1.1

Status
Not open for further replies.
Problem 1.
Removal of non-existent group, severity unclear. Only remove a group if you actually make one as at best it will waste CPU resources and at worst could result in unknown and unwanted behaviour.

Problem 2.
Incorrect structure move logic, severity critical. All members of the object structure must be moved during an object move for the object move to be performed correctly. You forgot to move the slash count member resulting in corruption of slash count member which will cause incorrect and hard to predict behaviour.

Problem 3.
Incorrect structure move logic, severity critical. All members of the object structure must be moved during an object move for the object move to be performed correctly. A code error resulted in assignation target destination being assigned itself rather than the value of object being moved. This will result in incorrect and had to predict behaviour.

A brief test of the ability with 3 heroes owned by the same player casting at the same time demonstrates expected behaviour with no noticeable faults or errors occurring.

Here is your map. I accept both Master Card and PayPal.
(no I am not expecting a payment for this, that is a joke that should not be taken seriously)
 

Attachments

Status
Not open for further replies.
Back
Top