• marja's avatar
    Revert of Preparse inner functions. (patchset #23 id:440001 of... · 7de8639e
    marja authored
    Revert of Preparse inner functions. (patchset #23 id:440001 of https://codereview.chromium.org/2322243002/ )
    
    Reason for revert:
    This approach is not good - breaks when we recompile.
    
    Original issue's description:
    > Preparse inner functions.
    >
    > This is an overly pessimistic approach where PreParser only keeps
    > track of unresolved variables, but doesn't declare anything. This
    > will result in context-allocating variables in the outer function
    > unnecessarily, if the variable names clash with variable names
    > used by the inner function (even if the variables are not the
    > same). However, we have been unable to prove that this approach
    > wouldn't be good enough for the practical purposes.
    >
    > Committed: https://crrev.com/e1341ca8fa486bb2c9e4236672a64ec7756a164d
    > Cr-Commit-Position: refs/heads/master@{#39469}
    
    TBR=adamk@chromium.org,vogelheim@chromium.org,nikolaos@chromium.org,nednguyen@google.com
    # Skipping CQ checks because original CL landed less than 1 days ago.
    NOPRESUBMIT=true
    NOTREECHECKS=true
    NOTRY=true
    
    Review-Url: https://codereview.chromium.org/2349473004
    Cr-Commit-Position: refs/heads/master@{#39471}
    7de8639e
preparser.h 50.6 KB