Skip to content

Recursion fix - #220

Merged
Perryvw merged 3 commits into
TypeScriptToLua:masterfrom
tomblind:recursion_fix
Oct 7, 2018
Merged

Recursion fix#220
Perryvw merged 3 commits into
TypeScriptToLua:masterfrom
tomblind:recursion_fix

Conversation

@tomblind

@tomblind tomblind commented Oct 7, 2018

Copy link
Copy Markdown
Collaborator

Separated variable assignment from declaration when assigning to function expression or arrow function to allow recursion.

fixes #187

@Perryvw

Perryvw commented Oct 7, 2018

Copy link
Copy Markdown
Member

Could you add a functional test (test/unit) that executes a recursive function?

@tomblind

tomblind commented Oct 7, 2018

Copy link
Copy Markdown
Collaborator Author

Good idea - I just added a couple.

@Perryvw
Perryvw merged commit 3c96b40 into TypeScriptToLua:master Oct 7, 2018
@tomblind
tomblind deleted the recursion_fix branch October 7, 2018 16:06
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

Successfully merging this pull request may close these issues.

Recursion doesn't work with arrow functions

2 participants