Thread overview
std.log review suspended
Mar 12, 2012
David Nadlinger
Mar 13, 2012
Johannes Pfau
Mar 14, 2012
Johannes Pfau
March 12, 2012
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
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
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.