Re: [PATCH] Colorized REPL

2012-12-09 Thread Nala Ginrut
On Sun, 2012-12-09 at 08:50 +0800, Daniel Hartwig wrote: On 9 December 2012 05:35, Ian Price ianpric...@googlemail.com wrote: Or, you could use the package manager I keep pimping :) Yes indeed, it works quite well. As does just adding such files to a site- or user-local module path.

Fix reader options for R6RS `get-datum'

2012-12-09 Thread Andreas Rottmann
This patch series addresses the problem that `get-datum' is using the global reader options, even for those options that have to have fixed values to make the reader behave in an R6RS-compatible way. * [PATCH 1/3] Split r6rs-ports.c according to module boundaries Needed by the last patch,

[PATCH 2/3] Add internal API to specify reader options at reader invocation

2012-12-09 Thread Andreas Rottmann
* libguile/private-options.h: Introduce a new enum indexing the read options, and use its values as indices for scm_read_opts. * libguile/read.c: Get rid of the bit field offsets #define-s, and instead use the enum values to caculate them. * libguile/read.c (enum t_keyword_style,

[PATCH 3/3] Make `get-datum' conform more closely to R6RS semantics

2012-12-09 Thread Andreas Rottmann
With Guile's default reader options, R6RS hex escape and EOL escape behavior is missing. This change enables the former via the `r6rs-hex-escapes' option, and gets us closer to the latter by setting `hungry-eol-escapes'. * libguile/r6rs-ports.c (R6RS_READ_OPTION_MASK): New macro, defines which

Re: [PATCH] Colorized REPL

2012-12-09 Thread Ludovic Courtès
Hi! Daniel Hartwig mand...@gmail.com skribis: I am not sure about the licensing there. That module is copyrighted, though GPLv3+. A merge /may/ require the original author to assign the copyright. It’s not a requirement, but we should probably try to get in touch with him (Richard Todd)

Re: [PATCH] Colorized REPL

2012-12-09 Thread Nala Ginrut
On Mon, 2012-12-10 at 00:29 +0100, Ludovic Courtès wrote: Hi! Daniel Hartwig mand...@gmail.com skribis: I am not sure about the licensing there. That module is copyrighted, though GPLv3+. A merge /may/ require the original author to assign the copyright. It’s not a requirement,