Fixed many unused initializations (fixes #75)

This commit is contained in:
Timo Smit 2017-02-21 10:33:10 +01:00
parent 3ed215ee44
commit 6c95703678
31 changed files with 43 additions and 65 deletions

View File

@ -15,18 +15,16 @@
-- You should have received a copy of the GNU General Public License
-- along with this program. If not, see <http://www.gnu.org/licenses/>.
local constants = require (wolfa_getLuaPath()..".util.constants")
local util = require (wolfa_getLuaPath()..".util.util")
local bits = require (wolfa_getLuaPath()..".util.bits")
local tables = require (wolfa_getLuaPath()..".util.tables")
local events = require (wolfa_getLuaPath()..".util.events")
local timers = require (wolfa_getLuaPath()..".util.timers")
local settings = require (wolfa_getLuaPath()..".util.settings")
local admin = require (wolfa_getLuaPath()..".admin.admin")
local teams = require (wolfa_getLuaPath()..".game.teams")
local constants = require (wolfa_getLuaPath()..".util.constants")
local bits = require (wolfa_getLuaPath()..".util.bits")
local events = require (wolfa_getLuaPath()..".util.events")
local settings = require (wolfa_getLuaPath()..".util.settings")
local tables = require (wolfa_getLuaPath()..".util.tables")
local timers = require (wolfa_getLuaPath()..".util.timers")
local util = require (wolfa_getLuaPath()..".util.util")
local balancer = {}
balancer.BALANCE_RANDOM = 0
@ -40,7 +38,6 @@ local lastJoined = {[constants.TEAM_AXIS] = {}, [constants.TEAM_ALLIES] = {}, [c
local evenerCount = 0
function balancer.balance(byAdmin, forceBalance)
local teamsData = teams.get()
local teamsDifference = teams.difference()
if teamsDifference == 0 then
@ -78,6 +75,8 @@ function balancer.balance(byAdmin, forceBalance)
evenerCount = evenerCount + 1
if forceBalance or evenerCount >= 3 then
local teamsData = teams.get()
for i = 1, math.floor(teamsDifference / 2) do
local player = balancer.findPlayer(teamsData[teamGreater], teamGreater, teamSmaller)

View File

@ -19,9 +19,6 @@ local db = require (wolfa_getLuaPath()..".db.db")
local players = require (wolfa_getLuaPath()..".players.players")
local events = require (wolfa_getLuaPath()..".util.events")
local settings = require (wolfa_getLuaPath()..".util.settings")
local bans = {}
function bans.get(banId)

View File

@ -19,9 +19,6 @@ local db = require (wolfa_getLuaPath()..".db.db")
local players = require (wolfa_getLuaPath()..".players.players")
local events = require (wolfa_getLuaPath()..".util.events")
local settings = require (wolfa_getLuaPath()..".util.settings")
local history = {}
function history.get(clientId, historyId)

View File

@ -42,7 +42,7 @@ function rules.load()
if amount == 0 then return 0 end
for id, rule in ipairs(array["rule"]) do
for _, rule in ipairs(array["rule"]) do
data[rule["shortcut"]] = rule["rule"]
end

View File

@ -16,7 +16,6 @@
-- along with this program. If not, see <http://www.gnu.org/licenses/>.
local events = require (wolfa_getLuaPath()..".util.events")
local files = require (wolfa_getLuaPath()..".util.files")
local settings = require (wolfa_getLuaPath()..".util.settings")
local auth = {}

View File

@ -15,9 +15,6 @@
-- You should have received a copy of the GNU General Public License
-- along with this program. If not, see <http://www.gnu.org/licenses/>.
local events = require (wolfa_getLuaPath()..".util.events")
local files = require (wolfa_getLuaPath()..".util.files")
local auth = require (wolfa_getLuaPath()..".auth.auth")
local shrubbot = {}

View File

@ -47,14 +47,16 @@ function commandBan(clientId, command, victim, ...)
end
local args = {...}
local duration, reason = 600, "banned by admin"
local duration, reason
if args[1] and util.getTimeFromString(args[1]) and args[2] then
duration = util.getTimeFromString(args[1])
reason = table.concat(args, " ", 2)
elseif args[1] and util.getTimeFromString(args[1]) then
duration = util.getTimeFromString(args[1])
reason = "banned by admin"
elseif args[1] then
duration = 600
reason = table.concat(args, " ")
elseif not auth.isPlayerAllowed(clientId, "8") then
et.trap_SendConsoleCommand(et.EXEC_APPEND, "csay "..clientId.." \"^dban usage: "..commands.getadmin("ban")["syntax"].."\";")

View File

@ -19,7 +19,7 @@ local commands = require (wolfa_getLuaPath()..".commands.commands")
local auth = require (wolfa_getLuaPath()..".auth.auth")
local voting = require (wolfa_getLuaPath()..".game.voting")
function commandEnableVote(clientId, command, ...)
function commandEnableVote(clientId, command)
et.trap_SendConsoleCommand(et.EXEC_APPEND, "cchat -1 \"^denablevote: ^9next map voting has been enabled.\";")
voting.force("nextmap")

View File

@ -22,7 +22,6 @@ local commands = require (wolfa_getLuaPath()..".commands.commands")
local players = require (wolfa_getLuaPath()..".players.players")
local settings = require (wolfa_getLuaPath()..".util.settings")
local util = require (wolfa_getLuaPath()..".util.util")
function commandFinger(clientId, command, victim)
if victim == nil then

View File

@ -15,10 +15,10 @@
-- You should have received a copy of the GNU General Public License
-- along with this program. If not, see <http://www.gnu.org/licenses/>.
local util = require (wolfa_getLuaPath()..".util.util")
local auth = require (wolfa_getLuaPath()..".auth.auth")
local commands = require (wolfa_getLuaPath()..".commands.commands")
local settings = require (wolfa_getLuaPath()..".util.settings")
local greetings = require (wolfa_getLuaPath()..".players.greetings")
function commandGreeting(clientId, command)

View File

@ -23,7 +23,6 @@ local players = require (wolfa_getLuaPath()..".players.players")
function commandIncognito(clientId, command)
local fileName = et.trap_Cvar_Get("g_shrubbot")
local functionStart = et.trap_Milliseconds()
local fileDescriptor, fileLength = et.trap_FS_FOpenFile(fileName, et.FS_READ)
if fileLength == -1 then
@ -36,7 +35,7 @@ function commandIncognito(clientId, command)
et.trap_FS_FCloseFile(fileDescriptor)
for entry, adminName, adminGUID, adminLevel, adminFlags in string.gmatch(fileString, "(%[admin%]\nname%s+=%s+([%a%d%p]+)\nguid%s+=%s+([%u%d]+)\nlevel%s+=%s+([%d]+)\nflags%s+=%s+([%a%d%p]*)\n\n)") do
for _, adminName, adminGUID, adminLevel, adminFlags in string.gmatch(fileString, "(%[admin%]\nname%s+=%s+([%a%d%p]+)\nguid%s+=%s+([%u%d]+)\nlevel%s+=%s+([%d]+)\nflags%s+=%s+([%a%d%p]*)\n\n)") do
-- et.G_Print(string.format("%s %s %d %s\n", adminName, adminGUID, adminLevel, adminFlags))
if players.getGUID(clientId) == adminGUID then
@ -63,7 +62,7 @@ function commandIncognito(clientId, command)
end
end
local fileDescriptor, fileLength = et.trap_FS_FOpenFile(fileName, et.FS_WRITE)
local fileDescriptor, _ = et.trap_FS_FOpenFile(fileName, et.FS_WRITE)
local writeCount = et.trap_FS_Write(fileString, string.len(fileString), fileDescriptor)

View File

@ -24,7 +24,6 @@ local commands = require (wolfa_getLuaPath()..".commands.commands")
local players = require (wolfa_getLuaPath()..".players.players")
local pagination = require (wolfa_getLuaPath()..".util.pagination")
local settings = require (wolfa_getLuaPath()..".util.settings")
local util = require (wolfa_getLuaPath()..".util.util")
function commandListAliases(clientId, command, victim, offset)

View File

@ -35,7 +35,6 @@ function commandListLevels(clientId, command, victim, offset)
return true
else
local fileName = et.trap_Cvar_Get("g_shrubbot")
local functionStart = et.trap_Milliseconds()
local fileDescriptor, fileLength = et.trap_FS_FOpenFile(fileName, et.FS_READ)
local levelsCount = 0
@ -49,7 +48,7 @@ function commandListLevels(clientId, command, victim, offset)
et.trap_FS_FCloseFile(fileDescriptor)
for entry, levelNr, levelName, levelFlags in string.gmatch(fileString, "(%[level%]\nlevel%s+=%s+(-?[0-9]+)\nname%s+=%s+([%a%d%p ]+)\nflags%s+=%s+([%a%d%p]*)\n\n)") do
for _, levelNr, levelName, levelFlags in string.gmatch(fileString, "(%[level%]\nlevel%s+=%s+(-?[0-9]+)\nname%s+=%s+([%a%d%p ]+)\nflags%s+=%s+([%a%d%p]*)\n\n)") do
-- et.G_Print(string.format("%d %s %s\n", levelNr, levelName, levelFlags))
local numberOfSpaces = 24 - string.len(util.removeColors(levelName))
@ -94,7 +93,7 @@ function commandListLevels(clientId, command, victim, offset)
et.trap_SendConsoleCommand(et.EXEC_APPEND, "csay "..clientId.." \"^dlistlevels: ^9there are no recorded levels for player ^7"..et.gentity_get(cmdClient, "pers.netname").."^9.\";")
else
et.trap_SendConsoleCommand(et.EXEC_APPEND, "csay "..clientId.." \"^dLevels for ^7"..et.gentity_get(cmdClient, "pers.netname").."^d:\";")
for id, level in pairs(levels) do
for _, level in pairs(levels) do
et.trap_SendConsoleCommand(et.EXEC_APPEND, "csay "..clientId.." \"^f"..string.format("%4s", level["id"]).." ^7"..string.format("%-20s", util.removeColors(db.getlastalias(level["invoker_id"])["alias"])).." ^f"..os.date("%d/%m/%Y", level["datetime"]).." ^7"..level["level_id"].."\";")
end

View File

@ -16,8 +16,9 @@
-- along with this program. If not, see <http://www.gnu.org/licenses/>.
local auth = require (wolfa_getLuaPath()..".auth.auth")
local util = require (wolfa_getLuaPath()..".util.util")
local commands = require (wolfa_getLuaPath()..".commands.commands")
local game = require (wolfa_getLuaPath()..".game.game")
function commandListMaps(clientId, command)

View File

@ -19,12 +19,10 @@ local auth = require (wolfa_getLuaPath()..".auth.auth")
local commands = require (wolfa_getLuaPath()..".commands.commands")
local game = require (wolfa_getLuaPath()..".game.game")
local fireteams = require (wolfa_getLuaPath()..".game.fireteams")
local players = require (wolfa_getLuaPath()..".players.players")
local constants = require (wolfa_getLuaPath()..".util.constants")
local settings = require (wolfa_getLuaPath()..".util.settings")
local util = require (wolfa_getLuaPath()..".util.util")

View File

@ -17,7 +17,6 @@
local auth = require (wolfa_getLuaPath()..".auth.auth")
local admin = require (wolfa_getLuaPath()..".admin.admin")
local history = require (wolfa_getLuaPath()..".admin.history")
local mutes = require (wolfa_getLuaPath()..".admin.mutes")
@ -50,14 +49,16 @@ function commandMute(clientId, command, victim, ...)
end
local args = {...}
local duration, reason = 600, "muted by admin"
local duration, reason
if args[1] and util.getTimeFromString(args[1]) and args[2] then
duration = util.getTimeFromString(args[1])
reason = table.concat(args, " ", 2)
elseif args[1] and util.getTimeFromString(args[1]) then
duration = util.getTimeFromString(args[1])
reason = "muted by admin"
elseif args[1] then
duration = 600
reason = table.concat(args, " ")
elseif not auth.isPlayerAllowed(clientId, "8") then
et.trap_SendConsoleCommand(et.EXEC_APPEND, "csay "..clientId.." \"^dmute usage: "..commands.getadmin("mute")["syntax"].."\";")

View File

@ -16,8 +16,9 @@
-- along with this program. If not, see <http://www.gnu.org/licenses/>.
local auth = require (wolfa_getLuaPath()..".auth.auth")
local commands = require (wolfa_getLuaPath()..".commands.commands")
local game = require (wolfa_getLuaPath()..".game.game")
local settings = require (wolfa_getLuaPath()..".util.settings")
function commandPause(clientId, command)

View File

@ -21,8 +21,6 @@ local commands = require (wolfa_getLuaPath()..".commands.commands")
local players = require (wolfa_getLuaPath()..".players.players")
local util = require (wolfa_getLuaPath()..".util.util")
function commandPlayerLock(clientId, command, victim)
if victim == nil then
et.trap_SendConsoleCommand(et.EXEC_APPEND, "csay "..clientId.." \"^dplock usage: "..commands.getadmin("plock")["syntax"].."\";")

View File

@ -21,8 +21,6 @@ local commands = require (wolfa_getLuaPath()..".commands.commands")
local players = require (wolfa_getLuaPath()..".players.players")
local util = require (wolfa_getLuaPath()..".util.util")
function commandPlayerUnlock(clientId, command, victim)
if victim == nil then
et.trap_SendConsoleCommand(et.EXEC_APPEND, "csay "..clientId.." \"^dpunlock usage: "..commands.getadmin("punlock")["syntax"].."\";")

View File

@ -16,8 +16,9 @@
-- along with this program. If not, see <http://www.gnu.org/licenses/>.
local auth = require (wolfa_getLuaPath()..".auth.auth")
local commands = require (wolfa_getLuaPath()..".commands.commands")
local game = require (wolfa_getLuaPath()..".game.game")
local settings = require (wolfa_getLuaPath()..".util.settings")
function commandUnpause(clientId, command)

View File

@ -48,14 +48,16 @@ function commandVoiceMute(clientId, command, victim, ...)
end
local args = {...}
local duration, reason = 600, "muted by admin"
local duration, reason
if args[1] and util.getTimeFromString(args[1]) and args[2] then
duration = util.getTimeFromString(args[1])
reason = table.concat(args, " ", 2)
elseif args[1] and util.getTimeFromString(args[1]) then
duration = util.getTimeFromString(args[1])
reason = "muted by admin"
elseif args[1] then
duration = 600
reason = table.concat(args, " ")
elseif not auth.isPlayerAllowed(clientId, "8") then
et.trap_SendConsoleCommand(et.EXEC_APPEND, "csay "..clientId.." \"^dvmute usage: "..commands.getadmin("vmute")["syntax"].."\";")

View File

@ -21,7 +21,7 @@ local commands = require (wolfa_getLuaPath()..".commands.commands")
local settings = require (wolfa_getLuaPath()..".util.settings")
function commandAclListLevels(...)
function commandAclListLevels()
for _, level in ipairs(acl.getLevels()) do
et.G_Print(string.format("%5d %30s %6d players", level["id"], level["name"], level["players"]).."\n")
end

View File

@ -15,7 +15,6 @@
-- You should have received a copy of the GNU General Public License
-- along with this program. If not, see <http://www.gnu.org/licenses/>.
local util = require (wolfa_getLuaPath()..".util.util")
local events = require (wolfa_getLuaPath()..".util.events")
local settings = require (wolfa_getLuaPath()..".util.settings")

View File

@ -17,7 +17,6 @@
local players = require (wolfa_getLuaPath()..".players.players")
local constants = require (wolfa_getLuaPath()..".util.constants")
local util = require (wolfa_getLuaPath()..".util.util")
local settings = require (wolfa_getLuaPath()..".util.settings")
local tables = require (wolfa_getLuaPath()..".util.tables")

View File

@ -15,11 +15,7 @@
-- You should have received a copy of the GNU General Public License
-- along with this program. If not, see <http://www.gnu.org/licenses/>.
local teams = require (wolfa_getLuaPath()..".game.teams")
local bits = require (wolfa_getLuaPath()..".util.bits")
local constants = require (wolfa_getLuaPath()..".util.constants")
local events = require (wolfa_getLuaPath()..".util.events")
local fireteams = {}

View File

@ -15,8 +15,6 @@
-- You should have received a copy of the GNU General Public License
-- along with this program. If not, see <http://www.gnu.org/licenses/>.
local constants = require (wolfa_getLuaPath()..".util.constants")
local util = require (wolfa_getLuaPath()..".util.util")
local settings = require (wolfa_getLuaPath()..".util.settings")
local SEVERITY_LEVELS = {

View File

@ -73,7 +73,7 @@ function files.loadFromCFG(fileName, idExpr)
end
function files.saveToCFG(fileName, array)
local fileDescriptor, fileLength = et.trap_FS_FOpenFile(fileName, et.FS_WRITE)
local fileDescriptor, _ = et.trap_FS_FOpenFile(fileName, et.FS_WRITE)
local arrayCount = 0
for id, subdata in pairs(array) do
@ -83,10 +83,11 @@ function files.saveToCFG(fileName, array)
local maxKeyLength = 0
for k, v in pairs(data) do
for k, _ in pairs(data) do
maxKeyLength = math.max(maxKeyLength, string.len(k))
end
local dataLine
for k, v in pairs(data) do
dataLine = string.format("%-"..maxKeyLength.."s = %s\n", k, v)
et.trap_FS_Write(dataLine, string.len(dataLine), fileDescriptor)

View File

@ -26,7 +26,7 @@ function logs.writeChat(clientId, type, ...)
return
end
local fileDescriptor, fileLength = et.trap_FS_FOpenFile(settings.get("g_logChat"), et.FS_APPEND)
local fileDescriptor, _ = et.trap_FS_FOpenFile(settings.get("g_logChat"), et.FS_APPEND)
local logLine
@ -52,7 +52,7 @@ function logs.writeAdmin(clientId, command, victimId, ...)
return
end
local fileDescriptor, fileLength = et.trap_FS_FOpenFile(settings.get("g_logAdmin"), et.FS_APPEND)
local fileDescriptor, _ = et.trap_FS_FOpenFile(settings.get("g_logAdmin"), et.FS_APPEND)
local logLine

View File

@ -74,7 +74,7 @@ function settings.load()
end
local files = require (wolfa_getLuaPath()..".util.files")
local amount, array = files.loadFromCFG("wolfadmin.cfg", "[a-z]+")
local _, array = files.loadFromCFG("wolfadmin.cfg", "[a-z]+")
for blocksname, settings in pairs(array) do
for k, v in pairs(settings[1]) do

View File

@ -36,7 +36,7 @@ function tables.unpack(tbl)
end
function tables.contains(tbl, needle)
for key, value in pairs(tbl) do
for _, value in pairs(tbl) do
if value == needle then
return true
end

View File

@ -15,8 +15,6 @@
-- You should have received a copy of the GNU General Public License
-- along with this program. If not, see <http://www.gnu.org/licenses/>.
local constants = require (wolfa_getLuaPath()..".util.constants")
local util = require (wolfa_getLuaPath()..".util.util")
local events = require (wolfa_getLuaPath()..".util.events")
local tables = require (wolfa_getLuaPath()..".util.tables")