Re: trunk r114048: * src/eval.c (Ffuncall): Fix handling of ((lambda ..) ..) in lexically

[Dmitry Gutov]

Hi Stefan,

Stefan Monnier <address@hidden> writes:
> ------------------------------------------------------------
> revno: 114048
> revision-id: address@hidden
> parent: address@hidden
> fixes bug: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=11258

This change lowers the performance of the js2-mode parser (which still
uses dynamic scoping, by the way) by the factor of five.

In interpreted mode, when having this file opened:

http://nice-popup.googlecode.com/svn-history/r2/trunk/mootools-core-1.3-full-nocompat.js

Evaluating (js2-time (js2-reparse t))

Shows 0.9s before this revision and some 4.7s after it.

Incidentally, the slowdown factor is similar to I've seen when trying to
measure its performance in lexical-binding mode before this revision.

pre-114048, lexical-binding: t -- 3.95 seconds
   114048+, lexical-binding: t -- 7-8 seconds

Should I file a dedicated bug?