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.

Sign up for GitHub

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

Jump to bottom

Reduce modifications to global lua metatables #16

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Reduce modifications to global lua metatables #16

wants to merge 3 commits into from

Conversation

daurnimator
Copy link

Work towards #3

@daurnimator daurnimator force-pushed the no-debug-setmetatable branch from 6fb45f4 to 2c0ff9e Compare March 28, 2017 00:48
@daurnimator daurnimator changed the title Don't mutate lua's function metatable Reduce modifications to global lua metatables Mar 28, 2017
@PaulBernier
Copy link
Owner

Hi daurnimator,
I wanted to thank you for your contribution and the recent issues you opened. I'm currently simply doing minimal maintenance for castl and fixing trivial bugs. I'm afraid I won't have time to review such a fundamental change in castl as it would require me to deep dive in the guts of castl and remember the ins and outs... I'm leaving this PR open for now as some people could be interested in using directly your fork (good thing you crossed referenced the issue).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@daurnimator @PaulBernier