Fix overly strict lifetime requirements for deserialization helpers

[phil-opp]

And other improvements:

• Don't deny clippy lints to avoid breakage when new lints are created
• Add Key bound on Keyed::K so that we don't need to specify it on every usage

[phil-opp]

Happy to split this up into separate PRs if you prefer.

[codecov-commenter]

Codecov Report

Patch coverage: 66.66% and project coverage change: -0.09 ⚠️

Comparison is base (3b4f224) 70.44% compared to head (48890c5) 70.36%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #291      +/-   ##
==========================================
- Coverage   70.44%   70.36%   -0.09%     
==========================================
  Files         111      111              
  Lines       20278    20261      -17     
==========================================
- Hits        14285    14256      -29     
- Misses       5993     6005      +12     

Impacted Files	Coverage Δ
src/dds/key.rs	64.47% <ø> (ø)
src/dds/pubsub.rs	85.95% <ø> (-0.10%)	⬇️
src/dds/with_key/datareader.rs	84.41% <ø> (ø)
src/dds/with_key/datasample.rs	40.35% <0.00%> (+2.01%)	⬆️
src/dds/with_key/datasample_cache.rs	81.34% <ø> (ø)
src/dds/with_key/datawriter.rs	60.18% <ø> (ø)
src/dds/with_key/simpledatareader.rs	54.57% <ø> (ø)
src/lib.rs	100.00% <ø> (ø)
src/ros2/ros_node.rs	50.27% <ø> (+0.27%)	⬆️
src/test/test_properties.rs	100.00% <ø> (ø)
... and 1 more

... and 4 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[jhelovuo]

Thanks. This is much appreciated. It seems that specifying trait bounds on an associated type works now better than it did ~3 years ago.

[ohuopio]

Yes, these seem to be good improvements. Thank you!