[1.0] Gauntlets reduce energy for melee/shooting speed

Where bugs go to lie down and rest

Moderator: Moderator

Post Reply
Message
Author
jenx
Sher'Tul Godslayer
Posts: 2263
Joined: Mon Feb 14, 2011 11:16 pm

[1.0] Gauntlets reduce energy for melee/shooting speed

#1 Post by jenx »

Using lukep's addon, I've been studying speed.

If you attack with a weapon or shooter, wearing gloves, your attack speed is as expected. But if you wear gauntlets, the energy used per attack increases by 10%.

So a regular melee attack takes 1100 energy with gauntlets, but 1000 with gloves.

My archer with Rapid Shot at 5/5 takes 667 energy with gauntlets, but 606 with gloves.

Thus I'll be wearing gloves from now on.

Is this a bug or an undocumented feature?
MADNESS rocks

lukep
Sher'Tul Godslayer
Posts: 1712
Joined: Mon Mar 14, 2011 10:32 am
Location: Canada

Re: [1.0] Gauntlets reduce energy for melee/shooting speed

#2 Post by lukep »

That's a bug with my addon, unless I'm missing something big in my code dive.

I'm calling self:combatSpeed(), which is returning self.combat as the weapon by default. If there was a simple way of getting the weapon, I'd use that instead.

Combat.lua:

Code: Select all

1048 --- Gets the weapon speed
1049 function _M:combatSpeed(weapon)
1050         weapon = weapon or self.combat or {}
1051         return (weapon.physspeed or 1) / self.combat_physspeed
1052 end
An easy way to check this would be to compare how much time/energy is actually used, instead of going off of what it calculates.
Some of my tools for helping make talents:
Melee Talent Creator
Annotated Talent Code (incomplete)

jenx
Sher'Tul Godslayer
Posts: 2263
Joined: Mon Feb 14, 2011 11:16 pm

Re: [1.0] Gauntlets reduce energy for melee/shooting speed

#3 Post by jenx »

I think it is an addon error.

Could you use a routine like this to get weapon info, from Combat.lua:

Code: Select all

-- Check if actor is unarmed
function _M:isUnarmed()
	local unarmed = true
	if not self:getInven("MAINHAND") or not self:getInven("OFFHAND") then return end
	local weapon = self:getInven("MAINHAND")[1]
	local offweapon = self:getInven("OFFHAND")[1]
	if weapon or offweapon then
		unarmed = false
	end
	return unarmed
end
MADNESS rocks

Post Reply