Skip to content
This repository has been archived by the owner on Oct 23, 2019. It is now read-only.

Remove "import" keyword #50

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

Conversation

robertleeplummerjr
Copy link
Contributor

No description provided.

@jakubmisek
Copy link
Member

We can't remove "import" in this way since it is Phalanger keyword, needed by older apps. The proper way would be to handle it in Scanner.cs, reinterpret T_IMPORT in case you don't want it (currently "import" is only interpreted as keyword if you are compiling in Pure mode)

@robertleeplummerjr
Copy link
Contributor Author

I figured it would be rejected. Loud and clear, let me see what I can roll
out in the next few minutes...

On Tue, Apr 7, 2015 at 12:36 PM, Jakub Míšek notifications@github.com
wrote:

We can't remove "import" in this way since it is Phalanger keyword, needed
by older apps. The proper way would be to handle it in Scanner.cs,
reinterpret T_IMPORT in case you don't want it (currently "import" is only
interpreted as keyword if you are compiling in Pure mode)


Reply to this email directly or view it on GitHub
#50 (comment).

Robert Plummer

@robertleeplummerjr
Copy link
Contributor Author

How about them apples?

@robertleeplummerjr
Copy link
Contributor Author

Does this implementation work for you?

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.

2 participants