The following review has been posted through the commitfest application:
make installcheck-world:  tested, passed
Implements feature:       tested, passed
Spec compliant:           not tested
Documentation:            tested, passed

The patch 8 works and addresses the things I noticed earlier.

It needs s/explicitely/explicitly/ in the docs.

The parsing of the seed involves matters of taste, I guess: if it were a signed 
int, then
sscanf's built-in %i would do everything those three explicit hex/octal/decimal 
branches
do, but there's no unsigned version of %i. Then there's strtoul(..., base=0), 
which accepts
the same choice of bases, but there's no unsigned-int-width version of that. 
Maybe it
would still look cleaner to use strtoul and just check that the result fits in 
unsigned int?
As I began, it comes down to taste ... this code does work.

I am not sure about the "idem for :random_seed" part: Does this mean that a 
value
could be given with -Drandom_seed on the command line, and become the value
of :random_seed, possibly different from the value given to --random-seed?
Is that intended? (Perhaps it is; I'm merely asking.)

-Chap

The new status of this patch is: Waiting on Author

Reply via email to