search

dhardy / rand

http://doc.rust-lang.org/rand
Other
2 stars 2 forks source link

Log feature: optional logging #43

Closed dhardy closed 6 years ago

dhardy commented 6 years ago

Implement #39

I'm not very happy with this code.

Maybe there should be some code to log error details including causes somewhere? Or format to one long string? (But I think it's better to output each cause separately.)

I could probably get rid of all the #[cfg(feature="log")] by re-implementing warn and other macros locally (extra boilerplate in lib.rs).

I'm also not sure how many details to print when we just delay or skip reseeding.

dhardy commented 6 years ago

Updated; this time I think it's ready for merge.