std.experimental.logger formal review round 3

Robert burner Schadek via Digitalmars-d digitalmars-d at puremagic.com
Sun Sep 28 07:06:13 PDT 2014


On Sunday, 28 September 2014 at 12:24:23 UTC, Dicebot wrote:
> Previous review thread : 
> http://forum.dlang.org/post/zhvmkbahrqtgkptdlcvh@forum.dlang.org
>
> Previous voting thread (also contains discussion in the end) : 
> http://forum.dlang.org/post/vbotavcclttrgvzcjjia@forum.dlang.org
>
> Code : 
> https://github.com/D-Programming-Language/phobos/pull/1500
>
> Important changes since last review:
> - new approach for compile-time log level filtering
> - thread-safe API (by Marco Leise, commits 3b32618..e71f317)
> - documentation enhancements all over the place
> - more @nogc annotations
> - "raw" log overload that expects pre-formatted message and all 
> metadata (file/line/function) as run-time function arguments
>
> (anything I have missed Robert?)

I added more unittests, but unfortunately didn't find any bugs.
The "raw" log overload is actually a template function that has 
one template parameter, the value you want to log.

That's all, I think



More information about the Digitalmars-d mailing list