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.
