[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: emacs packages
From: |
Mark H Weaver |
Subject: |
Re: emacs packages |
Date: |
Wed, 17 Jun 2015 14:42:50 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Federico Beffa <address@hidden> writes:
> What do you think about the attached implementation?
[...]
> +(define (emacs-byte-compile files)
> + "Byte compile FILES, a list of file names."
> + (if (pair? files)
> + (for-each (lambda (f)
> + (let ((expr `(let ()
> + (push ,(dirname f) load-path)
> + (byte-compile-file ,f))))
> + (emacs-batch-eval expr)))
> + files)
> + (error "emacs-byte-compile failed: not a list of files!" files)))
> +
I would prefer to let an emacs expert (Alex?) comment on this, but here
are some preliminary comments:
* I wouldn't bother checking the type of 'files', since 'for-each' will
do that. Also, () is a list but not a pair.
* Maybe use 'progn' instead of 'let ()' ?
* Instead of launching a separate emacs process for each file, how about
passing the list of filenames into emacs and do the loop within emacs
itself?
Mark
- Re: emacs packages, (continued)
- Re: emacs packages, Alex Kost, 2015/06/19
- Re: emacs packages, Ludovic Courtès, 2015/06/19
- Re: emacs packages, Federico Beffa, 2015/06/19
- Re: emacs packages, Ludovic Courtès, 2015/06/21
- Re: emacs packages, Federico Beffa, 2015/06/22
- Re: emacs packages, Ludovic Courtès, 2015/06/22
- Re: emacs packages, Federico Beffa, 2015/06/23
- Re: emacs packages, Ludovic Courtès, 2015/06/23
Re: emacs packages, Mark H Weaver, 2015/06/16