Phalanger icon indicating copy to clipboard operation
Phalanger copied to clipboard

Published 20 hours ago • verified publisher icon DEVSENSE

Remove "import" keyword

Open robertleeplummerjr opened this issue 9 years ago • 4 comments

robertleeplummerjr avatar Apr 07 '15 15:04 robertleeplummerjr

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)

jakubmisek avatar Apr 07 '15 16:04 jakubmisek

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 [email protected] 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 https://github.com/DEVSENSE/Phalanger/pull/50#issuecomment-90639741.

Robert Plummer

robertleeplummerjr avatar Apr 07 '15 16:04 robertleeplummerjr

How about them apples?

robertleeplummerjr avatar Apr 07 '15 18:04 robertleeplummerjr

Does this implementation work for you?

robertleeplummerjr avatar Apr 08 '15 17:04 robertleeplummerjr