www.digitalmars.com         C & C++   DMDScript  

digitalmars.D.announce - std.log review suspended

reply "David Nadlinger" <see klickverbot.at> writes:
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…
Mar 12 2012
parent reply Johannes Pfau <nospam example.com> writes:
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,=20
 the author of the proposed module, has requested some extra time=20
 to incorporate the suggestions made during the review without=20
 ending up with a butchered design. Thus, the review process has=20
 been suspended as to not block the queue for too long.
=20
 This also means that review can start on the next proposal=20
 immediately. From the records, std.uuid would be next =E2=80=93 are we=20
 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.
=20
 On a related note, I created a Trello card to keep track of=20
 review manager volunteers. So, if you are interested, please add=20
 yourself to it so that the next review can start smoothly=20
 (=C2=BBPhobos Review Queue=C2=AB board, request access at the Phobos ML).
=20
 As always, thanks to everybody who participated,
 David
=20
=20
 [1] Or will end in a few hours, depending on how you interpret=20
 the messed-up date I had given in the announcement. Since this is=20
 not a vote anyway, I also didn't specify a time zone, so=E2=80=A6
Mar 13 2012
parent Johannes Pfau <nospam example.com> writes:
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>:
=20
 The extended review period for std.log has ended [1], and Jose,=20
 the author of the proposed module, has requested some extra time=20
 to incorporate the suggestions made during the review without=20
 ending up with a butchered design. Thus, the review process has=20
 been suspended as to not block the queue for too long.
=20
 This also means that review can start on the next proposal=20
 immediately. From the records, std.uuid would be next =E2=80=93 are we=
=20
 good to go, Johannes? Other suggestions?
=20 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.=20
Mar 14 2012