Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

parser: refactor parser first step #4545

Merged
merged 4 commits into from
Sep 18, 2017
Merged

parser: refactor parser first step #4545

merged 4 commits into from
Sep 18, 2017

Commits on Sep 17, 2017

  1. parser: refactor parser first step 

    This is the first step to refactor parser. it solve several problems.

1. function name can be general identifier, we don't need to define a keyword for a builtin function.

2. write test to make sure keyword is consistent with lever token. this fixes #4538.

3. Removed ReservedKeyword, solve the identifier in scanner.

4. Simplified precedence definition.
    @coocood
    coocood committed Sep 17, 2017
    Configuration menu
    Copy the full SHA
    411e793 View commit details
    Browse the repository at this point in the history

Commits on Sep 18, 2017

  1. *: address comment

    @coocood
    coocood committed Sep 18, 2017
    Configuration menu
    Copy the full SHA
    e458b11 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'master' into coocood/parser-function

    @zz-jason
    zz-jason committed Sep 18, 2017
    Configuration menu
    Copy the full SHA
    40dd72b View commit details
    Browse the repository at this point in the history

  3. Merge branch 'master' into coocood/parser-function

    @coocood
    coocood committed Sep 18, 2017
    Configuration menu
    Copy the full SHA
    75ba877 View commit details
    Browse the repository at this point in the history