Skip to content

derive Debug for distributions#120

Closed
zackmdavis wants to merge 1 commit intorust-random:masterfrom
zackmdavis:derive_debug
Closed

derive Debug for distributions#120
zackmdavis wants to merge 1 commit intorust-random:masterfrom
zackmdavis:derive_debug

Conversation

@zackmdavis
Copy link

This commit adds derivations of the Debug format trait for our
distributions, as is apparently recommended by the std::fmt
documentation ("fmt::Debug implementations should be implemented for all
public types" —https://doc.rust-lang.org/std/fmt/#fmtdisplay-vs-fmtdebug
).

This is in the matter of #118.

This commit adds derivations of the Debug format trait for our
distributions, as is apparently recommended by the std::fmt
documentation ("fmt::Debug implementations should be implemented for all
public types" —https://doc.rust-lang.org/std/fmt/#fmtdisplay-vs-fmtdebug
).

This is in the matter of rust-random#118.
@zackmdavis
Copy link
Author

(Travis CI failure is nightly only; one can only imagine these errors will need to be fixed anyway before the upstream changes in the compiler hit stable?)

@alexcrichton
Copy link
Contributor

Closing in favor of #137, but thanks forthe PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants