Ludovic Courtès escreveu:
Not quite actually: the hell = scm_malloc (...) bit is still broken.
?
BTW, can you please avoid pushing small topic branches like nit and
dev/with-gnulib to Savannah, as we can't distinguish them from big
branches like vm?
Sorry, - contrary to what may appear, I
Han-Wen Nienhuys escreveu:
Ludovic Courtès escreveu:
Not quite actually: the hell = scm_malloc (...) bit is still broken.
?
BTW, can you please avoid pushing small topic branches like nit and
dev/with-gnulib to Savannah, as we can't distinguish them from big
branches like vm?
Sorry,
Ludovic Courtès escreveu:
hell = scm_calloc (hell_size * sizeof (*hell));
`sizeof (*hell)' is actually `sizeof (scm_t_bits *)', which is equal
to `sizeof (scm_t_bits)', but using `sizeof (*hell)' is clearer and
less error-prone.
Besides, is that code only used when
Ludovic Courtès escreveu:
Hi Han-Wen,
Han-Wen Nienhuys [EMAIL PROTECTED] writes:
Ludovic Courtès escreveu:
is kind of hard to review in a glimpse. Does it just randomly clean
things up (whatever that means---it does not follow the GCS, for
GCS?
GNU Coding Standards, the thing we're
Ludovic Courtès escreveu:
Han-Wen Nienhuys [EMAIL PROTECTED] writes:
Running the test suite through valgrind, I get some fishy errors.
So was that fixed by this commit?
commit 51ef99f7fa9fb766fbb48619fc5863ab9914591d
Author: Han-Wen Nienhuys [EMAIL PROTECTED]
Date: Sat Aug 16
Ludovic Courtès escreveu:
(I intend to squash into a single commit before pushing to master).
First of all, thanks for your work (I know it's not so much fun to hack
the GC), but I feel unhappy with your commit to both `master' and
`branch_release-1-8'.
On the contrary, I think the GC is
Hi,
Han-Wen Nienhuys [EMAIL PROTECTED] writes:
Ludovic Courtès escreveu:
Can you be more specific about this?
Off the top of my head: incorrect indentation, missing spaces around
brackets, and more importantly comments (see (standards.info)Comments).
See below - note that the old .scm file
Ludovic Courtès escreveu:
Ludovic Courtès escreveu:
Can you be more specific about this?
Off the top of my head: incorrect indentation, missing spaces around
brackets, and more importantly comments (see (standards.info)Comments).
The code I went through should not have that; please point
On Sun 17 Aug 2008 14:51, [EMAIL PROTECTED] (Ludovic Courtès) writes:
Han-Wen Nienhuys [EMAIL PROTECTED] writes:
I have a request: can you put the gnulib into the repository of GUILE itself?
Yeah, we could do that.
I think this would be good; and call be ornery, but a simple import,
Hi,
On Sat 16 Aug 2008 11:27, Han-Wen Nienhuys [EMAIL PROTECTED] writes:
As it turns out, the test suite is loading extension modules (i18n.so
to be precise) from the path I supplied in LD_LIBRARY_PATH. Since this
was an old version, this failed in mysterious ways.
Could someone modify the
Hi Han-Wen,
On Fri 15 Aug 2008 22:15, Han-Wen Nienhuys [EMAIL PROTECTED] writes:
Running the test suite through valgrind, I get some fishy errors.
Can someone shed a light on this? The culprit seems to be
#define SCM_NUMBER_OF_SLOTS(x) \
((SCM_STRUCT_DATA (x)[scm_struct_i_n_words]) -
Good day!
Still on holiday, but the train provides a lovely hacktime.
On Mon 11 Aug 2008 01:10, [EMAIL PROTECTED] (Ludovic Courtès) writes:
Andy Wingo [EMAIL PROTECTED] writes:
The removal of the scm_i_* functions is an ABI break in the stable 1.8
series. It should be reverted. (It's a
12 matches
Mail list logo