Skip to content
This repository has been archived by the owner on Apr 11, 2023. It is now read-only.

Update CodeParser.js #165

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

leonvaandrager
Copy link

In strict modus Firefox adds "use strict"; to functions. So the parseBuilder will fail cause the first statement in the body is "use strict".

This is probably not the best solution, but it will do the trick.

In strict modus Firefox adds "use strict"; to functions. So the parseBuilder will fail cause the first statement in the body is "use strict".

This is probably not the best solution, but it will do the trick.
@leonvaandrager
Copy link
Author

@marklagendijk second try ;)

@marklagendijk
Copy link
Contributor

@leonvaandrager Nice! Looks very simple, and seems a good solution. However, I am not from the JayData team, myself. I am just an enthusiastic user :).

@robesz could you have this reviewed by the right person, and merged if it is ok? I think this is an very important fix:

  • Users often run into 'strange issues' caused by this problem.
  • It is a pity that you cannot use strict mode in your application, because of JayData.

Sign up for free to 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.

None yet

3 participants