View mode: basic / threaded / horizontal-split · Log in · Help
March 12, 2012
std.log review suspended
The extended review period for std.log has ended [1], and Jose, 
the author of the proposed module, has requested some extra time 
to incorporate the suggestions made during the review without 
ending up with a butchered design. Thus, the review process has 
been suspended as to not block the queue for too long.

This also means that review can start on the next proposal 
immediately. From the records, std.uuid would be next – are we 
good to go, Johannes? Other suggestions?

On a related note, I created a Trello card to keep track of 
review manager volunteers. So, if you are interested, please add 
yourself to it so that the next review can start smoothly 
(»Phobos Review Queue« board, request access at the Phobos ML).

As always, thanks to everybody who participated,
David


[1] Or will end in a few hours, depending on how you interpret 
the messed-up date I had given in the announcement. Since this is 
not a vote anyway, I also didn't specify a time zone, so…
March 13, 2012
Re: std.log review suspended
Am Tue, 13 Mar 2012 00:07:01 +0100
schrieb "David Nadlinger" <see@klickverbot.at>:

> The extended review period for std.log has ended [1], and Jose, 
> the author of the proposed module, has requested some extra time 
> to incorporate the suggestions made during the review without 
> ending up with a butchered design. Thus, the review process has 
> been suspended as to not block the queue for too long.
> 
> This also means that review can start on the next proposal 
> immediately. From the records, std.uuid would be next – are we 
> good to go, Johannes? Other suggestions?

Give me a day or two. A recent change to dmd/phobos broke some of the
unittests, I'll fix those first.

> 
> On a related note, I created a Trello card to keep track of 
> review manager volunteers. So, if you are interested, please add 
> yourself to it so that the next review can start smoothly 
> (»Phobos Review Queue« board, request access at the Phobos ML).
> 
> As always, thanks to everybody who participated,
> David
> 
> 
> [1] Or will end in a few hours, depending on how you interpret 
> the messed-up date I had given in the announcement. Since this is 
> not a vote anyway, I also didn't specify a time zone, so…
March 14, 2012
Re: std.log review suspended
Am Tue, 13 Mar 2012 19:18:31 +0100
schrieb Johannes Pfau <nospam@example.com>:

> Am Tue, 13 Mar 2012 00:07:01 +0100
> schrieb "David Nadlinger" <see@klickverbot.at>:
> 
> > The extended review period for std.log has ended [1], and Jose, 
> > the author of the proposed module, has requested some extra time 
> > to incorporate the suggestions made during the review without 
> > ending up with a butchered design. Thus, the review process has 
> > been suspended as to not block the queue for too long.
> > 
> > This also means that review can start on the next proposal 
> > immediately. From the records, std.uuid would be next – are we 
> > good to go, Johannes? Other suggestions?
> 
> Give me a day or two. A recent change to dmd/phobos broke some of the
> unittests, I'll fix those first.

Ok, everything's working now (thanks to the folks in D.learn).

Here's a copy of the mail I sent to the phobos list some weeks ago:
________________________________________

About std.uuid (copied from the module documentation):
---------------------
This is a port of boost.uuid from the boost project with some minor
additions and API changes for a more D-like API. A UUID, or Universally
unique identifier, is intended to uniquely identify information in a
distributed environment without significant central coordination. It
can be used to tag objects with very short lifetimes, or to reliably
identify very persistent objects across a network. UUIDs have many
applications. [...]
---------------------

Code: https://github.com/jpf91/phobos/blob/std.uuid/std/uuid.d
API-Docs: http://dl.dropbox.com/u/24218791/d/src/uuid.html

Note: The code and documentation for sha1UUID has already been written,
but until phobos has support for SHA1, that can't be included. The code
is currently commented out in the source file (it's well tested
with some 3rd party SHA1 code), but the documentation for those
functions is included in the API-docs. I think those functions should
be reviewed as well, so that it's possible to add them to phobos with a
simple pull request at a later date.

Note2: std.uuid also needs this pull request:
https://github.com/D-Programming-Language/phobos/pull/398
It adds a isRandomNumberGenerator template to detect if a template
parameter is a random-number generator type.
Top | Discussion index | About this forum | D home