Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't warn about unused globals when both defined implicitly and through std #196

Open
NutchapolSal opened this issue Jul 16, 2019 · 0 comments

Comments

@NutchapolSal
Copy link

here's an example:

a game has the function foo it calls, but it lets the lua file define it

function foo()
	local x = math.random(1, 999)
	draw.bars_on_screen(x)

	x = math.some_stuff(10, x)
	draw.some_more_baz(x)
end

i have allow_defined enabled, and draw and it's fields defined in std

foo() doesn't come up in the rest of the lua file, so it now warns that foo() is an unused global

i'm suggesting that when this new option is enabled then if it is both defined in std and implicitly inside the file, then don't warn about it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant