On Monday, 20 May 2013 at 21:08:36 UTC, Jesse Phillips wrote:
On Monday, 20 May 2013 at 21:01:36 UTC, Jesse Phillips wrote:
On Saturday, 18 May 2013 at 16:58:19 UTC, Idan Arye wrote:
OK, I implemented everything and made a pull request: https://github.com/D-Programming-Language/phobos/pull/1294

I've added this to the review queue.

The module is in need of documentation (publishing not required) before it can be formally reviewed.

http://wiki.dlang.org/Review_Queue

Should have been more specific with this. The documentation looks to be sparse with few examples and little introduction. I'll have to do more examining of the specific functions/classes. Due to the nature of the module I may need to reconsider how much can actually go into an introduction.

Well, obviously an example introduction like in `std.concurrency` or `std.variant` will not fit - since `std.idioms` is not centered around a single feature - but a table organized by idioms will be in order, and ofcourse - a short general description.

I'll get to that as soon as I can.

Reply via email to