std.experimental.logger formal review round 3

[Kapps via Digitalmars-d]

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

Agreed. Logger is such a subjective module. No matter what you 
do, there is a guarantee that people will have something more 
they desire. D badly needs at least some basic logging though, 
and if people need more complex features, they then do their own. 
But so long as the default is fine for most people, which this 
looks like it should be, then having it is a valuable addition; 
it's an important module. At this point, the most useful next 
step would be experience as you mention. There might be some 
improvements desired before full Phobos integration, such as 
cleaning up the documentation for some symbols (particularly the 
12 'alias trace/info/etc(f)') by using version(D_Doc) to hide 
part of it, but that's certainly not necessary for 
std.experimental inclusion.