On Sun 18 Jul 2010 18:01, Andreas Rottmann <a.rottm...@gmx.at> writes:

> Here is a first version of the patch to add an external representation
> for the RNG state (needed for implementing SRFI 27 "Sources of random
> bits" in terms of Guile's RNG). I'm not quite sure about the naming of
> procedures, both one the C and the Scheme side of things -- suggestions
> highly appreciated!

Great, thanks!

There's something still that doesn't feel quite right about it -- that
the docs don't define adequately what a seed is, and these 'external'
values serve a similar purpose -- but oh well. I thought about it for 20
minutes or so and couldn't think of something better.

I replaced "external" with "datum", in your names. It felt better. Hope
that's OK.

In the future would you mind making a few test cases when you submit
bugfixes or feature patches? It is a pain, but it helps maintain Guile's
quality through time.

Thanks again!

Andy
-- 
http://wingolog.org/

Reply via email to