Re: [PATCH] In string-split, add support for character sets and predicates.

2012-10-09 Thread Daniel Hartwig
On 10 October 2012 11:25, Mark H Weaver wrote: > Daniel Hartwig writes: > >> On 10 October 2012 10:14, Mark H Weaver wrote: >>> Your latest patch looks good, but I just remembered one more thing: >>> doc/ref/api-data.texi needs to be updated. >> >> I had assumed that was updated automatically fr

Re: [PATCH] In string-split, add support for character sets and predicates.

2012-10-09 Thread Mark H Weaver
Daniel Hartwig writes: > On 10 October 2012 10:14, Mark H Weaver wrote: >> Your latest patch looks good, but I just remembered one more thing: >> doc/ref/api-data.texi needs to be updated. > > I had assumed that was updated automatically from the doc strings, > which are in the right format. No

Re: [PATCH] In string-split, add support for character sets and predicates.

2012-10-09 Thread Daniel Hartwig
On 10 October 2012 10:14, Mark H Weaver wrote: > Your latest patch looks good, but I just remembered one more thing: > doc/ref/api-data.texi needs to be updated. I had assumed that was updated automatically from the doc strings, which are in the right format. ?

Re: [PATCH] Implement ‘hash’ for structs

2012-10-09 Thread Mark H Weaver
Hi Ludovic! l...@gnu.org (Ludovic Courtès) writes: > As incredible as it may seem, ‘hash’ until now always returned 263 % n > for structs, leading to interesting experiences when using structs as > hash table keys. Yes, do you remember us talking about this long ago on IRC? I wanted to fix this,

Re: [PATCH] In string-split, add support for character sets and predicates.

2012-10-09 Thread Mark H Weaver
Daniel Hartwig writes: > On 10 October 2012 01:48, Mark H Weaver wrote: >> Please drop the unneeded curly braces above. > > Are guile's macros safe to always use that way? They should be. If they aren't, that's a bug IMO. > In libguile/__scm.h > the SCM_ASSERT definition is conditional on SCM_

Re: [PATCH] In string-split, add support for character sets and predicates.

2012-10-09 Thread Daniel Hartwig
On 10 October 2012 01:48, Mark H Weaver wrote: >> + if (!SCM_CHARSETP (char_pred)) >> { >> - last_idx = idx; >> - while (idx > 0 && buf[idx-1] != SCM_CHAR(chr)) >> -idx--; >> - if (idx >= 0) >> -{ >> - res = scm_cons (sc

[PATCH] Implement ‘hash’ for structs

2012-10-09 Thread Ludovic Courtès
Hello! As incredible as it may seem, ‘hash’ until now always returned 263 % n for structs, leading to interesting experiences when using structs as hash table keys. The attached patch provides a simple hasher for structs. I’ll commit it within a few days if there are no objections. Ludo’. diff

Re: Guile 2.0.6 - some tests fail

2012-10-09 Thread Ludovic Courtès
Hi, And now with the patch. Mark H Weaver skribis: >> FAIL: ftw.test: file-system-fold: test-suite (never enter) >> FAIL: ftw.test: file-system-fold: test-suite/lib.scm (flat file) >> FAIL: ftw.test: scandir: no select >> FAIL: ports.test: %file-port-name-canonicalization: absolute >> canonica

Re: Guile 2.0.6 - some tests fail

2012-10-09 Thread Ludovic Courtès
Hi, Mark H Weaver skribis: >> FAIL: ftw.test: file-system-fold: test-suite (never enter) >> FAIL: ftw.test: file-system-fold: test-suite/lib.scm (flat file) >> FAIL: ftw.test: scandir: no select >> FAIL: ports.test: %file-port-name-canonicalization: absolute >> canonicalization from ice-9 Can

Re: [PATCH] Preserve keyword in 'syntax-rules' and 'define-syntax-rule'

2012-10-09 Thread Ludovic Courtès
Hi Mark! Makes sense to me, you can apply it. Thanks, Ludo’.

Re: [PATCH] In string-split, add support for character sets and predicates.

2012-10-09 Thread Mark H Weaver
Hi Daniel, The updated patch looks good except for a few minor nitpicks: Daniel Hartwig writes: > From 46178db9eecc9ca402d9571c3ee520074f15ef5a Mon Sep 17 00:00:00 2001 > From: Daniel Hartwig > Date: Mon, 8 Oct 2012 18:35:00 +0800 > Subject: [PATCH] In string-split, add support for character se