oUF

99.9k Downloads

magic numbers in runebar element can be removed

Evilpaul opened this issue ยท 2 comments

commented

In patch 4.0.1 Blizzard have added in a constant for the max number of runes (at last), called MAX_RUNES.
http://github.com/tekkub/wow-ui-source/blob/live/FrameXML/RuneFrame.lua#L7

This can be used in the runebar element as the max loop value, rather than the current "magic number" of 6.

commented

local variable is local!

commented

bah, I need to stop reading stuff after a long day at work reading C code with Japanese comments :-/