[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Emacs-diffs] master 05dc679: Turn on lexical-binding in json.el
From: |
Mark Oteiza |
Subject: |
Re: [Emacs-diffs] master 05dc679: Turn on lexical-binding in json.el |
Date: |
Wed, 10 Aug 2016 21:30:50 -0400 |
User-agent: |
Mutt/1.6.2+18 (69bbe4da959f) (2016-07-01) |
On 10/08/16 at 11:45am, Stefan Monnier wrote:
> > Measuring with (benchmark-run 100 (json-read-file "foobar.json"))
> > showed 12-31% reduction in execution time.
>
> Yay!
>
> I think it's the first time I see lexical-binding used for performance ;-)
json.el can use all the help it can get ;)
In any case, I was putzing around and turning on lexical-binding all
over the place and found that many files byte-compile happily with
lexical-binding enabled, or with minimal changes. Seems like a lot of
low hanging fruit to pick and I should post a patch, WDYT?
FTR I was going down the list:
head -n1 -q lisp/*.el|grep -v 'lexical-binding'
- Re: [Emacs-diffs] master 05dc679: Turn on lexical-binding in json.el, Stefan Monnier, 2016/08/10
- Re: [Emacs-diffs] master 05dc679: Turn on lexical-binding in json.el,
Mark Oteiza <=
- Re: [Emacs-diffs] master 05dc679: Turn on lexical-binding in json.el, Richard Stallman, 2016/08/11
- Re: [Emacs-diffs] master 05dc679: Turn on lexical-binding in json.el, Stefan Monnier, 2016/08/12
- [PATCH] lexical-binding mega-patch (was: Re: [Emacs-diffs] master 05dc679: Turn on lexical-binding in json.el), Mark Oteiza, 2016/08/13
- Re: [PATCH] lexical-binding mega-patch (was: Re: [Emacs-diffs] master 05dc679: Turn on lexical-binding in json.el), Eli Zaretskii, 2016/08/13
- Re: [PATCH] lexical-binding mega-patch, John Wiegley, 2016/08/14
- Re: [PATCH] lexical-binding mega-patch, Mark Oteiza, 2016/08/14
- Re: [PATCH] lexical-binding mega-patch (was: Re: [Emacs-diffs] master 05dc679: Turn on lexical-binding in json.el), Chad Brown, 2016/08/13
Re: [Emacs-diffs] master 05dc679: Turn on lexical-binding in json.el, John Wiegley, 2016/08/11