Currently Inquest is creating (at least) 4 global variables for things that don't need to be in the global namespace.
Core.lua
1: global "Inquest", add "local " prefix
2: global "quixote", add "local " prefix
3: global "LSM", add "local " prefix
77: global "InquestLDB", change to "Inquest.LDB" (or "self.LDB" since it's in a scope where "self" == "Inquest")
Options.lua
1: insert
Code:
local Inquest = LibStub("AceAddon-3.0"):GetAddon("Inquest")
QuestLog.lua
1: insert
Code:
local Inquest = LibStub("AceAddon-3.0"):GetAddon("Inquest")
local quixote = LibStub("LibQuixote-2.0")
local LSM = LibStub("LibSharedMedia-3.0")
Tooltip.lua
1: insert
Code:
local Inquest = LibStub("AceAddon-3.0"):GetAddon("Inquest")
local quixote = LibStub("LibQuixote-2.0")
QuestLog.lua
1: insert
Code:
local Inquest = LibStub("AceAddon-3.0"):GetAddon("Inquest")
local quixote = LibStub("LibQuixote-2.0")
local LSM = LibStub("LibSharedMedia-3.0")
|
»
Unnecessary globals
4
22,611
126
Notes