Review of Jose Armando Garcia Sancio's std.log
Jose Armando Garcia
jsancio at gmail.com
Mon Feb 13 18:55:50 PST 2012
On Mon, Feb 13, 2012 at 9:37 PM, so <so at so.so> wrote:
> On Monday, 13 February 2012 at 15:50:05 UTC, David Nadlinger wrote:
>> There are several modules in the review queue right now, and to get things
>> going, I have volunteered to manage the review of Jose's std.log proposal.
>> Barring any objections, the review period starts now and ends in three
>> weeks, on March 6th, followed by a week of voting.
>> Docs: http://jsancio.github.com/phobos/phobos/std_log.html
>> Known remaining issues:
>> - Proof-reading of the docs is required.
>> - Not yet fully tested on Windows.
>> Depends on: https://github.com/D-Programming-Language/druntime/pull/141
>> (will be part of 2.058)
>> Earlier drafts of this library were discussed last year, just search the
>> NG and ML archives for "std.log".
>> I think getting this right is vitally important so that we can avoid an
>> abundance of partly incompatible logging libraries like in Java. Thus, I'd
>> warmly encourage everyone to actively try out the module or compare it with
>> any logging solution you might already be using in your project.
>> Please post all feedback in this thread, and remember: Although
>> comprehensive reviews are obviously appreciated, short comments are very
>> welcome as well!
> Good work.
> One suggestion. Instantiating a template for each log rather verbose for
> such common thing. I suggest:
> (Just to demonstrate)
> alias global_logger!sev_info info;
> alias global_logger!sev_warning warning;
> alias global_logger!sev_error error;
> alias global_logger!sev_critical critical;
> alias global_logger!sev_dfatal dfatal;
> alias global_logger!sev_fatal fatal;
> As we are pulling severity levels to global namespace anyway, this will save
> us some verbosity and the keyword "log".
I like it! I think I tried this before but I thought "compiled time"
instantiation of the templates was not working properly with alias.
Let me try it again and report back...
More information about the Digitalmars-d