std.experimental.logger formal review round 3

Dicebot via Digitalmars-d digitalmars-d at puremagic.com
Sat Jan 24 07:06:01 PST 2015


On Friday, 23 January 2015 at 20:24:54 UTC, Andrei Alexandrescu 
wrote:
> I propose we pull this in today and make it available for 2.067 
> as std.experimental.logger.
>
> We've been through a number of iterations with this and the 
> best way to move forward is to accumulate a bit of real-world 
> experience with it. Since we're deploying to std.experimental 
> there is understanding breaking changes are still possible.
>
> Dicebot, as the review manager you get to decide. What do you 
> say?
>
>
> Andrei

I was in favor of merging it long time ago. It is _your_ 
defintion of being good enough for std.experimental I am trying 
to comply now ;) Merging it right now and continuing with fixes 
as follow-up pull requests is a good idea but you must understand 
that chance of breaking chance is very high.

That said I have just now tested last version (with thread-local 
log support) and apart from one bug found it works as expected. 
Assuming you give your LGTM I will merge the PR as soon as this 
bug is fixed and PR is rebased/squashed.


More information about the Digitalmars-d mailing list