Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

derive Debug for distributions #120

Closed
wants to merge 1 commit into from

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.

None yet

2 participants