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

Improved lookahead for arrow functions. #203

Merged
merged 5 commits into from Jul 25, 2014
Merged

Improved lookahead for arrow functions. #203

merged 5 commits into from Jul 25, 2014

Commits on Jul 23, 2014

  1. Improved lookahead for arrow functions. 

    Fixes issue #34.

Specifically:
    * We no longer automatically assume "()" is a function expression. It must be followed by an arrow, colon, or curly brace.
    * If an arrow is missing following a signature, but a curly brace is present, we assume the user forgot the arrow and try to parse the body anyway.
    @DanielRosenwasser
    DanielRosenwasser committed Jul 23, 2014
    Configuration menu
    Copy the full SHA
    5b6bb5b View commit details
    Browse the repository at this point in the history

  2. Covered more cases for arrow functions omitting arrows. 

    Specifically where we have a full signature followed by an open curly brace.
    @DanielRosenwasser
    DanielRosenwasser committed Jul 23, 2014
    Configuration menu
    Copy the full SHA
    5fc2792 View commit details
    Browse the repository at this point in the history

  3. Improved "certainty" when parsing arrow-function-lookin' expressions.

    @DanielRosenwasser
    DanielRosenwasser committed Jul 23, 2014
    Configuration menu
    Copy the full SHA
    57d7cf5 View commit details
    Browse the repository at this point in the history

Commits on Jul 25, 2014

  1. Addressed code review feedback.

    @DanielRosenwasser
    DanielRosenwasser committed Jul 25, 2014
    Configuration menu
    Copy the full SHA
    b76c13c View commit details
    Browse the repository at this point in the history

  2. Merge branch 'master' into arrowLookAhead

    @DanielRosenwasser
    DanielRosenwasser committed Jul 25, 2014
    Configuration menu
    Copy the full SHA
    b0c59e7 View commit details
    Browse the repository at this point in the history