On Sunday, 28 September 2014 at 12:24:23 UTC, Dicebot wrote:
Previous review thread :
http://forum.dlang.org/post/zhvmkbahrqtgkptdl...@forum.dlang.org
Previous voting thread (also contains discussion in the end) :
http://forum.dlang.org/post/vbotavcclttrgvzcj...@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