fix gcd mods

core/core.lua

@@ -257,8 +257,7 @@ local function CalcSpell(spellId, calcedSpell, parentSpellData, parentEffCastTim
 
     if parentEffCastTime == nil then
         if stats.gcdMods[spellId] ~= nil then
-            -- TODO: this should probably be / 1000
-            GCD = GCD + stats.gcdMods[spellId].val / 100;
+            GCD = GCD + stats.gcdMods[spellId].val / 1000;
             calcedSpell:AddToBuffList(stats.gcdMods[spellId].buffs);
         end
 
@@ -522,7 +521,6 @@ local function CalcSpell(spellId, calcedSpell, parentSpellData, parentEffCastTim
 
     --------------------------
     -- Handle triggered spell effect
-    -- TODO: for dmg spells use mitigation (NOT RESIST) values from triggering spell in triggered spell calc
 
     do
         local triggerFromSpell = spellId;

core/effectHandlers.lua

@@ -339,7 +339,6 @@ local function ManaShield(calcedSpell, effNum, spellBaseInfo, spellRankInfo, eff
     calcedEffect.perResource = calcedEffect.avgAfterMitigation / calcedSpell.effectiveCost;
 end
 
--- TODO: test with searing totem (2 levels to go!)
 --- Handler for things like Arcane Missiles or Searing Totem.
 --- That is, periodic spells with ticks that behave much like a direct damage effect.
 ---@param calcedSpell CalcedSpell

data/auras.lua

@@ -107,8 +107,6 @@ _addon.buffData = {
         effect = _addon.EFFECT_TYPE.MP5,
         value = 12,
     },
-
-    -- TODO: Other buffs that are outside the scope of base API queries
 };
 
 -- Buffs the target can have (NYI)

data/itemEffects.lua

@@ -80,6 +80,5 @@ _addon.itemEffects = {
             value = 0
         }
     },
-    -- TODO: https://classic.wowhead.com/item=23197/idol-of-the-moon
     -- TODO: https://classic.wowhead.com/item=23004/idol-of-longevity
 }