search

time-rs / time

The most used Rust library for date and time handling.
https://time-rs.github.io
Apache License 2.0
1.06k stars 261 forks source link

Implement `arbitrary::Arbitrary` trait for exported types #642

Closed OrangeTux closed 6 months ago

OrangeTux commented 6 months ago

First of all, thanks for this library!

I'm adding support for fuzzing to a private project of mine. That requires me to implement the arbitrary::Arbitrary (a crate maintained by the rust-fuzz organisation) manually for a few types of this library. I'm happy to upstream these changes into this project. Are you interested?

This library implements the quickcheck::Arbitrary trait for a few exported types. I'm thinking to implement arbitrary::Arbitrary for all types that implement quickcheck::Arbitrary.

You can find my current progress in the branch 'arbitrary-support' of my fork.

jhpratt commented 6 months ago

I previously addressed this in #502. I was open to it as part of #549, but that never happened for some reason.

OrangeTux commented 6 months ago

Thanks for linking to those tickets. It's a pity that #549 is stalled. I read through the issue, but I fail to understand the discussion. Therefore, I can't take over #549 and move it forward.

I'll close this issue