[Classic] Untested warrior autoshoot concept

[scimmy]

Ran into

Spell_C_CancelAutoRepeat_7FF732813060

and saw that it references

autoRepeatingSpell_7FF734377814

Somewhere in the middle of this function it calls

CGUnit_C::SetCastingAutoRepeat_7FF732965160(v8, 0);

followed by

CGUnit_C::CancelRangedMode(v9);

And from the macos dump:

int __cdecl CGUnit_C::CancelRangedMode(CGUnit_C *this)
{
  if ( !s_trackingType )
    CGUnit_C::ClearTrackingTarget(this, 0, 1);
  return CUnitDisplay::CancelBowPull((CGUnit_C *)((char *)this + 2360));
}

Which is pretty obvious that this is a hunter autorepeating thing.

So I'm wondering if I were to write the spell id of "shoot bow" or "shoot gun" to autoRepeatingSpell and then called SetCastingAutoRepeat with "true", would this end up being a warrior autoshoot hack?

[ChrisIsMe]

Why would you not just go internal and call CastSpell when you still have to put in 95% of the work to determine what spell to cast anyways.

[scimmy]

Why would you not just go internal and call CastSpell when you still have to put in 95% of the work to determine what spell to cast anyways.

Uhhhh I am internal? I never said this is anything super useful, just a thought I had when reversing the function at the time so I posted it

[ChrisIsMe]

Uhhhh I am internal? I never said this is anything super useful, just a thought I had when reversing the function at the time so I posted it

Fair, a lot of people try to do dumb stuff externally thinking they're being safer.