Module:Sandbox/Aidan9382/CodeAnalysis - Revision history

Aidan9382: this should not be an error-generating condition

2024-08-23T07:58:16Z

this should not be an error-generating condition

← Previous revision		Revision as of 07:58, 23 August 2024
Line 243:		Line 243:
	content = content .. string.char(tonumber(num))		content = content .. string.char(tonumber(num))
	else		else
			-- ignore the \
	generateError("invalid escape sequence near '"..delim.."'")
	end		end
	end		end

2023-09-27T05:57:53Z

Fix handling of repeat statements

← Previous revision		Revision as of 05:57, 27 September 2023
Line 1,013:		Line 1,013:
	end		end
	--		--
	local st, cond = ParseExpr(~~scope~~)		local st, cond = ParseExpr(body.Scope)
	if not st then return false, cond end		if not st then return false, cond end
	--		--

2023-07-11T09:13:08Z

Slight fix to AccessedLocals

← Previous revision		Revision as of 09:13, 11 July 2023
Line 449:		Line 449:
	scope.Line = tok:Peek().Line		scope.Line = tok:Peek().Line
	function scope:GetLocal(name)		function scope:GetLocal(name)
⚫	scope.AccessedLocals[name] = true
	--first, try to get my variable		--first, try to get my variable
	local my = scope.LocalMap[name]		local my = scope.LocalMap[name]
	if my then ~~return my end~~		if my then
		⚫	scope.AccessedLocals[name] = true
			return my
			end

	--next, try parent		--next, try parent

2023-06-26T09:08:45Z

fix top comment

← Previous revision		Revision as of 09:08, 26 June 2023
Line 1:		Line 1:
	--[[		--[[
	This module is designed to perform analysis on other modules, ~~and~~ is ~~to be~~		This module is designed to perform analysis on other modules. The majority of
	~~invoked via the Debug console. The majority of~~ the code here is copied from a		the code here is copied from a seperate project,
	~~seperate project,~~ https://github.com/9382/Dump/tree/main/LuaObfusactor, which in		https://github.com/9382/Dump/tree/main/LuaObfusactor, which in turn has code
	~~turn has code~~ borrowed from https://github.com/stravant/LuaMinify.		borrowed from https://github.com/stravant/LuaMinify.
	--]]		--]]

2023-06-11T13:43:27Z

This should be a HexDigits check

← Previous revision		Revision as of 13:43, 11 June 2023
Line 228:		Line 228:
	if next == "x" then		if next == "x" then
	local n1 = get()		local n1 = get()
	if n1 == "" or n1 == delim or not ~~Digits~~[n1] then		if n1 == "" or n1 == delim or not HexDigits[n1] then
	generateError("invalid escape sequence near '"..delim.."'")		generateError("invalid escape sequence near '"..delim.."'")
	end		end
	local n2 = get()		local n2 = get()
	if n2 == "" or n2 == delim or not ~~Digits~~[n2] then		if n2 == "" or n2 == delim or not HexDigits[n2] then
	generateError("invalid escape sequence near '"..delim.."'")		generateError("invalid escape sequence near '"..delim.."'")
	end		end

2023-06-01T20:31:33Z

If nothing provided, automatically attempt the subject page

← Previous revision		Revision as of 20:31, 1 June 2023
Line 6:		Line 6:
	--]]		--]]

⚫	require("strict")
	local print = mw and mw.log or print		local print = mw and mw.log or print
		⚫	require("strict")

	local function lookupify(tb)		local function lookupify(tb)
Line 1,200:		Line 1,200:


			-- Analysis code actually begins here, the above is just the parser
	local DontIngoreReturns = {require=1}		local DontIngoreReturns = {require=1}

Line 1,211:		Line 1,212:
	out = page:getContent() or ""		out = page:getContent() or ""
	elseif obj == nil then		elseif obj == nil then
	out = mw.title.getCurrentTitle():getContent() or ""		out = mw.title.getCurrentTitle().subjectPageTitle:getContent() or ""
	else		else
	error("Unrecognised page input type '"..type(obj).."'")		error("Unrecognised page input type '"..type(obj).."'")
Line 1,302:		Line 1,303:
	--Template entry point		--Template entry point
	local function main(frame)		local function main(frame)
	local page = mw.title.getCurrentTitle().prefixedText		local page = mw.title.getCurrentTitle().subjectPageTitle.prefixedText
	page = string.gsub(page, "(.+)/doc$","%1") --strip /doc		page = string.gsub(page, "(.+)/doc$","%1") --strip /doc
	if frame.args[1] and frame.args[1] ~= "" then		if frame.args[1] and frame.args[1] ~= "" then

2023-05-17T10:07:08Z

better message

← Previous revision		Revision as of 10:07, 17 May 2023
Line 1,246:		Line 1,246:
	for global,_ in pairs(GlobalVarGetMap) do		for global,_ in pairs(GlobalVarGetMap) do
	if not rawget(_G,global) and global ~= "self" then --self picking up wrongly is an issue in the parser, and generally not an expected global name, so we lazily ignore		if not rawget(_G,global) and global ~= "self" then --self picking up wrongly is an issue in the parser, and generally not an expected global name, so we lazily ignore
	AddMessage("Entire script", "Global variable <code>"..global.."</code> was referenced or defined, yet isn't in _G")		AddMessage("Entire script", "Global variable <code>"..global.."</code> was referenced or defined, yet isn't a standard global")
	end		end
	end		end

2023-05-13T10:04:20Z

+ast check for ignored returns

@@ Line 1,199: / Line 1,199: @@
 end
 local function TryFetchPageContent(obj)
@@ Line 1,215: / Line 1,217: @@
 	return out
 end
 local function GenerateMessages(ast)
 	local messages = {}
@@ Line 1,248: / Line 1,251: @@
 	--AST based checks (E.g. ignored returns of certain calls)
 	return messages
 end

2023-05-13T09:17:20Z

centralise system so we dont have 2 copies of code, also scrap the metatable

2023-05-12T12:17:33Z

+internal function to entry points

← Previous revision		Revision as of 12:17, 12 May 2023
Line 1,326:		Line 1,326:
	return setmetatable({		return setmetatable({
	run = run,		run = run,
	main = main		main = main,
			ParseLua = ParseLua --For debugging or just general curiosity
	}, {		}, {
	__call = function(self, C)		__call = function(self, C)