Skip to content
This repository was archived by the owner on Jun 15, 2026. It is now read-only.

Fix nested function literal. - #47

Merged
John-David Dalton (jdalton) merged 1 commit into
winjs:masterfrom
tetious:master
Jul 7, 2015
Merged

Fix nested function literal.#47
John-David Dalton (jdalton) merged 1 commit into
winjs:masterfrom
tetious:master

Conversation

@tetious

Copy link
Copy Markdown
Contributor

This violates strict-mode, and breaks Safari on MacOS/iOS.

@msftclas

Hi Greg Lincoln (@tetious), I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.

TTYL, MSBOT;

@jdalton

Copy link
Copy Markdown
Contributor

Nice catch Greg Lincoln (@tetious)!

For reference a summary of the issue with function declarations in blocks can be found at https://jslinterrors.com/function-statements-should-not-be-placed-in-blocks

John-David Dalton (jdalton) added a commit that referenced this pull request Jul 7, 2015
@jdalton
John-David Dalton (jdalton) merged commit 3d974eb into winjs:masterJul 7, 2015
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@tetious@msftclas@jdalton@AmazingJaze