tailhook / rust-argparse

The command-line argument parser library for rust
MIT License
240 stars 39 forks source link

Remove Copy constraint in Actions, and document weaker Clone constraint #35

Closed zrneely closed 8 years ago

zrneely commented 8 years ago

This is not a breaking change - existing code will continue working since Copy: Clone. Removes the Copy constraint on types involved with Actions, and replaces it with a weaker Clone constraint. There should be no performance impact on existing code.

Additionally, this pull request adds to the readme to show that those constraints are required.

Finally, when I was running the tests, I noticed an unused "use" statement, which I removed in an individual commit.

tailhook commented 8 years ago

Looks fine. Thanks!