Re: [PATCH] emacs: Rewrite scheme side in a functional manner.

2014-09-24 Thread Ludovic Courtès
Alex Kost skribis: > Ludovic Courtès (2014-09-21 23:27 +0400) wrote: > >> Alex Kost skribis: >> >>> Ludovic Courtès (2014-09-20 18:11 +0400) wrote: [...] >> I was thinking about things generic enough to be in (guix profiles), >> things that ‘guix package’ or guix-web could use. >> >> That said

Re: [PATCH] emacs: Rewrite scheme side in a functional manner.

2014-09-23 Thread Alex Kost
Ludovic Courtès (2014-09-21 23:27 +0400) wrote: > Alex Kost skribis: > >> Ludovic Courtès (2014-09-20 18:11 +0400) wrote: [...] >>> Overall it looks OK. I’m concerned with code duplication between emacs/ >>> and guix/, though, and there are also a few stylistic issues, and >>> missing or terse

Re: [PATCH] emacs: Rewrite scheme side in a functional manner.

2014-09-21 Thread Ludovic Courtès
Alex Kost skribis: > Ludovic Courtès (2014-09-20 18:11 +0400) wrote: [...] >> There are still a bunch of ‘set!’ and hash tables, though. ;-) > > There are only 2 ‘set!’s in: > > (define manifest->hash-table > (let ((current-manifest #f) > (current-table #f)) > (lambda (manifest)

Re: [PATCH] emacs: Rewrite scheme side in a functional manner.

2014-09-21 Thread Alex Kost
Ludovic Courtès (2014-09-20 18:11 +0400) wrote: > Alex Kost skribis: > >> From d42829fe03271e633e43cc35cf277705203e6080 Mon Sep 17 00:00:00 2001 >> From: Alex Kost >> Date: Thu, 18 Sep 2014 16:24:02 +0400 >> Subject: [PATCH 2/3] emacs: Rewrite scheme side in a functional manner. > > Good idea!

Re: [PATCH] emacs: Rewrite scheme side in a functional manner.

2014-09-20 Thread Ludovic Courtès
Alex Kost skribis: > From d42829fe03271e633e43cc35cf277705203e6080 Mon Sep 17 00:00:00 2001 > From: Alex Kost > Date: Thu, 18 Sep 2014 16:24:02 +0400 > Subject: [PATCH 2/3] emacs: Rewrite scheme side in a functional manner. Good idea! There are still a bunch of ‘set!’ and hash tables, though.