Name skill setter, missile, aura and player mode parameters
Ghidra v1805 · 8293ab3
SKILL_SetParam1 to 4 take the D2SkillStrc fields they write. CreatingAMissile and SKILLS_CreateMissileFromUnit take names from the D2SkillArgStrc fields they fill: the offsets are added to the owner's position, and the target pair goes to sSkillArg.nUnitX and nUnitY. D2SetAurastatsFromSkills reaches D2GetSkillCalc's nSkillLvl. The two PLRMODES functions skip TryCancelModeAndMoveToLocation when the flag is nonzero. SKILLNEC_ReviveCorpseAsMinion adds its parameter to 3/4 of the caster level. Not proven: bConsumeAmmo beyond its gate on RemoveQuanityOnUse, and bSkipCancelAttempt is the observed effect only. Chosen names that look wrong: nX and nY on PLRMODES_TryCancelModeAndMoveToLocation 0057eec0, whose callers pass the mode index and the constant 1 there.
Changes 19, applied together
Rename variable
win 0x00644560 Stack[0x8]namenParam1→nPar1Rename variable
win 0x006445a0 Stack[0x8]namenParam2→nPar2Rename variable
win 0x006445e0 Stack[0x8]namenParam3→nPar3Rename variable
win 0x00644620 Stack[0x8]namenParam4→nPar4Rename variable
win 0x0056ecb0 Stack[0x10]namenParam1→nOffsetXRename variable
win 0x0056ecb0 Stack[0x14]namenParam2→nOffsetYRename variable
win 0x0056ecb0 Stack[0x18]namenParam3→nTargetXRename variable
win 0x0056ecb0 Stack[0x1c]namenParam4→nTargetYRename variable
win 0x0056ecb0 Stack[0x20]namenParam5→bConsumeAmmoRename variable
win 0x0056ee90 Stack[0xc]namenParam1→nSkillLvlRename variable
win 0x0056ee90 Stack[0x10]namenParam2→nOffsetXRename variable
win 0x0056ee90 Stack[0x14]namenParam3→nOffsetYRename variable
win 0x0056ee90 Stack[0x18]namenParam4→nTargetXRename variable
win 0x0056ee90 Stack[0x1c]namenParam5→nTargetYRename variable
win 0x0056ee90 Stack[0x20]namenParam6→bConsumeAmmoRename variable
win 0x005c6cc0 Stack[0xc]namenParam→nSkillLvlRename variable
win 0x005809d0 Stack[0x14]namenParam→bSkipCancelAttemptRename variable
win 0x00580a70 Stack[0x14]namenParam→bSkipCancelAttemptRename variable
win 0x005c49e0 Stack[0xc]namenParam2→nLevelOffset