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…
Re: std.log review suspended
Am Tue, 13 Mar 2012 00:07:01 +0100 schrieb "David Nadlinger" : > 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…
Re: std.log review suspended
Am Tue, 13 Mar 2012 19:18:31 +0100 schrieb Johannes Pfau : > Am Tue, 13 Mar 2012 00:07:01 +0100 > schrieb "David Nadlinger" : > > > 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.
